Skip to content
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

Add an annotation / label to Pods with their old image reference when mutating #2754

Closed
rdecuir opened this issue Jul 23, 2024 · 0 comments · Fixed by #2755
Closed

Add an annotation / label to Pods with their old image reference when mutating #2754

rdecuir opened this issue Jul 23, 2024 · 0 comments · Fixed by #2755
Labels
enhancement ✨ New feature or request

Comments

@rdecuir
Copy link

rdecuir commented Jul 23, 2024

In some cases, like controllers/operators etc, we might not know all the dependencies as far as images, even after using the provided tools like zarf dev find-images. In our case, we found images that were dependencies but what what repo we should be point to. The initial attempts were pointing at a repo that didnt match what the pod expected, but we couldnt see that due to the mutating webhook effect on the pods spec.

Adding a label or annotation to the pods which would state the original image repo, version, and name that the pods would be looking for before the mutating webhook changed it to point to the internal image repo would be useful. This would make it extremely easy to find what images are missing and add those missing images to their zarf.yaml.

Discussed in your discord, I originally thought there might be a command to reverse the UID, but its a one way hash so options are limited. Alternate option which I will add here for historical value, was given in the discord which is helpful:
"You can check if an image name you have matches the hash through the command zarf internal crc32 my_image at least"

@rdecuir rdecuir added the enhancement ✨ New feature or request label Jul 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ✨ New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant