Skip to content

Conversation

@benji78
Copy link
Contributor

@benji78 benji78 commented Jan 15, 2026

Quel est le comportement actuel ?

Actuellement, l’URL et la révision du dépôt Git "socle" utilisé par les jobs Ansible pour les tâches de post-installation sont codées en dur. Le dépôt est cloné avec une URL fixe (https://github.com/cloud-pi-native/socle.git) sans possibilité de spécifier une révision (branche, tag ou commit) autre que la branche par défaut.

Quel est le nouveau comportement ?

Avec ce changement, l’URL et la révision du dépôt "socle" deviennent configurables via le CRD. Deux nouveaux champs ont été ajoutés :

  • repoSocle.url : Permet de spécifier une URL personnalisée pour le dépôt Git contenant les playbooks Ansible.
  • repoSocle.revision : Permet de spécifier une branche, un tag ou un commit spécifique à cloner.

Exemple d’utilisation dans le CRD :

repoSocle:
  url: "https://mon-depot.fr/mon-socle.git"
  revision: "branche-tag-ou-commit-id"

Cette PR introduit-elle un breaking change ?

Non, pas de modification du comportement par défaut.

Autres informations

Ce changement est particulièrement utile pour :

  • Les environnements air-gapped où l’URL du dépôt peut être différente.
  • Les phases de développement ou de test, où il est nécessaire de pointer vers une branche ou un commit spécifique.

Je l'ai testé et, même si le \ se met sur la ligne suivante, cela ne pose pas de problème lors de execution du job :

    - 'git clone https://github.com/benji78/pi-native-socle.git -b develop socle &&
      \

@benji78 benji78 changed the base branch from main to develop January 15, 2026 17:06
@Falltrades Falltrades added this to the MCO milestone Jan 19, 2026
@Falltrades Falltrades added the enhancement New feature or request label Jan 19, 2026
Copy link
Collaborator

@Falltrades Falltrades left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Globalement ok, même si c’était possible de surcharger les values des jobs depuis la dsc (ex: https://github.com/cloud-pi-native/socle/blob/v4.4.2/roles/socle-config/templates/crd-conf-dso.yaml#L428).

- restore
exposed:
description: Whether or not the cnpg cluster shoul be exposed via NodePort.
description: Whether or not the cnpg cluster should be exposed via NodePort.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thought: seems like we forgot to remove support of this option for this service.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed, it seems none of these dsc.awx.cnpg options were ever even used since they were added here: 5451db5

Configuration for the socle Git repository used by Ansible jobs for post-installation tasks. Especially useful in air-gapped environments and during development.
@benji78 benji78 force-pushed the feat/configurable-socle-repo branch from fd77c3f to 10ee564 Compare January 19, 2026 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants