-
Notifications
You must be signed in to change notification settings - Fork 88
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
Allow adding additional containers to a pod. #2211
base: main
Are you sure you want to change the base?
Conversation
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
Need to be able to add more than just a container. Need to be able to add a (shared) volumemount to the main container and a volume to the pod. Environment variables too. |
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.
Those changes shouldn't be required. You can already generate the default cluster spec, modify it and then create the FDB cluster from this spec. That's basically what the default CreateFdbCluster
(see: https://github.com/FoundationDB/fdb-kubernetes-operator/blob/main/e2e/fixtures/factory.go#L167-L174) is doing.
Is there any need to expose additional fields like this?
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
Thanks for the suggestion. Turns out I can't add volumemounts+envars to the main container; I can only do what is in ContainerOverrides which does not include envvars or volumemounts. Restored patch before this one...... which allows me do volumemounts, volumes, envvar, and containers (You probably have a cleaner way of doing what I want). |
E.g. In tests, I want to optionally add a sidecar. * e2e/fixtures/cluster_config.go Add new AddtionalContainers member. * e2e/fixtures/fdb_cluster_specs.go Add AddtionalContainers to the spec if present.
main container. Also make it so can optionally add a volume to the pod. * e2e/fixtures/cluster_config.go Add AdditionalEnvVars, AdditionalVolumeMounts, and AdditionalVolumes to the ClusterConfig. * e2e/fixtures/fdb_cluster_specs.go Add inserting env, volumemounts, and volumes if any.
3391cc6
to
ae6ad2b
Compare
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
|
E.g. In tests, I want to be able to optionally add a sidecar.
e2e/fixtures/cluster_config.go Add new AddtionalContainers member.
e2e/fixtures/fdb_cluster_specs.go Add AddtionalContainers to the spec if present.