-
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
If the database is unavailable and caching is enabled, allow operator to proceed #1932
If the database is unavailable and caching is enabled, allow operator to proceed #1932
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
|
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.
The failure that I'm seeing is quite interesting and I have to spend some more time on this to fix it.
ed5882e
to
e812e19
Compare
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
… the operator to proceed and just try to refetch the machine-readable status later again
e812e19
to
f885be1
Compare
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.
LGTM aside from the one comment!
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
… Pods are deleted
Seems like we are hitting this issue: apple/foundationdb#11104 I try to back port that change to 6.2 (which we use right now), to see it that solve the issue. In that case I will back port that change in FDB. |
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
|
Description
Fixes: #1931
Type of change
Please select one of the options below.
Discussion
See the issue. The idea is to allow the operator to proceed and try to fetch the machine-readable status in the later steps again. This could resolve cases where multiple Pods are down and they must be recreated.
Testing
Have to add an e2e test for this.
Documentation
Follow-up