Skip to content

Ajout de l'IAM pour Github backups#40

Open
leo-naeka wants to merge 2 commits intomainfrom
leo/add-github-backups-iam
Open

Ajout de l'IAM pour Github backups#40
leo-naeka wants to merge 2 commits intomainfrom
leo/add-github-backups-iam

Conversation

@leo-naeka
Copy link
Copy Markdown
Collaborator

Permet de définir une application, son API key associée et sa policy pour le script de backup Github.
Insère l'API key dans le secret manager pour pouvoir la passer aux Github Actions de https://github.com/gip-inclusion/gh-backups/

@leo-naeka leo-naeka self-assigned this Oct 22, 2025
Copy link
Copy Markdown

@aminedhobb aminedhobb left a comment

Choose a reason for hiding this comment

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

Merci @leo-naeka 🙏 !
J'ai posé une question pour être sûr d'avoir compris un trick. Ça m'a l'air ok mais je laisse quelqu'un de plus expérimenté que moi approuver la PR :)

@@ -0,0 +1,9 @@
provider "scaleway" {
alias = "tmp"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

ok donc si je comprends bien on fait ça pour exposer notre projet scaleway dans les data et pouvoir récupérer le default id quand on déclare "vraiment" le provider en bas ? L'idée c'est que le provider ne peut pas se référencer lui-même si j'ai compris

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Tout à fait, car la définition du provider dépend de la réponse de la source de données, mais pour requêter la source de données, il faut un provider.

https://github.com/gip-inclusion/infrastructure/pull/40/files/e2ee8261c08b715435bf9cee3cfbf88414fabee3#diff-b4b8e40a68685bbd22342674bf7c3b20abc06604f0caf418b2c71f04d20b583fR8

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.

3 participants