Skip to content

Delete the msix blob if it's already there#24353

Merged
TravisEz13 merged 1 commit intoPowerShell:masterfrom
SeeminglyScience:fix-msix-bundle-error-on-rerun
Oct 14, 2024
Merged

Delete the msix blob if it's already there#24353
TravisEz13 merged 1 commit intoPowerShell:masterfrom
SeeminglyScience:fix-msix-bundle-error-on-rerun

Conversation

@SeeminglyScience
Copy link
Contributor

PR Summary

If we're re-running the release pipeline and the MSIX blob is already present from a previous run, remove it before reuploading. If we want to retain the existing MSIX then we can deselect the stage.

Currently this would throw due to Set-AzStorageBlobContent calling ShouldContinue when the blob exists. So if we wanted to actually replace the MSIX we'd need to manually delete the blob.

PR Context

PR Checklist

@TravisEz13 TravisEz13 self-assigned this Oct 3, 2024
@TravisEz13 TravisEz13 added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Oct 3, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added the Review - Needed The PR is being reviewed label Oct 10, 2024
@TravisEz13 TravisEz13 merged commit 14c82b5 into PowerShell:master Oct 14, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot removed the Review - Needed The PR is being reviewed label Oct 14, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot removed the Review - Needed The PR is being reviewed label Oct 14, 2024
adityapatwardhan pushed a commit to adityapatwardhan/PowerShell that referenced this pull request Oct 15, 2024
adityapatwardhan pushed a commit to adityapatwardhan/PowerShell that referenced this pull request Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BackPort-7.4.x-Done BackPort-7.5.x-Done CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants