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

Start eclipse/che-server docker image as root by default #3297

Merged
merged 1 commit into from
Dec 7, 2016
Merged

Conversation

benoitf
Copy link
Contributor

@benoitf benoitf commented Dec 6, 2016

What does this PR do?

Start eclipse/che-server docker image as root by default

What issues does this PR fix or reference?

#3276

Previous behavior

User user had to be in docker group but with a specific gid which is not always the case

New behavior

Start che server as root user by default

Use root user by default to start che-server
another PR is about specifying how to run che-server with a custom uid/gid
#3265

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

Use root user by default to start cha-server
another PR is about specifying how to run cha-server with a custom uid/gid

Change-Id: Ib2635dc9b8364b92caa768dc8e4de0603cbcf14f
Signed-off-by: Florent BENOIT <fbenoit@codenvy.com>
@benoitf benoitf added this to the 5.0.0-M8 milestone Dec 6, 2016
@benoitf benoitf added kind/enhancement A feature request - must adhere to the feature request template. sprint/current status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. labels Dec 6, 2016
@benoitf benoitf self-assigned this Dec 6, 2016
@skabashnyuk
Copy link
Contributor

What I need to rebuild to test?

@benoitf
Copy link
Contributor Author

benoitf commented Dec 6, 2016

@skabashnyuk here are the steps:

$ git pull branch CHE-3276
$ cd assembly/assembly-main
$ mvn clean install

# go back to root folder
$ cd ../../
$ docker build -t eclipse/che-server:nightly .

and then launch cli

@@ -121,10 +121,6 @@ init() {
export CHE_WORKSPACE_STORAGE_CREATE_FOLDERS=false
fi

# Ensure that the user "user" has permissions for CHE_HOME and CHE_DATA
sudo chown -R user:user ${CHE_HOME}
sudo chown -R user:user ${CHE_DATA}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@benoitf I think that these may be needed for the workspaces (that are still run as user user)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@l0rd issue is that user 'user' may have another uid so it doesn't work as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@benoitf that makes sense 👍

@TylerJewell
Copy link

What about agents and other software that you want to install outside of /projects?

@benoitf
Copy link
Contributor Author

benoitf commented Dec 6, 2016

agents need to have sudo access so they have root access to do what they want. ?

@TylerJewell
Copy link

yep - got it.

@codenvy-ci
Copy link

@benoitf
Copy link
Contributor Author

benoitf commented Dec 7, 2016

@skabashnyuk do you have tested ?

@benoitf benoitf merged commit e5abb1e into master Dec 7, 2016
@benoitf benoitf deleted the CHE-3276 branch December 7, 2016 09:44
@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 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A feature request - must adhere to the feature request template. sprint/current
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants