Skip to content

Sync eng/common directory with azure-sdk-tools for PR 17084 - #5435

Merged
Helen Gao (helen229) merged 21 commits into
mainfrom
sync-eng-common-feat/eval-blob-publishing-17084
Oct 19, 2026
Merged

Helen Gao (helen229) merged 21 commits into
mainfrom
sync-eng-common-feat/eval-blob-publishing-17084

Conversation

@azure-sdk-automation

Copy link
Copy Markdown
Contributor

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

Copilot AI balanced review requested due to automatic review settings October 9, 2026 19:58
@azure-sdk-automation
azure-sdk-automation Bot requested review from a team as code owners October 9, 2026 19:58
@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 9, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
2 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.

🟡 Changes recommended

Redirect retry handling is incorrect, and the repository’s 1ES redirect lacks the required Azure Storage parameter.

3 open findings
What changed in this PR

Syncs shared evaluation infrastructure from azure-sdk-tools PR 17084, adding complete result collection and optional dashboard publication.

Changes:

  • Stages and validates latest shard attempts for complete summaries.
  • Builds immutable result bundles and publishes them to Azure Blob Storage.
  • Adds pipeline gating, documentation, dependencies, and comprehensive tests.
File Description
eng/​common/​scripts/​eval/​test/​shard-results.test.ts Tests shard staging and selection.
eng/​common/​scripts/​eval/​README.md Documents publication and recovery contracts.
eng/​common/​scripts/​eval/​publisher/​test/​publisher.test.ts Tests bundling, publication, and pipeline gates.
eng/​common/​scripts/​eval/​publisher/​storage.ts Implements Blob upload and dashboard notification.
eng/​common/​scripts/​eval/​publisher/​publish-bundle.ts Adds the publication CLI.
eng/​common/​scripts/​eval/​publisher/​package.json Defines publisher dependencies and tests.
eng/​common/​scripts/​eval/​publisher/​package-lock.json Locks publisher dependencies.
eng/​common/​scripts/​eval/​publisher/​bundle.ts Creates and validates result bundles.
eng/​common/​scripts/​eval/​package.json Narrows the evaluation test pattern.
eng/​common/​scripts/​eval/​lib/​shard-results.ts Stages and selects authoritative shard attempts.
eng/​common/​scripts/​eval/​build-eval-summary.ts Adds completeness-aware summaries.
eng/​common/​pipelines/​workflow-eval.yml Enables gated workflow publication.
eng/​common/​pipelines/​templates/​variables/​eval-dashboard.yml Defines trusted publication eligibility.
eng/​common/​pipelines/​templates/​steps/​eval-publish-results.yml Adds bundle and upload steps.
eng/​common/​pipelines/​templates/​stages/​archetype-eval.yml Wires publication into evaluation stages.
eng/​common/​pipelines/​templates/​jobs/​eval-summarize.yml Consolidates authoritative results.
eng/​common/​pipelines/​templates/​jobs/​eval-shard.yml Publishes complete per-shard artifacts.
eng/​common/​pipelines/​skill-eval.yml Enables gated skill-evaluation publication.
eng/​common/​pipelines/​live-eval.yml Enables gated live-evaluation publication.
Files not reviewed (1)
  • eng/common/scripts/eval/publisher/package-lock.json: Generated file

🧠 Review effort: Balanced


💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

Comment thread eng/common/scripts/eval/publisher/test/publisher.test.ts Outdated
Comment thread eng/common/scripts/eval/lib/shard-results.ts
Comment thread eng/common/scripts/eval/publisher/storage.ts Outdated
@azure-sdk-automation
azure-sdk-automation Bot force-pushed the sync-eng-common-feat/eval-blob-publishing-17084 branch 4 times, most recently from 9f7fe59 to 99bd65a Compare October 16, 2026 18:36
@azure-sdk-automation
azure-sdk-automation Bot force-pushed the sync-eng-common-feat/eval-blob-publishing-17084 branch from 99bd65a to d6d4c23 Compare October 16, 2026 19:16
@helen229
Helen Gao (helen229) merged commit 17cf1c6 into main Oct 19, 2026
15 checks passed
@helen229
Helen Gao (helen229) deleted the sync-eng-common-feat/eval-blob-publishing-17084 branch October 19, 2026 21:01
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