Skip to content

🐛 fix bug where release pipeline is triggered on every branch creation #336

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

Merged
merged 1 commit into from
Jul 14, 2021

Conversation

ryandens
Copy link
Member

Description

Currently, when a new branch is created, a release pipeline is triggered. i believe that is because the syntax of the GitHub actions workflow for our release pipeline needs to be tweaked to use the YAML path on.push.tags rather than on.create.tags.

See:

@ryandens
Copy link
Member Author

I noticed that this occurred when I created a new branch, so I cancelled the workflow. This has been happening for a while, but up until the migration away from CircleCI this pipeline was essentially a no-op so it did no harm. Now, it could

@ryandens
Copy link
Member Author

Example canceled workflow: https://github.com/hypertrace/javaagent/actions/runs/1026610635

@ryandens ryandens merged commit b77175d into main Jul 14, 2021
@ryandens ryandens deleted the fix-release-workflow branch July 14, 2021 20:49
@pavolloffay
Copy link
Member

It's a common practice to use branches in forks and keep the upstream with only main or maintenance branches.

@pavolloffay
Copy link
Member

I am curious do you have a link to CI run for your branch? The release job uses pattern match (e.g. x.x.x) that should not match your feature branch.

@ryandens
Copy link
Member Author

The problem was the syntax of the yaml file, which was the yaml path on.create.tags. This isn't a valid configuration option for GitHub actions, it should be on.push.tags as this PR changes it to be. Now, the pattern match is enforced

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.

3 participants