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
Open
Ajout du support des dépôts Helm OCI pour les apps d'infra, cert-manager et Kyverno#1042benji78 wants to merge 1 commit into
benji78 wants to merge 1 commit into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Quel est le comportement actuel ?
Les rôles utilisent uniquement des dépôts Helm HTTP via
helm_repository. Leschart_refsont 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 surhelm search repouniquement.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. :
socle/roles/gitops/rendering-apps-files/templates/keycloak/Chart.yaml.j2
Line 14 in 6982842
socle/roles/gitops/rendering-apps-files/templates/nexus/Chart.yaml.j2
Line 10 in 6982842
Quel est le nouveau comportement ?
oci://et adaptent l’ajout de repo ainsi quechart_ref.helm show chartpour OCI).J'ai également ajouté un
whenconditionnel 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.