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

CODENVY-413: Make it possible to mark MachineNode as 'sheduled for maintenace' #1705

Merged
merged 1 commit into from
Jul 14, 2016

Conversation

mmorhun
Copy link
Contributor

@mmorhun mmorhun commented Jul 11, 2016

This PR adds new parameters to build docker image method.

@mmorhun
Copy link
Contributor Author

mmorhun commented Jul 11, 2016

@garagatyi review please

private List<File> files;
private String dockerfile;
private String remote;
private Boolean q;

Choose a reason for hiding this comment

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

it is not natural to have such a field in Java. Rename it to something meaningful

@codenvy-ci
Copy link

@codenvy-ci
Copy link

Build # 1191 - FAILED

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

@codenvy-ci
Copy link

@mmorhun mmorhun changed the title CODENVY-413: Make it possible to mark MachineNode as 'sheduled for maintenace' and not start new containers on it CODENVY-413: Make it possible to mark MachineNode as 'sheduled for maintenace' Jul 13, 2016
@codenvy-ci
Copy link

Build # 1200 - FAILED

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

@codenvy-ci
Copy link

@garagatyi
Copy link

LGTM

@mmorhun
Copy link
Contributor Author

mmorhun commented Jul 14, 2016

@sleshchenko take a look please

@sleshchenko
Copy link
Member

OK for me

Signed-off-by: Mykola Morhun <mmorhun@codenvy.com>
@codenvy-ci
Copy link

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.

4 participants