-
Notifications
You must be signed in to change notification settings - Fork 426
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
Merge CRD objects into each other #1107
Comments
I'm not aware of anything in controller-tools that would help here |
IHMO it should be closed as won't to do |
Agree. I think there is nothing we can/should implement in controller-tools |
@sbueringer: Closing this issue. In response to this:
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-sigs/prow repository. |
Hi,
I’m currently working on reusing a struct that defines a container configuration for both jobs and deployments. Since these configurations can have different values (e.g., the image field), I structured it like this:
The goal is as follows:
If Container specifies
image:abc
and AdminDeploymentContainer specifiesimage: xyz
, the admin deployment should run withxyz
, while everything else usesabc
.I was considering adding a Merge function to handle these cases, in addition to the existing DeepCopyInto function.
Does this approach make sense, or is there an existing PR or issue I might have overlooked?
The text was updated successfully, but these errors were encountered: