Skip to content

Sync eng/common directory with azure-sdk-tools for PR 14420#37620

Closed
azure-sdk wants to merge 8 commits intomainfrom
sync-eng/common-azsdk-benchmark-pipeline-14420
Closed

Sync eng/common directory with azure-sdk-tools for PR 14420#37620
azure-sdk wants to merge 8 commits intomainfrom
sync-eng/common-azsdk-benchmark-pipeline-14420

Conversation

@azure-sdk
Copy link
Collaborator

Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#14420 See eng/common workflow

@azure-sdk azure-sdk requested a review from a team as a code owner March 12, 2026 19:46
@azure-sdk azure-sdk requested review from Copilot and jeo02 March 12, 2026 19:46
@azure-sdk azure-sdk added EngSys This issue is impacting the engineering system. Central-EngSys This issue is owned by the Engineering System team. labels Mar 12, 2026
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

Adds a new shared Azure Pipelines job template intended to run azsdk-cli benchmark scenarios via dotnet run, with parameters for scenario selection, tagging, model override, and parallelism.

Changes:

  • Introduces benchmark-job.yml job template with configurable benchmark project/scenario inputs.
  • Uses sparse-checkout.yml to fetch only the needed repo paths before executing benchmarks.
  • Runs the benchmark harness with a computed CLI argument list (scenario name vs tags vs --all).

- '${{ parameters.BenchmarkProject }}/**'
- 'eng/common/**'

- template: /eng/pipelines/templates/steps/install-dotnet.yml
Copy link

Copilot AI Mar 12, 2026

Choose a reason for hiding this comment

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

The template references /eng/pipelines/templates/steps/install-dotnet.yml, but that file does not exist in this repo (there is no eng/pipelines/templates/steps/install-dotnet.yml in eng/pipelines/templates/steps/). This will break the job at template-expansion time. Consider inlining the UseDotNet@2 tasks (as done in eng/common/pipelines/templates/jobs/ai-eval-job.yml) or referencing an existing shared template path.

Suggested change
- template: /eng/pipelines/templates/steps/install-dotnet.yml
- task: UseDotNet@2
displayName: 'Install .NET SDK'
inputs:
packageType: 'sdk'
useGlobalJson: true

Copilot uses AI. Check for mistakes.
@azure-sdk azure-sdk force-pushed the sync-eng/common-azsdk-benchmark-pipeline-14420 branch from 6727269 to 3b4b124 Compare March 17, 2026 18:29
@jeo02 jeo02 closed this Mar 17, 2026
@azure-sdk azure-sdk deleted the sync-eng/common-azsdk-benchmark-pipeline-14420 branch March 22, 2026 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants