Switch ovn-controller to StatefulSets #183
Conversation
|
Skipping CI for Draft Pull Request. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: booxter The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
Note: most of the patches here come from a different "dependent" series posted elsewhere. (Github doesn't support dependencies, sorry.) |
This allows to get rid of a hostMount shared between the job pod and the main ovsdb-server pod container (to enable communication of vsctl command with the database socket). Getting rid of hostMounts is needed to be able to eventually stop running ovn-controller pods as privileged containers.
This is to prove that this is possible, now that configJob is squashed into the main ovn-controller pod.
We log to stdout. Nothing else is needed.
This directory can be local to the container and does not need to persist. Note the rundir directory is not present in the image, so we have to switch OVN_RUNDIR to point to /tmp, same as we do for other ovn services managed by the operator.
I am not aware of anything using the directory.
TODO: - anti-affinity rule - expose storageClass? - maintain the proper number of replicas
booxter
force-pushed
the
statefulset-for-ovn-controller
branch
from
583213f to
89c523f
Compare
|
/ok-to-test |
|
@booxter: The following tests failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
|
PR needs rebase. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
No description provided.