hypertrace / javaagent

Hypertrace OpenTelemetry Java agent with payload/body and headers data capture.
Apache License 2.0
30 stars 15 forks source link

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

Closed ryandens closed 3 years ago

ryandens commented 3 years ago

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 commented 3 years ago

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 commented 3 years ago

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

pavolloffay commented 3 years ago

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

pavolloffay commented 3 years ago

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 commented 3 years ago

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