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

Wait for nodes to be ready before proceeding with upgrade. #3190

Merged
merged 1 commit into from
Jan 26, 2017

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Jan 26, 2017

Backport from master. Without this you can end up with all nodes 'NotReady' for a brief period.

Near the end of node upgrade, we now wait for the node to report Ready
before marking it schedulable again. This should help eliminate delays
when pods need to relocate as the next node in line is evacuated.

Happens near the end of the process, the only remaining task would be to
mark it schedulable again so easy for admins to detect and recover from.

( discussed pulling this in with @sdodson )

Near the end of node upgrade, we now wait for the node to report Ready
before marking it schedulable again. This should help eliminate delays
when pods need to relocate as the next node in line is evacuated.

Happens near the end of the process, the only remaining task would be to
mark it schedulable again so easy for admins to detect and recover from.
@tremble
Copy link
Contributor Author

tremble commented Jan 26, 2017

aos-ci-test

@detiber
Copy link
Contributor

detiber commented Jan 26, 2017

aos-ci-test

@openshift-bot
Copy link

@sdodson sdodson merged commit cb47a0b into openshift:release-1.4 Jan 26, 2017
@sdodson
Copy link
Member

sdodson commented Jan 26, 2017

Andrew is fixing travis jobs on release-1.4 and release-1.3 real soon now.

@tremble tremble deleted the 1_4_wait_for_nodes branch January 26, 2017 15:58
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.

5 participants