-
Notifications
You must be signed in to change notification settings - Fork 86
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
Initial changes for new maintenance mode integration #1967
Initial changes for new maintenance mode integration #1967
Conversation
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
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.
first pass, I will need to do a second tomorrow
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
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.
Minor comment nits, but the functionality looks good to me!
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
…tting the maintneance mode
01b7097
to
03cbe3f
Compare
I'll take a look at this failure 👀 |
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
There was a single failure:
It seems like the replacement Pod took longer than expected to come up. |
* Initial changes for new maintenance mode integration
Description
This PR provides changes for a better integration with the maintenance mode for the operator. I'll provide more information in the docs.
fixes: #1656
fixes: #1655
fixes: #1654
fixes: #1650
Type of change
Please select one of the options below.
Discussion
Testing
Manual testing. I added a new e2e test for this and added a new HA test with maintenance mode enabled.
I will also add some additional unit tests to the once I already added.
Documentation
Will be updated before merging.
Follow-up