-
Notifications
You must be signed in to change notification settings - Fork 77
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
ImageUpdateAutomation misinterprets Registry URL when port is defined #858
Comments
How about the statuses of |
I would guess the root cause is this code. The good thing is that only the status file is affected but all other functionality should still work as expected. |
Here status from 'ImageRepositoty' and 'ImagePolicy', its seem correctly tag image with 0.1.x.
I cant tell its related but whenever new image tag being push its not automatically update the deployments. |
FluxCD's ImageUpdateAutomation is incorrectly parsing the image repository URL when using a local registry (registry.localhost:5000). Instead of properly recognizing the repository and tag, it misinterprets the port number as part of the image name.
To Reproduce
Set up a local registry: http://registry.localhost:5000
Push an image: registry.localhost:5000/sample-web:0.1.0
Define the following ImageRepository:
Expected:
The tag should be 0.1.1, and the repository should remain registry.localhost:5000/sample-web.
Actual:
Flux treats 5000/sample-web as the repository, misplacing the port number.
Expected behavior
FluxCD should correctly interpret the image repository as registry.localhost:5000/sample-web without breaking the tag format.
Environment
FluxCD Version: 2.5.1
Kubernetes Version: v1.32.2
Additional Context:
I'm using k3d to setup kubernetes local cluster
Running Flux with flux-system namespace
Local registry is accessible and works with docker pull registry.localhost:5000/sample-web:0.1.0
Issue persists even with allowInsecure: true
Changing localhost to registry.localhost in /etc/hosts does not fix the issue
Is this a known issue, and is there a workaround?
The text was updated successfully, but these errors were encountered: