-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Support the ability to replace the image registry domain #4414
Comments
@vsabella: This issue is currently awaiting triage. SIG CLI takes a lead on issue triage for this repo, but any Kubernetes member can accept issues by applying the The 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. |
Thank you for submitting your issue. I was wondering if you could provide some more detail about what you are trying to do with option 1, perhaps with an example? I am open to the suggestion of a new field in I'm also curious if your use case can be resolved by the existing
and a ConfigMap with the following:
You could write a kustomization file with the following:
The biggest downside with this approach is its verbosity and that you would have to list every container individually. This will be fixed by #4053 which will allow you to select every element in an array without having to list them out individually. If you have any other feedback that might make this approach work for you, please let us know. That being said, a new field targeted specifically for images seems like a reasonable alternative, especially considering how popular the feature is and that replacements is still new and working out its kinks. /kind feature |
@natasha41575 Thank you so much for your quick reply! I think our use-case may be solved by the replacements feature - let me try it and see if it works. Since each container right now consists of just two images it might be OK even with the limitation fixed by #4053 , I'll try this week. Generically all we are looking to do is replace every image in every Deployment/Daemonset/Etc that matches
But in a single component without having to use the "images:" feature in every single one of our deployments. |
@vsabella: This issue is currently awaiting triage. SIG CLI takes a lead on issue triage for this repo, but any Kubernetes member can accept issues by applying the The 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. |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle rotten |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /close |
@k8s-triage-robot: 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/test-infra repository. |
any update on this? |
kustomize/api/types/image.go
Line 8 in f61b075
In our usage of Kustomize, we use two different registries for images depending on the deployment region (due to use of sovereign clouds).
We are looking to simplify our kustomize so we can have a singe component to replace all instances of with in a single place, instead of the newName tag everywhere for our hundreds of containers.
We're happy to submit a patch but are looking for the best way to solve this problem that is in line with Kustomize philosophy.
We could amend the image.go to support a new field (newDomain) and update the processing of image: newImage/NewDigest/newTag to also support newRegistry by processing image tags into the format specified by cncf: (https://github.com/distribution/distribution/blob/main/reference/regexp.go)
We could add regex capture group / capture replace support to the image replacement - so we can capture part of the old value and re-use it in the new value.
Are there any other options that we haven't considered?
Thanks!
The text was updated successfully, but these errors were encountered: