Skip to content

Ajout du support des dépôts Helm OCI pour les apps d'infra, cert-manager et Kyverno#1042

Open
benji78 wants to merge 1 commit into
cloud-pi-native:mainfrom
benji78:oci-helm-repo
Open

Ajout du support des dépôts Helm OCI pour les apps d'infra, cert-manager et Kyverno#1042
benji78 wants to merge 1 commit into
cloud-pi-native:mainfrom
benji78:oci-helm-repo

Conversation

@benji78
Copy link
Copy Markdown
Contributor

@benji78 benji78 commented Apr 30, 2026

Quel est le comportement actuel ?

Les rôles utilisent uniquement des dépôts Helm HTTP via helm_repository. Les chart_ref sont fixés sur des alias HTTP (ex: argo/argo-cd, hashicorp/vault), ce qui empêche l’utilisation de dépôts OCI. La récupération de la version Argo CD infra repose sur helm search repo uniquement.
Depuis l'utilisation de Helm chart wrappers, les applications de la forge le supporte nativement, mais peu-être pas (à vérifier) pour Nexus (que je n'installe pas) et Keycloak (en HTTP) qui ont des conditions différentes. c.f. :

repository: "{{ dsc.keycloak.helmRepoUrl if 'oci://' in dsc.keycloak.helmRepoUrl else dsc.keycloak.helmRepoUrl ~ '/keycloak' }}"

repository: "{{ dsc.nexus.helmRepoUrl if 'oci://' in dsc.nexus.helmRepoUrl else dsc.nexus.helmRepoUrl ~ '/nexus' }}"

Quel est le nouveau comportement ?

  • Les rôles infra (Argo CD, Keycloak, Vault), cert-manager, CloudNativePG et Kyverno détectent oci:// et adaptent l’ajout de repo ainsi que chart_ref.
  • La récupération de version Argo CD infra est gérée pour HTTP et OCI (avec helm show chart pour OCI).
  • L’installation conserve le même rendu de values, mais change la source Helm en fonction du type de dépôt.

J'ai également ajouté un when conditionnel sur OpenShift pour le CRB Argo CD infra, sans impact sur les plateformes non‑OpenShift (à ma connaissance).

Cette PR introduit-elle un breaking change ?

Non. Les dépôts HTTP continuent de fonctionner comme avant.

Autres informations

J'ai ainsi déployé Argo CD (d'infra et de la forge) avec un dépot OCI sans problème.
Pour information, j'avais fait ce changement juste avant le passage au déploiement de la forge par Argo CD (gitops) mais avec toutes les modifications et corrections apportés à ce moment là, j'ai retardé cette contribution.

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.

2 participants