Skip to content

Sync eng/common directory with azure-sdk-tools for PR 17436 - #10395

Open
azure-sdk-automation[bot] wants to merge 9 commits into
mainfrom
sync-eng-common-joyer/author2common-template-support-17436
Open

azure-sdk-automation[bot] wants to merge 9 commits into
mainfrom
sync-eng-common-joyer/author2common-template-support-17436

Conversation

@azure-sdk-automation

Copy link
Copy Markdown
Contributor

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

@azure-sdk-automation
azure-sdk-automation Bot requested review from JoyerJin and a balanced review from Copilot October 30, 2026 02:50
@azure-sdk-automation
azure-sdk-automation Bot requested review from a team as code owners October 30, 2026 02:50
@azure-sdk-automation azure-sdk-automation Bot added EngSys This issue is impacting the engineering system. Central-EngSys This issue is owned by the Engineering System team. labels Oct 30, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
1 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No active review issues block the configurable shard execution and multi-track evaluation changes.

No open issues 🎉

What changed in this PR

Syncs shared evaluation infrastructure in eng/common from Azure/azure-sdk-tools PR #17436, adding configurable shard execution and support for multiple evaluation tracks in one build.

Changes:

  • Makes Azure authentication connections caller-configured while preserving the live pipeline’s explicit connection.
  • Adds validated supplementary Vally arguments passed directly to Node without shell interpretation.
  • Adds configurable stage names and artifact namespaces, with tests and documentation for track-specific result selection.
File Description
eng/​common/​scripts/​eval/​test/​verdict.test.ts Tests authentication configuration, argument validation, and literal argument forwarding.
eng/​common/​scripts/​eval/​test/​shard-results.test.ts Tests track-specific artifacts, retry selection, and namespace validation.
eng/​common/​scripts/​eval/​README.md Documents shard customization, authentication, and multi-track contracts.
eng/​common/​scripts/​eval/​lib/​shard-results.ts Supports configurable evaluation stages and namespaced result selection.
eng/​common/​scripts/​eval/​invoke-eval-shard.ts Validates supplementary arguments and invokes the CLI directly through Node.
eng/​common/​scripts/​eval/​build-eval-summary.ts Passes the artifact namespace into result selection.
eng/​common/​pipelines/​templates/​steps/​eval-invoke.yml Validates authentication configuration and forwards arguments as JSON.
eng/​common/​pipelines/​templates/​stages/​archetype-eval.yml Exposes and forwards service connection and argument parameters.
eng/​common/​pipelines/​templates/​jobs/​eval-summarize.yml Configures track-specific summaries and rejects namespaced dashboard output.
eng/​common/​pipelines/​templates/​jobs/​eval-shard.yml Adds matrix-stage, artifact-namespace, and invocation parameters.
eng/​common/​pipelines/​live-eval.yml Explicitly supplies the existing Azure service connection.

Review effort: Balanced

@azure-sdk-automation
azure-sdk-automation Bot force-pushed the sync-eng-common-joyer/author2common-template-support-17436 branch 5 times, most recently from 77c7629 to 67f60a9 Compare November 5, 2026 09:15
@azure-sdk-automation
azure-sdk-automation Bot force-pushed the sync-eng-common-joyer/author2common-template-support-17436 branch 2 times, most recently from 00d3960 to 205d65c Compare November 6, 2026 09:17
@azure-sdk-automation
azure-sdk-automation Bot force-pushed the sync-eng-common-joyer/author2common-template-support-17436 branch from 205d65c to 3dee547 Compare November 9, 2026 07:19

This branch has not been deployed

No deployments
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.

2 participants