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

After successfully updating the image, the Helm values passed in the application are reset #692

Open
MairaTariq16 opened this issue Mar 19, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@MairaTariq16
Copy link

MairaTariq16 commented Mar 19, 2024

Describe the bug
After successfully updating the image, the Helm values passed in the application are reset.

To Reproduce
Steps to reproduce the behavior:

  1. Create an Argo Application resource with Helm as source. Pass values in valuesObject. Annotate with "argocd-image-updater.argoproj.io/image.update-strategy": "digest" and other relevant annotations for image-updater.
  2. Sync the application, resources are rendered successfully.
  3. Push a new image on same tag. Argocd-image-updater successfully detects the change but resets the values passed to Helm chart in Application spec.

Expected behavior
The Helm values passed in the Application spec should remain unchanged after updating the image.

Additional context
I assume image-updater updates the resources by passing image.tag as parameter to helm, and dismisses the values set previously. I have tried to pass values as parameters in Application spec and it worked fine. Works with values passed too. But I want to resort back to using valuesObject to specify values for my Helm release.

Version
ArgoCD: 2.10.2
Argocd-image-updater: 0.12.2

@MairaTariq16 MairaTariq16 added the bug Something isn't working label Mar 19, 2024
@llinros
Copy link

llinros commented Apr 3, 2024

If the helm values are passed using spec.source.helm.values: | block, those are also not removed like other parameters

The issue seems to be only affecting spec.source.helm.valuesObject, which gets removed when image-updater does updates to the Application

@MairaTariq16
Copy link
Author

If the helm values are passed using spec.source.helm.values: | block, those are also not removed like other parameters

The issue seems to be only affecting spec.source.helm.valuesObject, which gets removed when image-updater does updates to the Application

Oh yes, thank you for pointing that out. Edited the issue with correction.

@Kiliandeca
Copy link

Hello, I was experiencing the same issue you described and found out the problem doesn't appear on the current master branch.

The latest ArgoCD Image Updater release 0.12.2 was done 1 year and 4 months ago in January 2023 but ArgoCD merged the valuesObject feature 6 months later in June 2023 argoproj/argo-cd#11538 so that's why it wasn't supported in 0.12.2.

It's hard to tell if there will be a new release soon so your best chance is to use the latest docker image tag quay.io/argoprojlabs/argocd-image-updater:latest for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants