Open
Conversation
6188281 to
354fa47
Compare
Member
Author
|
Le check Run Terraform fail car je n'ai pousse le fichier imports.tf ni les locals lie au code. Est-ce qu'on peut deja valider l'archi generale du module avant de versionner ces points? |
Member
francoisfreitag
left a comment
There was a problem hiding this comment.
Chouette de voir le sujet avancer et ça donne une base, après on a sûrement pas mal d’ajustements à apporter:
- Idéalement, comme pour les DNS, on aurait un module réutilisable qui applique la config souhaitée au niveau organisationnel, pour permettre à chaque équipe de lister ses divergences. Et donc un module par base de code.
- Pas la peine de faire exécuter l’import par la CI, on peut le faire en local et push les ressources importées.
- le script est relativement difficile à suivre et fait quelques erreurs, avec un certain nombre de propriétés qui changent si on
plan. Il y a du travail pour peaufiner ça. - l’énumération de chaque propriété de GitHub dans les HCL n’est pas très pratique. J’imaginerais plutôt :
- ne pas spécifier les propriétés que nous n’utilisons pas -> clarté, suivi des défauts de GitHub
- spécifier les propriétés que nous configurons
- accepter en
inputsles propriétés que nous autorisons les repos à configurer
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.
🤔 Pourquoi ?
Implementer l'IaC pour nos repos GH
🍰 Comment ?
Un module Terraform dedie. Un script pour genere le fichier imports.tf et completer les
localsdes differentes ressources. Un apply a prevoir pour l'import et ensuite gerer les plan et apply via la CI.🚨 À vérifier