Skip to content

fix(gitops): apontar helm.image-tag pro campo tag, nao digest - #16

Merged
guesant merged 1 commit into
mainfrom
fix/image-updater-helm-tag-target
Aug 28, 2026
Merged

guesant merged 1 commit into
mainfrom
fix/image-updater-helm-tag-target

Conversation

@guesant

@guesant guesant commented Aug 28, 2026

Copy link
Copy Markdown
Member

Summary

Achado ao vivo: o pod novo ficava InvalidImageName, imagem renderizando como ghcr.io/ladesa-ro/docs/docs:development@development@sha256:..., duplicado.

Causa raiz: o Image Updater sempre escreve o valor combinado tag@digest no parametro Helm apontado por helm.image-tag (confirmado lendo pkg/argocd/argocd.go do argocd-image-updater, funcao SetHelmImage, usa newImage.GetTagWithDigest()). O chart stakater/application (templates/deployment.yaml) monta a imagem como repo:{{tag}}@{{digest}}, dois campos separados. Apontar helm.image-tag pro campo .digest fazia o valor combinado (que ja contem development@sha256:...) virar o .digest, e o .tag estatico (development, de values-development.yaml) se somar em cima, duplicando.

Corrigido apontando pro campo .tag em vez de .digest: o chart monta repo:development@sha256:... a partir de um unico campo dinamico, que e exatamente a forma de referencia de imagem nome:tag@digest valida.

Test plan

  • Apos merge: limpar o parametro application.deployment.image.digest que ficou setado errado na Application (via argocd app unset ou live), esperar o Image Updater reescrever application.deployment.image.tag, confirmar pod Running

@guesant
guesant requested review from a team as code owners August 28, 2026 16:36
@guesant
guesant merged commit 468b9e3 into main Aug 28, 2026
3 checks passed
@guesant
guesant deleted the fix/image-updater-helm-tag-target branch August 28, 2026 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant