Skip to content

Fix initialization of servers statuses by DockerInfrastructure#9777

Merged
mkuznyetsov merged 3 commits intomasterfrom
che-9094
May 31, 2018
Merged

Fix initialization of servers statuses by DockerInfrastructure#9777
mkuznyetsov merged 3 commits intomasterfrom
che-9094

Conversation

@mkuznyetsov
Copy link
Copy Markdown
Contributor

@mkuznyetsov mkuznyetsov commented May 23, 2018

What does this PR do?

Machine servers will have "UNKNOWN" status, instead of null, when it is not being set by server checks/liveness probes

What issues does this PR fix or reference?

#9094

Release Notes

Docs PR

@mkuznyetsov mkuznyetsov requested a review from garagatyi as a code owner May 23, 2018 09:37
@mkuznyetsov mkuznyetsov requested a review from sleshchenko May 23, 2018 09:38
@benoitf benoitf added status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. kind/bug Outline of a bug - must adhere to the bug report template. labels May 23, 2018
@mkuznyetsov
Copy link
Copy Markdown
Contributor Author

mkuznyetsov commented May 23, 2018

ci-test

@codenvy-ci
Copy link
Copy Markdown

codenvy-ci commented May 23, 2018

ci-test build report:
Build details
Test report
selenium tests report data
docker image: eclipseche/che-server:9777
https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

@mkuznyetsov
Copy link
Copy Markdown
Contributor Author

mkuznyetsov commented May 24, 2018

ci-test

@codenvy-ci
Copy link
Copy Markdown

codenvy-ci commented May 24, 2018

ci-test build report:
Build details
Test report
selenium tests report data
docker image: eclipseche/che-server:9777
https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

@mkuznyetsov
Copy link
Copy Markdown
Contributor Author

mkuznyetsov commented May 29, 2018

ci-test

@codenvy-ci
Copy link
Copy Markdown

codenvy-ci commented May 29, 2018

ci-test build report:
Build details
Test report
selenium tests report data
docker image: eclipseche/che-server:9777
https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

@mkuznyetsov
Copy link
Copy Markdown
Contributor Author

mkuznyetsov commented May 30, 2018

ci-test

@codenvy-ci
Copy link
Copy Markdown

codenvy-ci commented May 30, 2018

ci-test build report:
Build details
Test report
selenium tests report data
docker image: eclipseche/che-server:9777
https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

@mkuznyetsov mkuznyetsov merged commit fc15631 into master May 31, 2018
@mkuznyetsov mkuznyetsov deleted the che-9094 branch May 31, 2018 06:55
@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 Jun 4, 2018
@benoitf benoitf added this to the 6.7.0 milestone Jun 4, 2018
hbhargav pushed a commit to hbhargav/che that referenced this pull request Dec 5, 2018
@mkuznyetsov
Fix initialization of servers statuses by DockerInfrastructure (eclip…
31ccb65 
…se-che#9777)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sleshchenko sleshchenko sleshchenko approved these changes

+1 more reviewer

@garagatyi garagatyi garagatyi approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

kind/bug Outline of a bug - must adhere to the bug report template.

Projects

None yet

Milestone

6.7.0

Development

Successfully merging this pull request may close these issues.

5 participants

@mkuznyetsov @codenvy-ci @garagatyi @sleshchenko @benoitf