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

chore(ci): improve queue by canceling jobs #641

Merged
merged 1 commit into from
Nov 5, 2023

Conversation

bobslept
Copy link
Contributor

@bobslept bobslept commented Nov 5, 2023

Yesterday I noticed when I made a typo, and pushed a commit into my PR, the actions keep on running. Which make no sense when there is a new commit, and with 28 images builds per commit, the queue would be pretty full after a few commits. Building stuff that doesn't matter anymore.

I've found the following documentation and tried to implement it on my personal fork to see if this would work.
https://docs.github.com/en/actions/using-jobs/using-concurrency#example-only-cancel-in-progress-jobs-or-runs-for-the-current-workflow

It did work perfect, so I would like to try it out here. If this works good here, it would definitely something to implement for other repo's building a lot of images to keep the queue clean.

Hope it works out as intended here, it would be a big improved. If not well, we can revert this PR.

@bobslept bobslept requested a review from castrojo as a code owner November 5, 2023 15:20
@bobslept bobslept enabled auto-merge November 5, 2023 15:22
@bobslept
Copy link
Contributor Author

bobslept commented Nov 5, 2023

Current build failures showing up are unrelated to this PR.

Copy link
Member

@castrojo castrojo left a comment

Choose a reason for hiding this comment

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

Oh this is neat and might be useful in main, bazzite, etc!

@bobslept bobslept added this pull request to the merge queue Nov 5, 2023
Merged via the queue into ublue-os:main with commit cfda6a1 Nov 5, 2023
28 of 32 checks passed
@bobslept bobslept deleted the improve-build-queue branch November 5, 2023 16:28
@bobslept
Copy link
Contributor Author

bobslept commented Nov 5, 2023

Yeah if it is working as intended, we should for sure implement in other repo's. But lets see how it goes in here first. 👍

b- pushed a commit to briorg/bluefin that referenced this pull request Dec 9, 2023
awesomekyle pushed a commit to awesomekyle/bluefin that referenced this pull request Apr 24, 2024
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.

2 participants