Skip to content

[release/v7.2] Make some release tests run in a hosted pools#24400

Merged
TravisEz13 merged 1 commit intoPowerShell:release/v7.2from
TravisEz13:backport-24270-0
Oct 8, 2024
Merged

[release/v7.2] Make some release tests run in a hosted pools#24400
TravisEz13 merged 1 commit intoPowerShell:release/v7.2from
TravisEz13:backport-24270-0

Conversation

@TravisEz13
Copy link
Member

Backport #24270

This pull request includes several changes to the .pipelines/PowerShell-Release-Official.yml and .pipelines/templates/release-validate-sdk.yml files to improve the pipeline configuration and job execution. The most important changes include adding pool names and image names for different SDK validation jobs, updating pool and image parameters, and refining the handling of repository paths and NuGet configuration.

Pipeline configuration improvements:

  • .pipelines/PowerShell-Release-Official.yml: Added poolName and imageName parameters for Windows, MacOS, and Linux SDK validation jobs. ([.pipelines/PowerShell-Release-Official.ymlL114-R130](https://github.com/PowerShell/PowerShell/pull/24400/files#diff-b25fb4a8cfad5e36ba1c9c9c7cbe494fce4eed0067878610ebb12ffcab7e41cdL114-R130))
  • .pipelines/PowerShell-Release-Official.yml: Added a new variable group PoolNames. ([.pipelines/PowerShell-Release-Official.ymlR54](https://github.com/PowerShell/PowerShell/pull/24400/files#diff-b25fb4a8cfad5e36ba1c9c9c7cbe494fce4eed0067878610ebb12ffcab7e41cdR54))

Job parameter updates:

  • .pipelines/templates/release-validate-sdk.yml: Replaced jobtype with poolName and imageName parameters to better specify the job environment. ([.pipelines/templates/release-validate-sdk.ymlL4-R32](https://github.com/PowerShell/PowerShell/pull/24400/files#diff-7fd1753df22414f945db9dae9369e8d700d930705d20e4e3fbff38eef3f4a433L4-R32))
  • .pipelines/templates/release-validate-sdk.yml: Updated job pool configuration to use the new poolName and imageName parameters. ([.pipelines/templates/release-validate-sdk.ymlL4-R32](https://github.com/PowerShell/PowerShell/pull/24400/files#diff-7fd1753df22414f945db9dae9369e8d700d930705d20e4e3fbff38eef3f4a433L4-R32))

Repository path and NuGet configuration refinements:

  • .pipelines/templates/release-validate-sdk.yml: Unified repository path handling by removing conditional logic for MacOS. ([[1]](https://github.com/PowerShell/PowerShell/pull/24400/files#diff-7fd1753df22414f945db9dae9369e8d700d930705d20e4e3fbff38eef3f4a433L47-R49), [[2]](https://github.com/PowerShell/PowerShell/pull/24400/files#diff-7fd1753df22414f945db9dae9369e8d700d930705d20e4e3fbff38eef3f4a433L82-R84))
  • .pipelines/templates/release-validate-sdk.yml: Added logic to handle different NuGet configuration paths and ensure correct file updates. ([.pipelines/templates/release-validate-sdk.ymlL104-R116](https://github.com/PowerShell/PowerShell/pull/24400/files#diff-7fd1753df22414f945db9dae9369e8d700d930705d20e4e3fbff38eef3f4a433L104-R116))
  • .pipelines/templates/release-validate-sdk.yml: Added a step to insert NuGet config for Azure feed. ([.pipelines/templates/release-validate-sdk.ymlL4-R32](https://github.com/PowerShell/PowerShell/pull/24400/files#diff-7fd1753df22414f945db9dae9369e8d700d930705d20e4e3fbff38eef3f4a433L4-R32))

@TravisEz13 TravisEz13 self-assigned this Oct 8, 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 8, 2024
@TravisEz13 TravisEz13 merged commit 07df459 into PowerShell:release/v7.2 Oct 8, 2024
@TravisEz13 TravisEz13 deleted the backport-24270-0 branch October 8, 2024 18:09
@microsoft-github-policy-service
Copy link
Contributor

microsoft-github-policy-service bot commented Oct 8, 2024

📣 Hey @TravisEz13, how did we do? We would love to hear your feedback with the link below! 🗣️

🔗 https://aka.ms/PSRepoFeedback

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

1 participant