Bring More Release Branch Changes to release/v7.5 #24379
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Rather than an intermediate folder that must be manually copied into the static site. ---- #### AI description (iteration 1) #### PR Classification Code modification to enhance functionality. #### PR Summary This pull request updates the release pipeline to copy global tools to a static site. - Changes in `/.pipelines/templates/release-MakeBlobPublic.yml` to set `prefix` and `destinationPrefix` for blob copying. - Adjusted destination container name to use `$web` and updated blob naming conventions.
… tool copy #### AI description (iteration 1) #### PR Classification Code cleanup #### PR Summary This pull request updates the container name string to use single quotes for consistency. - Changes in `/.pipelines/templates/release-MakeBlobPublic.yml` to switch `$destinationContainerName` from double quotes to single quotes.
jshigetomi
requested review from
daxian-dbw,
TravisEz13,
adityapatwardhan and
anmenaga
as code owners
TravisEz13
approved these changes
Oct 3, 2024
|
📣 Hey @jshigetomi, how did we do? We would love to hear your feedback with the link below! 🗣️ 🔗 https://aka.ms/PSRepoFeedback |
adityapatwardhan
added
the
CL-BuildPackaging
21 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR Summary
This pull request includes several changes to the
.pipelinesdirectory, primarily focusing on adding new parameters and modifying existing job conditions to improve the flexibility and functionality of the release pipeline.New Parameters and Conditions
.pipelines/PowerShell-Release-Official.yml: Added a new boolean parameterSkipPSInfraInstallersto allow skipping the copying of archives and installers to the PSInfrastructure public location. [1] [2].pipelines/templates/release-MakeBlobPublic.yml: Introduced theSkipPSInfraInstallersparameter and added a condition to thePSInfraReleaseBlobPublicjob to skip execution if this parameter is true. [1] [2]Resource Configuration
.pipelines/PowerShell-Release-Official.yml: Added feature flags forLinuxHostVersionandWindowsHostVersionunder theresourcessection to specify network configurations.Blob Storage Adjustments
.pipelines/templates/release-MakeBlobPublic.yml: Modified the blob storage copying logic to include a destination prefix and updated the destination container configuration.Miscellaneous Changes
.pipelines/templates/release-publish-pmc.yml: Changed theBlobFolderNameparameter from$(AzureVersion)to$(ReleaseTag)to better reflect the release tag in the blob folder name.PR Context
PR Checklist
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright headerWIP:or[ WIP ]to the beginning of the title (theWIPbot will keep its status check atPendingwhile the prefix is present) and remove the prefix when the PR is ready.- [ ] Issue filed:
(which runs in a different PS Host).