-
Notifications
You must be signed in to change notification settings - Fork 24
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
(PC-32510) chore(pipeline): slack message only on failure or success #7058
Conversation
cf5422e
to
1245ab0
Compare
167c40d
to
2110516
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ça me va si ça fonctionne comme on s'y attend
les questions, c'est pour si tu as envie de creuser plus loin
.github/workflows/dev_on_workflow_environment_android_deploy.yml
Outdated
Show resolved
Hide resolved
@@ -129,13 +129,13 @@ jobs: | |||
"attachments": [ | |||
{ | |||
"mrkdwn_in": ["text"], | |||
"color": "${{ fromJSON('["#A30002","#36a64f"]')[steps.sentry_and_deploy.outputs.status == 'success'] }}", | |||
"color": "${{ fromJSON('["#36a64f", "#A30002"]')[needs.sentry_and_deploy.result == 'failure'] }}", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
je ne savais pas qu'on avait accès à cette info dans needs
est-ce tu sais pourquoi steps
semblait fonctionner la semaine dernière ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
aucune idée, j'ai juste suivi les préconisations de Matthias !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
réponse de Matthias pour dans quelques mois quand on se posera la question de ces modifications :
pourquoi on peut utiliser needs et pas steps ?
"parce qu'on est pas dans le meme job
la on a deux runners distincts qui ont une relation de co dépendance (le needs))
et c'est grace à cette relation qu'on peut appeler le result du premier job
si on était dans le meme job on aurait pu appeler le result de la step voulu (en lui donnant un petit nom), de mémoire cest ce qui est fait pour le workflow de test "
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
je suis code cité, j'aime ça
Performance Comparison ReportSignificant Changes To Render DurationThere are no entries Meaningless Changes To Render DurationShow entries
Show details
Changes To Render CountThere are no entries Added ScenariosThere are no entries Removed ScenariosThere are no entries |
2110516
to
94020c5
Compare
94020c5
to
e181975
Compare
Quality Gate passedIssues Measures |
Quality Gate passedIssues Measures |
Link to JIRA ticket: https://passculture.atlassian.net/browse/PC-XXXXX
Flakiness
If I had to re-run tests in the CI due to flakiness, I add the incident on Notion
Checklist
I have:
Screenshots
delete if no UI change