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.

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

CHE-661 unzip in quiet mode #587

Merged
merged 1 commit into from
Mar 1, 2016
Merged

CHE-661 unzip in quiet mode #587

merged 1 commit into from
Mar 1, 2016

Conversation

benoitf
Copy link
Contributor

@benoitf benoitf commented Mar 1, 2016

Change-Id: If3a51cc6e01e243c47033d4d891858e3997ab725
Signed-off-by: Florent BENOIT fbenoit@codenvy.com

@benoitf
Copy link
Contributor Author

benoitf commented Mar 1, 2016

I can issue PR on erb files

@garagatyi
Copy link

Why you want to make it quiet?

@riuvshin
Copy link
Contributor

riuvshin commented Mar 1, 2016

@garagatyi on a meeting we've decided to get rid of that useless output when ws is starting.

@benoitf
Copy link
Contributor Author

benoitf commented Mar 1, 2016

@garagatyi because it looks ugly to see that when we start agent and if unzip fails we will have error BTW

@garagatyi
Copy link

than ok

@garagatyi
Copy link

LGTM

Change-Id: If3a51cc6e01e243c47033d4d891858e3997ab725
Signed-off-by: Florent BENOIT <fbenoit@codenvy.com>
@benoitf benoitf merged commit 48a2e54 into master Mar 1, 2016
@benoitf benoitf deleted the CHE-661 branch March 5, 2016 07:37
Katka92 pushed a commit to Katka92/che that referenced this pull request Jun 27, 2018
Use subject of a user that started workspace instead of current
subject if user ID of current subject differ from user ID
of the user that started the workspace.
Rework logging of a warning to be thrown in case when current
user ID differs from a workspace starter user ID and there is no
subject of the user that started workdpace in subjects cache.

Signed-off-by: Oleksandr Garagatyi <ogaragat@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants