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

ci: Fix cancelation of Docker builds in CI #331

Merged
merged 1 commit into from
May 22, 2024

Conversation

joeyparrish
Copy link
Contributor

The Docker publication workflow tries to optimize and avoid rebuilding GCC if nothing has changed. However, the very first workflow run that should have created the very first GCC Docker image was canceled.

This changes the concurrency setting on the workflow so that it does not cancel an existing workflow. This way, if a series of changes are merged rapidly, and an earlier one requires a GCC rebuild, the later workflow runs will see that image.

This allows us to keep the optimization of skipping GCC rebuilds most of the time.

The Docker publication workflow tries to optimize and avoid rebuilding
GCC if nothing has changed.  However, the very first workflow run that
should have created the very first GCC Docker image was canceled.

This changes the concurrency setting on the workflow so that it does
not cancel an existing workflow.  This way, if a series of changes are
merged rapidly, and an earlier one requires a GCC rebuild, the later
workflow runs will see that image.

This allows us to keep the optimization of skipping GCC rebuilds most
of the time.
@joeyparrish
Copy link
Contributor Author

Canceled build, for reference: https://github.com/Stephane-D/SGDK/actions/runs/9180679521

@Stephane-D Stephane-D merged commit 43fb489 into Stephane-D:master May 22, 2024
2 checks passed
@joeyparrish joeyparrish deleted the fix-docker-ci-cancelation branch May 22, 2024 17:04
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.

None yet

2 participants