Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

No che-server logs in logs folder #3913

Closed
garagatyi opened this issue Jan 26, 2017 · 2 comments · Fixed by #3926
Closed

No che-server logs in logs folder #3913

garagatyi opened this issue Jan 26, 2017 · 2 comments · Fixed by #3926
Assignees
@benoitf @riuvshin
Labels
kind/bug Outline of a bug - must adhere to the bug report template.

Comments

@garagatyi
Copy link

New Che CLI creates logs folder in instance folder but doesn't write there logs of che-server.
Also there is folder /data/logs where logs of machines are stored.
For example Codenvy CLI stores logs of both in /logs/codenvy.

Che version: 5.2.0-SNAPSHOT

OS and version: Ubuntu 16.04

Docker version: 1.12.5
@benoitf benoitf self-assigned this Jan 26, 2017
@benoitf benoitf added the status/open-for-dev An issue has had its specification reviewed and confirmed. Waiting for an engineer to take it. label Jan 26, 2017
@riuvshin
Copy link
Contributor

it seems this is simple issue, seems we just need mounts for logs in compose file

benoitf added a commit that referenced this issue Jan 27, 2017
@benoitf
fix #3913 : che-server should reuse property of CHE_LOGS_DIR if exter…
31b1d13 
…nally defined

Change-Id: I73b8beed66b63f5368edb7b607fbdd96cbf69d93
Signed-off-by: Florent BENOIT <[email protected]>
@benoitf benoitf mentioned this issue Jan 27, 2017
Merged
@benoitf benoitf added status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. and removed status/open-for-dev An issue has had its specification reviewed and confirmed. Waiting for an engineer to take it. labels Jan 27, 2017
@benoitf benoitf added this to the 5.2.0 milestone Jan 27, 2017
@benoitf
Copy link
Contributor

benoitf commented Jan 27, 2017

@garagatyi I've made a pull request

@benoitf benoitf added the kind/bug Outline of a bug - must adhere to the bug report template. label Jan 27, 2017
benoitf added a commit that referenced this issue Jan 27, 2017
@benoitf
fix #3913 : che-server should reuse property of CHE_LOGS_DIR if exter…
02f91a3 
…nally defined

Change-Id: I73b8beed66b63f5368edb7b607fbdd96cbf69d93
Signed-off-by: Florent BENOIT <[email protected]>
@benoitf benoitf removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Jan 27, 2017
@JamesDrummond JamesDrummond removed this from the 5.2.0 milestone Feb 2, 2017
JPinkney pushed a commit to JPinkney/che that referenced this issue Aug 17, 2017
@benoitf
fix eclipse-che#3913 : che-server should reuse property of CHE_LOGS_D…
c3b6183 
…IR if externally defined

Change-Id: I73b8beed66b63f5368edb7b607fbdd96cbf69d93
Signed-off-by: Florent BENOIT <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@benoitf benoitf

@riuvshin riuvshin

Labels
kind/bug Outline of a bug - must adhere to the bug report template.
Projects
None yet
Milestone
No milestone
4 participants
@benoitf @riuvshin @garagatyi @JamesDrummond