Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TECH] Essai de déclenchement de la CI via une Github Action, uniquement pour les Pull Requests. #10133

Merged
merged 1 commit into from
Sep 23, 2024

Conversation

HEYGUL
Copy link
Contributor

@HEYGUL HEYGUL commented Sep 17, 2024

🦄 Problème

Actuellement, la CI (pipeline dans CircleCI) est déclenchée sur tous les commits de toutes les branches.
On souhaite diminuer notre consommation sans transiger sur notre pipeline de production.
Pour cela, on veut exécuter le pipeline de CI uniquement pour les commits d'une branche faisant l'objet d'une Pull Request prête à être revue (ie. non draft).
Nous avons essayé le réglage dans les paramètres de CircleCI permettant de lancer les pipelines uniquement pour les Pull Requests mais cela a permis à des branches d'être mergée sans pipeline du tout ...

🤖 Proposition

CircleCI propose une Github Action officielle permettant de déclencher un pipeline de CI.
On créé donc une Github Action qui s'exécute uniquement pour les PRs non draft et lance le pipeline de CI sur CircleCI.
On prend soin également d'autoriser tous les commits de la branche dev.

🌈 Remarques

RAS

💯 Pour tester

Changer l'état de la PR et constater qu'une PR draft ne déclenche pas la CI, alors qu'une PR prête pour revue la déclenche.
Mettre à jour la branche (par rebase par exemple) et constater que la CI est déclenchée également.

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@HEYGUL HEYGUL force-pushed the tech/trigger-ci-from-github-action branch from 52d2d27 to 53a1291 Compare September 17, 2024 06:11
@HEYGUL HEYGUL closed this Sep 17, 2024
@HEYGUL HEYGUL reopened this Sep 17, 2024
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@HEYGUL HEYGUL force-pushed the tech/trigger-ci-from-github-action branch 3 times, most recently from 3359eab to 731a667 Compare September 18, 2024 08:53
@HEYGUL HEYGUL marked this pull request as draft September 19, 2024 07:36
@HEYGUL HEYGUL force-pushed the tech/trigger-ci-from-github-action branch 2 times, most recently from 6567fd3 to 696bdc1 Compare September 22, 2024 17:15
@HEYGUL HEYGUL self-assigned this Sep 22, 2024
@HEYGUL HEYGUL marked this pull request as ready for review September 22, 2024 17:16
@HEYGUL HEYGUL added Development in progress cross-team Toutes les équipes de dev labels Sep 22, 2024
@HEYGUL HEYGUL force-pushed the tech/trigger-ci-from-github-action branch from 696bdc1 to ebe0e37 Compare September 23, 2024 09:40
@VincentHardouin VincentHardouin marked this pull request as draft September 23, 2024 12:23
@VincentHardouin VincentHardouin marked this pull request as ready for review September 23, 2024 12:23
@pix-service-auto-merge pix-service-auto-merge force-pushed the tech/trigger-ci-from-github-action branch 2 times, most recently from 3f2df84 to 452bead Compare September 23, 2024 13:54
This will let us trigger build-and-test workflow
on pull request only

See https://circleci.com/blog/trigger-circleci-pipeline-github-action/
@HEYGUL HEYGUL force-pushed the tech/trigger-ci-from-github-action branch from 452bead to 18546f4 Compare September 23, 2024 14:13
@pix-service-auto-merge pix-service-auto-merge merged commit dae9ec1 into dev Sep 23, 2024
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the tech/trigger-ci-from-github-action branch September 23, 2024 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cross-team Toutes les équipes de dev Func Review OK PO validated functionally the PR 🚀 Ready to Merge Tech Review OK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants