-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Refactor workspace infrastructure provisioning #3577
Conversation
7846ce5
to
054b0f7
Compare
Build # 1490 - FAILED Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/1490/ to view the results. |
ci-build |
Build success. https://ci.codenvycorp.com/job/che-pullrequests-build/1513/ |
/** | ||
* Modifies machine config and internal machine representation with everything needed for infrastructure of workspace. | ||
* | ||
* @param machineConfig configuration of machine |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please fix formatting here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, thanks
054b0f7
to
0f042d7
Compare
ci-build |
Build # 1531 - FAILED Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/1531/ to view the results. |
b56433c
to
198ef45
Compare
Build # 1536 - FAILED Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/1536/ to view the results. |
Build # 1542 - FAILED Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/1542/ to view the results. |
412e5d2
to
3cacbf7
Compare
Build success. https://ci.codenvycorp.com/job/che-pullrequests-build/1568/ |
Move agents volume addition into infrastructure provisioner. Suppress some false positive warnings. Add some javadocs. Signed-off-by: Alexander Garagatyi <agaragatyi@codenvy.com>
3cacbf7
to
f4a0e16
Compare
Build # 1575 - FAILED Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/1575/ to view the results. |
What does this PR do?
Move agents volume addition into infrastructure provisioner.
Suppress some false positive warnings.
Add some javadocs.
What issues does this PR fix or reference?
Related to codenvy/codenvy#1149
Tests written?
No