Skip to content

[v13] Fixed tag builds using commit instead of tag ref#34676

Merged
camscale merged 1 commit intobranch/v13from
camh/v13/fix-windows-signing-tag-build
Nov 16, 2023
Merged

[v13] Fixed tag builds using commit instead of tag ref#34676
camscale merged 1 commit intobranch/v13from
camh/v13/fix-windows-signing-tag-build

Conversation

@camscale
Copy link
Copy Markdown
Contributor

Tag builds for Windows are broken in the following ways:

  • The ref used to determine asset version includes a short commit hash
    (this will cause promotion issues)

  • Upon retry of a build (or a new tag with the same commit), builds will
    fail as the old release is not properly cleaned up

Changing the reference used to a branch/tag reference should fix the
issue for both. Note that the second issue is still bugged for push
pipelines, but we almost never retry them so it's not really worth
fixing there right now.

Backport: #34668

Tag builds for Windows are broken in the following ways:

* The ref used to determine asset version includes a short commit hash
  (this will cause promotion issues)

* Upon retry of a build (or a new tag with the same commit), builds will
  fail as the old release is not properly cleaned up

Changing the reference used to a branch/tag reference should fix the
issue for both. Note that the second issue is still bugged for push
pipelines, but we almost never retry them so it's not really worth
fixing there right now.
@camscale camscale added the no-changelog Indicates that a PR does not require a changelog entry label Nov 16, 2023
@camscale camscale enabled auto-merge November 16, 2023 20:46
@camscale camscale added this pull request to the merge queue Nov 16, 2023
Merged via the queue into branch/v13 with commit 98d4250 Nov 16, 2023
@camscale camscale deleted the camh/v13/fix-windows-signing-tag-build branch November 16, 2023 21:11
@camscale camscale mentioned this pull request Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport no-changelog Indicates that a PR does not require a changelog entry size/sm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants