Skip to content
This repository has been archived by the owner on Oct 30, 2024. It is now read-only.

Exclude default branch in some CircleCI workflows #3774

Merged
merged 1 commit into from
Oct 7, 2024

Conversation

marians
Copy link
Member

@marians marians commented Oct 7, 2024

Towards https://github.com/giantswarm/giantswarm/issues/31824

This PR should exclude commits to the default branch from triggering the "push-to-registries" and "push-to-app-catalog" workflows. The assumption is that these workflows should be triggered only by pushes to non-default branches and by release tags.

The reason we are making this PR is that we have encountered problems with container image replication that are likely caused by fast pushes of the same image tag with different content. In addition, this will help us avoid obsolete CI runs.

As a code owner, please review and merge this PR if possible.

If you have reasons to keep these workflows running for the default branch, please let us (Honeybadger) know. Thank you!

@marians marians self-assigned this Oct 7, 2024
@marians marians requested a review from a team as a code owner October 7, 2024 10:47
@marians marians enabled auto-merge (squash) October 7, 2024 11:52
@marians
Copy link
Member Author

marians commented Oct 7, 2024

FYI: I'm not going to make the CI checks pass

@marians marians merged commit dda2c10 into master Oct 7, 2024
3 checks passed
@marians marians deleted the circleci-exclude-default-branch branch October 7, 2024 12:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants