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-1160: Update Vagrantfile #1279

Merged
merged 5 commits into from
May 22, 2016
Merged

CHE-1160: Update Vagrantfile #1279

merged 5 commits into from
May 22, 2016

Conversation

TylerJewell
Copy link

@TylerJewell TylerJewell commented May 17, 2016

Moved the Vagrantfile to now launch the Che Docker container instead of Che as a server. Simpler syntax, more reliable, and operates as a Daemon by default "--restart=always" syntax. Also adds in better checks for Vagrant plugin configuration. Maps Che workspaces and projects into the Vagrant image sync'd folder. Also now lets the user set the port that the image is listening on.

Updated 4.3 documentation: https://eclipse-che.readme.io/v4.3/docs/usage-vagrant
@eivantsov @riuvshin - please approve.

Moved the Vagrantfile to now launch the Che Docker container instead of Che as a server.  Simpler syntax, more reliable, and operates as a Daemon by default "--restart=always" syntax.
@TylerJewell TylerJewell changed the title Update Vagrantfile CHE-1160: Update Vagrantfile May 17, 2016
Tyler Jewell added 2 commits May 17, 2016 14:18
1. Added check for vagrant proxy plugin install.
2. Writes proxy configuration details into che.properties if they are present.
3. Launches Che container with provided che.properties instead of default.
4. Give the user the ability to change the port that the VM is listening on and map it to internal port 8080
@codenvy-ci
Copy link

Build # 608 - FAILED

Please check console output at http://ci.codenvy-dev.com/jenkins/job/che-pullrequests-build/608/ to view the results.

@codenvy-ci
Copy link

Build # 614 - FAILED

Please check console output at http://ci.codenvy-dev.com/jenkins/job/che-pullrequests-build/614/ to view the results.

@riuvshin
Copy link
Contributor

LGTM

Added port mapping capability.
@TylerJewell TylerJewell merged commit a5791d1 into master May 22, 2016
@TylerJewell TylerJewell deleted the CHE-1160 branch May 22, 2016 17:18
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.

None yet

3 participants