Skip to content

fix(argocd): set metadata.namespace on repo-creds secrets so they land in the ArgoCD namespace - #1126

Open
KepoParis wants to merge 1 commit into
mainfrom
fix/argocd-repo-creds-namespace
Open

KepoParis wants to merge 1 commit into
mainfrom
fix/argocd-repo-creds-namespace

Conversation

@KepoParis

Copy link
Copy Markdown
Contributor

Issues liées

Issues numéro: #1098


Quel est le comportement actuel ?

Les secrets gitlab, oci-dockerhub-creds et oci-harbor-creds (repo-creds.yml.j2) ne définissent pas metadata.namespace. Comme install-zone-argocd.sh applique le manifeste sans -n, ils sont créés dans le namespace courant du kubeconfig (souvent default) au lieu de dso-argocd.

Quel est le nouveau comportement ?

Ces trois secrets ont maintenant namespace: {{ .Release.Namespace }}. Le commentaire du script, qui affirmait à tort que helm template --namespace renseigne metadata.namespace, est corrigé.

Cette PR introduit-elle un breaking change ?

Non. Les secrets créés par erreur dans un autre namespace peuvent être supprimés à la main.

Autres informations

Remettre -n dso-argocd dans le script ne fonctionnerait pas : le manifeste contient aussi des ressources dso-backup dans infra-velero, et kubectl apply -n refuse tout objet dont le namespace diffère de celui passé en argument (the namespace from the provided object "infra-velero" does not match the namespace "dso-argocd"). Il faut donc définir le namespace dans chaque template.

Il faut regénérer le dépôt GitOps pour que la correction y soit prise en compte.

@KepoParis KepoParis self-assigned this Oct 9, 2026
@KepoParis
KepoParis requested a review from a team October 9, 2026 16:34
@KepoParis KepoParis added the bug Something isn't working label Oct 9, 2026
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

Successfully merging this pull request may close these issues.

1 participant