Skip to content
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

contrib: Ensure release tag is upstream before push #15903

Merged
merged 1 commit into from
Apr 28, 2021

Conversation

joestringer
Copy link
Member

Reuse the logic from the backporting cherrypick commit for determining
that the commit is upstream before tagging and pushing the tags for
release preparation. This should catch errors where the tagged release
version is not a commit in the branch where the release is made.

@joestringer joestringer requested a review from a team as a code owner April 28, 2021 02:51
@joestringer joestringer added needs-backport/1.10 release-note/misc This PR makes changes that have no direct user impact. labels Apr 28, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot added dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Apr 28, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.10.0-rc2 Apr 28, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.8.10 Apr 28, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.9.7 Apr 28, 2021
Reuse the logic from the backporting cherrypick commit for determining
that the commit is upstream before tagging and pushing the tags for
release preparation. This should catch errors where the tagged release
version is not a commit in the branch where the release is made.

Signed-off-by: Joe Stringer <joe@cilium.io>
@aanm aanm merged commit 6fcdb29 into cilium:master Apr 28, 2021
@joestringer joestringer deleted the queue/fix-wrong-tag-version branch April 28, 2021 17:27
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.10 in 1.10.0-rc2 Apr 28, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.9 in 1.9.7 Apr 28, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.8 in 1.8.10 Apr 28, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.8 to Backport done to v1.8 in 1.8.10 May 4, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.10 to Backport done to v1.10 in 1.10.0-rc2 May 6, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.9 to Backport done to v1.9 in 1.9.7 May 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/misc This PR makes changes that have no direct user impact.
Projects
No open projects
1.10.0-rc2
Backport done to v1.10
1.8.10
Backport done to v1.8
1.9.7
Backport done to v1.9
Development

Successfully merging this pull request may close these issues.

None yet

5 participants