Skip to content

Fix vmr-license-scan pipeline for 1ES template compatibility#52919

Merged
mthalman merged 1 commit intodotnet:release/9.0.1xxfrom
mthalman:license-scan
Feb 10, 2026
Merged

Fix vmr-license-scan pipeline for 1ES template compatibility#52919
mthalman merged 1 commit intodotnet:release/9.0.1xxfrom
mthalman:license-scan

Conversation

@mthalman
Copy link
Member

@mthalman mthalman commented Feb 9, 2026

  • Use absolute paths with @self for template references inside extends block
  • Declare container in extends.parameters.containers instead of inline on job
  • Replace publish step alias with templateContext output pipelineArtifact

Fixes the issue at dotnet/installer#20787 (comment)

- Use absolute paths with @self for template references inside extends block
- Declare container in extends.parameters.containers instead of inline on job
- Replace publish step alias with templateContext output pipelineArtifact
@mthalman mthalman requested review from a team as code owners February 9, 2026 19:57
Copilot AI review requested due to automatic review settings February 9, 2026 19:57
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the VMR license scan Azure DevOps pipeline to be compatible with the 1ES official pipeline template expectations (template reference resolution, container declaration, and artifact publishing via templateContext).

Changes:

  • Moves the LicenseScan job container definition into extends.parameters.containers and references it by name.
  • Publishes BuildLogs via templateContext.outputs (pipelineArtifact) instead of an inline publish: step.
  • Updates template includes to use absolute paths with @self for compatibility inside the extends block.

@mthalman mthalman enabled auto-merge (squash) February 9, 2026 20:26
@akoeplinger
Copy link
Member

/azp run sdk-source-build

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mthalman mthalman merged commit d71d920 into dotnet:release/9.0.1xx Feb 10, 2026
14 checks passed
@mthalman mthalman deleted the license-scan branch February 10, 2026 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants