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
Persist userlist and PGB config on controller restart #24
Conversation
…rt to be more detailed
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.
Does killing the controller pod (to force it to be recreated) stop the test run?
@@ -68,7 +68,7 @@ commands = | |||
description = Run all integration tests | |||
deps = | |||
pytest | |||
juju | |||
juju==2.9.11 # juju 3.0.0 has issues with retrieving action results |
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.
JFMI, why do we lock such an old juju version? 2.9.37 is the latest as for today.
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.
This is the last 2.9 version of libjuju, which doesn't follow the same version as juju :)
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.
I have single question as JFMI only. LGTM. Tnx!
46f1145
…om/canonical/pgbouncer-k8s-operator into persist-data-on-controller-restart
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.
Looks sane and remind me that we need to take a look at cases like this on mysqland
Proposal
When we restart the k8s controller on which a PGB operator runs, we lose our containers and therefore our config. This PR checks the peer databag for config files if they aren't available in the filesystem.
Context
Release Notes
Testing