-
Notifications
You must be signed in to change notification settings - Fork 32
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
Switch ovn-controller to StatefulSets #183
base: main
Are you sure you want to change the base?
Switch ovn-controller to StatefulSets #183
Commits on Jan 4, 2024
-
Squash configJob into ovn-controller pod
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.
Configuration menu - View commit details
-
Copy full SHA for 915b67b - Browse repository at this point
Copy the full SHA 915b67bView commit details -
Switch ovn-controller:var-run volume to emptyDir
This is to prove that this is possible, now that configJob is squashed into the main ovn-controller pod.
Configuration menu - View commit details
-
Copy full SHA for 97f766c - Browse repository at this point
Copy the full SHA 97f766cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 4d4147d - Browse repository at this point
Copy the full SHA 4d4147dView commit details -
Remove log hostpath mounts for ovn-controller
We log to stdout. Nothing else is needed.
Configuration menu - View commit details
-
Copy full SHA for 7572c99 - Browse repository at this point
Copy the full SHA 7572c99View commit details -
Remove /var/run/ovn hostPath mount
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.
Configuration menu - View commit details
-
Copy full SHA for 87ca3ca - Browse repository at this point
Copy the full SHA 87ca3caView commit details -
Remove var-lib hostMount from ovn-controller pod
I am not aware of anything using the directory.
Configuration menu - View commit details
-
Copy full SHA for a76e967 - Browse repository at this point
Copy the full SHA a76e967View commit details -
Configuration menu - View commit details
-
Copy full SHA for 33d49aa - Browse repository at this point
Copy the full SHA 33d49aaView commit details -
Switch to StatefulSet for ovn-controller pod
TODO: - anti-affinity rule - expose storageClass? - maintain the proper number of replicas
Configuration menu - View commit details
-
Copy full SHA for 89c523f - Browse repository at this point
Copy the full SHA 89c523fView commit details