Skip to content

[v12] Fixed tag builds using commit instead of tag ref#34677

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

[v12] Fixed tag builds using commit instead of tag ref#34677
camscale merged 1 commit intobranch/v12from
camh/v12/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

@camscale camscale added the no-changelog Indicates that a PR does not require a changelog entry label Nov 16, 2023
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 force-pushed the camh/v12/fix-windows-signing-tag-build branch from 31d1c0e to 0a15309 Compare November 16, 2023 20:44
@camscale camscale changed the title {v12] Fixed tag builds using commit instead of tag ref [v12] Fixed tag builds using commit instead of tag ref 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/v12 with commit a6aa28b Nov 16, 2023
@camscale camscale deleted the camh/v12/fix-windows-signing-tag-build branch November 16, 2023 21:11
This was referenced 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