ci: prevent triggering release builds accidentally #7401
Merged
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.
There is currently a serious bug in the CI which may result in image tags being re-pushed post-release.
The issue is that our current system to detect when the
releasing/version/VERSION
file has changed (and thus a new "release" image tag needs to be created) will trigger 100% of the time.This is because, by default, the
dorny/paths-filter@v2
will compare againstmaster
, even if the push was to thev1.8-branch
, and there will always be a different on thereleasing/version/VERSION
between master and release branches.This PR simply uses
${{ github.ref }}
as the base of comparison, as suggested by the author ofdorny/paths-filter
for long-lived branches.