This repository has been archived by the owner on Jul 30, 2024. It is now read-only.
ci: limiting mirroring concurrency #93
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
DeveloperC286
force-pushed
the
mirroring_concurrency
branch
from
e995f79 to
97bf35c
Compare
## Problem Mirroring actions are failing because other actions have already mirrored what it was about to mirror. E.g. * https://github.com/DeveloperC286/starlingbanktechnicalchallenge/actions/runs/7639729071/job/20813352947 ``` Run git push --mirror "***gitlab.com/DeveloperC/${REPOSITORY_NAME}" warning: redirecting to https://gitlab.com/DeveloperC/starlingbanktechnicalchallenge.git/ remote: error: cannot lock ref 'refs/heads/earthly_upgrade': unable to resolve reference 'refs/heads/earthly_upgrade' remote: error: cannot lock ref 'refs/pull/92/head': reference already exists To https://gitlab.com/DeveloperC/starlingbanktechnicalchallenge f4c0fa2..dfc1c32 main -> main ! [remote rejected] earthly_upgrade (failed to update ref) ! [remote rejected] refs/pull/92/head -> refs/pull/92/head (failed to update ref) error: failed to push some refs to 'https://gitlab.com/DeveloperC/starlingbanktechnicalchallenge' ``` * https://github.com/DeveloperC286/starlingbanktechnicalchallenge/actions/runs/7639728988/job/20813352607 ``` Run git push --mirror "***gitlab.com/DeveloperC/${REPOSITORY_NAME}" warning: redirecting to https://gitlab.com/DeveloperC/starlingbanktechnicalchallenge.git/ remote: error: cannot lock ref 'refs/heads/main': is at dfc1c32 but expected f4c0fa2 To https://gitlab.com/DeveloperC/starlingbanktechnicalchallenge - [deleted] earthly_upgrade * [new reference] refs/pull/92/head -> refs/pull/92/head ! [remote rejected] main -> main (failed to update ref) error: failed to push some refs to 'https://gitlab.com/DeveloperC/starlingbanktechnicalchallenge' ``` ## Solution So we are limiting the concurrency to prevent this. But we are not cancelling in progress actions, as the cancellation is shown as a failure in the UI. So even though some actions will mirror nothing and are unnecessary we will do it so the UI shows a passing workflow. ## Related * https://docs.github.com/en/actions/using-jobs/using-concurrency * #85
DeveloperC286
force-pushed
the
mirroring_concurrency
branch
from
97bf35c to
a6c703f
Compare
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Problem
Mirroring actions are failing because other actions have already mirrored what it was about to mirror.
E.g.
Solution
So we are limiting the concurrency to prevent this.
Related