Skip to content

[v14] Fixed tag builds using commit instead of tag ref#34675

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

[v14] Fixed tag builds using commit instead of tag ref#34675
camscale merged 1 commit intobranch/v14from
camh/v14/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
@github-actions github-actions Bot requested a review from tcsc November 16, 2023 20:37
@github-actions github-actions Bot requested a review from wadells November 16, 2023 20:37
@camscale camscale force-pushed the camh/v14/fix-windows-signing-tag-build branch from c73ed14 to 260e063 Compare November 16, 2023 20:38
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/v14/fix-windows-signing-tag-build branch from 260e063 to fc4b38c Compare November 16, 2023 20:41
@camscale camscale enabled auto-merge November 16, 2023 20:46
@camscale camscale added this pull request to the merge queue Nov 16, 2023
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Nov 16, 2023
@camscale camscale added this pull request to the merge queue Nov 16, 2023
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Nov 16, 2023
@camscale camscale added this pull request to the merge queue Nov 16, 2023
Merged via the queue into branch/v14 with commit 21c7624 Nov 16, 2023
@camscale camscale deleted the camh/v14/fix-windows-signing-tag-build branch November 16, 2023 22:01
@camscale camscale mentioned this pull request Nov 16, 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