-
Notifications
You must be signed in to change notification settings - Fork 158
[POC] ci: skip previously passed workflows on pipeline restart #2099
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
Conversation
2ec014b to
9efc940
Compare
ScharfViktor
left a comment
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.
very impressive, thank you so much 👍
Could you please write the logic in the pr description
that we keep info about previous running in s3 and evaluate it when restarting piplene and decide skip or not
|
Cool :), quick question, following scenario: 1st run the dev fixes the bug which made pipeline B fail woodpecker starts a new run this sounds really good, but what happens if the fix which made workflow B green introduced a new bug on workflow A? |
|
then it will be a new |
|
Ahhhh thanks @ScharfViktor so, it only applies for restarts? That’s really helpful, thanks for explaining 🤗 |
yeah, correct. We only want to restart the failing workflows and the change in commit would mean full CI run |
9efc940 to
0e6c411
Compare
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
0e6c411 to
73e6a96
Compare
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
73e6a96 to
2e2ac84
Compare
Updated the PR description |
Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
Description
How does it work?
SKIP_WORKFLOW=trueis found in.woodpecker.envNotes:
Implemented workflows
This behaviour is implemented in all the test workflows.
Working example
✔️ When the workflow passed previously (steps skipped):
❌ When the workflow failed previously (steps run):
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: