feat: ✨ Ajout des champs de saisie pour choisir les spécificités du dépôt d'infra

[mathieulaude]

Issues liées

Issues numéro: #1806

---

Quel est le comportement actuel ?

Les valeurs de chemin, branche et valuesFile sont définies par défaut.

Quel est le nouveau comportement ?

Possibilité pour l'utilisateur de personnaliser son déploiement avec le choix du chemin, branche et valuesFile.

Cette PR introduit-elle un breaking change ?

Autres informations

[github-actions]

🤖 Hey !

The @cpn-console/shared (v1.3.0) package already exists on npm but the source code has changed, you should consider updating the package version.

The version update warning should be ignored in the case of modifications that do not affect the code once it has been built, such as code formatting, etc...

[github-actions]

🤖 Hey !

The @cpn-console/argocd-plugin (v2.3.0) package already exists on npm but the source code has changed, you should consider updating the package version.

The version update warning should be ignored in the case of modifications that do not affect the code once it has been built, such as code formatting, etc...

[github-actions]

🤖 Hey !

The @cpn-console/hooks (v4.0.0) package already exists on npm but the source code has changed, you should consider updating the package version.

The version update warning should be ignored in the case of modifications that do not affect the code once it has been built, such as code formatting, etc...

[github-actions]

🤖 Hey !

A preview of the application is available at : https://console-pr-1807.sdid-hp.cpin.numerique-interieur.com

Please be patient, deployment may take a few minutes.

[github-actions]

🤖 Hey !

The security scan report for the current pull request is available here.

[StephaneTrebel]

Quelques points que je voudrais clarifier, même si dans l'ensemble je ne vois rien de flagrant. C'est plus pour le tur-fu

[cloud-pi-native-sonarqube]

Quality Gate passed

Issues

• 0 New Issues
• 0 Fixed Issues
• 0 Accepted Issues

Measures

• 0 Security Hotspots
• 100.00% Coverage (73.70% Estimated after merge)
• 21.80% Duplicated Code (4.60% Estimated after merge)

Project ID: cloud-pi-native_console_AYsa46O31ebDtzs-pYyZ

View in SonarQube

[this-is-tobi]

Hello,

Je me permet de vous poser quelques petites questions, peut-être à côté de la plaque ne m'en voulez pas je vois sujet de loin :

• Est ce que ces modifications ne sont prévues que sur les clusters externes ou sur l'ensemble des apps gérées par CPiN ?
• Dans le formulaire vous avez bien prévu de pouvoir renseigner les sources de l'application ArgoCD ? (multi-sources pour pouvoir séparer les sources Helm et les fichiers de values)
• Avez-vous pensé à la possibilité de surcharger des values directement dans l'interface ArgoCD pendant les phases de tests qui pourrait être impactées ? (lié à la question précédente )

Et aussi qu'est ce qui est prévu côté reprise de données des modifications qui ont pu être faites par les projets sur l'interface ArgoCD (et donc sur la ressource Kube associée) ?

Force et honneur 🖖🏻

[mathieulaude]

Hello @this-is-tobi ,
Merci pour ces questions très pertinentes !

• Ces modifications visent précisément à considérer tous les clusters de la même manière : "externes", c'est-à-dire pilotés par ArgoCD en fonction des inputs de la Console (et plus par la Console directement, soit la fin du geste mode Push vs Pull)
• Non ce premier dev ne gère pas les Applications multi-sources, car nous n'avons pas encore décidé du meilleur moyen de gérer ces cas particuliers : ajout de nouveaux champs customs ou bien ouverture complète vers le pattern App of Apps
• Oui le coeur du sujet c'est précisément de passer par la Console, pour limiter le nombre d'éditions manuelles à réaliser dans la WebUI ArgoCD et aller vers la promesse du déploiement "clé en main". Il restera la possibilité de désactiver l'auto-sync lors des phases de tests, avant de pérenniser les values dans un fichier dédié et réactiver l'auto-sync ;)

[ArnaudTA]

Je me permets de réagir sur les clusters externes, cela veut dire que tous les clusters vont perdre le mode push ?

Je suis inquiet pour la feedback loop en phase de test à vrai dire

[StephaneTrebel]

LGTM