Skip to content

Conversation

@pjcollins
Copy link
Member

Commit b864f93 appears to have broken the GitHub status for the .NET 6
installers. These changes resulted in us attempting to use the same
blob storage location for both legacy and .NET 6 installers, and the
step which ran last (often legacy, as signing takes a while) would write
over the previous status. Adding net6 to the .NET 6 installer blob
storage path should fix this.

Commit b864f93 appears to have broken the GitHub status for the .NET 6
installers.  These changes resulted in us attempting to use the same
blob storage location for both legacy and .NET 6 installers, and the
step which ran last (often legacy, as signing takes a while) would write
over the previous status.  Adding `net6` to the .NET 6 installer blob
storage path should fix this.
@pjcollins
Copy link
Member Author

When complete this build should create and post aritfacts statuses for both installer types, and should be used to confirm the fix: https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=4236920&view=results

Copy link
Member

@jonathanpeppers jonathanpeppers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Github statuses I see currently seem like they worked.

The Queue Vsix Signing step is ongoing, so I'm not sure if that will affect anything.

@pjcollins
Copy link
Member Author

All three statuses have now posted with live links, I'll merge this.

@pjcollins pjcollins merged commit 2c07645 into master Nov 18, 2020
@pjcollins pjcollins deleted the unique-storage-links branch November 18, 2020 17:51
@github-actions github-actions bot locked and limited conversation to collaborators Jan 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants