Skip to content
This repository was archived by the owner on Mar 26, 2026. It is now read-only.

Add method to skip wrong reference format#54

Open
paretl wants to merge 2 commits intoindeedeng:mainfrom
paretl:check-ref-format
Open

Add method to skip wrong reference format#54
paretl wants to merge 2 commits intoindeedeng:mainfrom
paretl:check-ref-format

Conversation

@paretl
Copy link
Copy Markdown

@paretl paretl commented Mar 26, 2025

Observation
When an image reference format is wrong, the mutating webhook is failing and the deployment cannot go further. Even if the regex does not include this image.

I think that it should be great to skip that case and let the mutating webhook deploies this deployment anyway.

I detected that in a case where ArgoCD + Image updater need to deploy a pod without tag before finding / applying a new tag. So without this the ArgoCD deployments could be broken

Solution

Add a method to verify the reference format, if it's wrong, just skip the mutating webhook changes and return the same imageRef

Tests

  • unit test for this case is done
  • I have tested it in a real environment and it works as expected

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant