Remove duplicate containers node#53021
Conversation
|
/backport to release/9.0.3xx |
|
Started backporting to |
There was a problem hiding this comment.
Pull request overview
Updates the internal Azure DevOps CI pipeline YAML to eliminate a duplicated containers node and keep container definitions in the single, expected location for the 1ES pipeline templates.
Changes:
- Adds
sourceBuildManagedContainerunderextends.parameters.containers. - Removes the duplicate
containersblock that previously existed elsewhere in the parameters structure.
| @@ -79,6 +79,8 @@ extends: | |||
| image: mcr.microsoft.com/dotnet-buildtools/prereqs:azurelinux-3.0-net9.0-cross-arm-alpine | |||
| ubuntu2204DebPkg: | |||
| image: mcr.microsoft.com/dotnet-buildtools/prereqs:ubuntu-22.04-debpkg | |||
There was a problem hiding this comment.
sourceBuildManagedContainer uses the same image as the existing centosStream9 container. If nothing specifically requires this extra alias, consider removing it (or switching any consumers to centosStream9) to avoid having to update the CentOS Stream 9 image tag in multiple places. If it is required by an external template, adding a short comment explaining why this separate name is needed would help prevent accidental cleanup later.
| image: mcr.microsoft.com/dotnet-buildtools/prereqs:ubuntu-22.04-debpkg | |
| image: mcr.microsoft.com/dotnet-buildtools/prereqs:ubuntu-22.04-debpkg | |
| # Alias required by source-build templates; intentionally uses same image as centosStream9. |
No description provided.