Skip to content

[release/9.0.1xx] Restrict CI package publishing to Windows Release - #10502

Merged
nagilson merged 2 commits into
release/9.0.1xxfrom
mmitche-publishing-backport-9-0-1xx
Sep 24, 2026
Merged

nagilson merged 2 commits into
release/9.0.1xxfrom
mmitche-publishing-backport-9-0-1xx

Conversation

@mmitche

@mmitche mmitche commented Sep 21, 2026

Copy link
Copy Markdown
Member

Problem

Backport #10496 to release/9.0.1xx. Multiple OS/configuration legs can publish the same packages to shared PackageArtifacts/BlobArtifacts containers.

Solution

Semantically adapt the publishing restriction to this release branch rather than blindly cherry-picking main, because this branch lacks main's existing Debug suppression:

  • In azure-pipelines-pr.yml, add Windows Debug suppression and disable publishing for every Linux/macOS configuration, leaving Windows Release publishing unchanged.
  • In azure-pipelines.yml, suppress publishing for non-Release Windows configurations on both PR and non-PR paths and for every Linux/macOS configuration.
  • Preserve this branch's older official product pipeline, signing, source-build, build/pack/test/integration-test coverage, log/test-result publishing, localization, and legacy pipeline-publishing settings. Do not introduce main's newer assetless official-path structure.

The CI wrappers inject Publish=true; the trailing /p:Publish=false property overrides it. Reuses the existing backport commit 203ebb5413c16b9fceb756c363f971744d64909d unchanged.

Checks:

  • Added unit tests

YAML-only change. Refreshed origin/release/9.0.1xx and verified it remains an ancestor of the backport. Reused the existing release validator, restricted to this branch: 96 local YAML scenarios / 388 legs passed, covering public/internal, PR/non-PR, both pipeline definition names, main/release source-branch conditions, and Release/Debug/Checked configurations (including Debug-only). Only Windows Release publishes when present; Debug-only configurable runs have no publisher. All non-publishing expanded YAML and build-leg coverage were preserved. Also parsed both YAML files, checked wrapper argument ordering, verified exact two-file scope (+15 lines), and passed scoped git diff --check.

Validation uses a local expression evaluator, not Azure server expansion. External templates were not expanded; no dependencies installed, full builds, or live CI retries were performed.

Adapt main commit 44cea09, including Debug suppression missing from this release branch. Preserve build and test coverage, signing, and per-leg diagnostic artifacts.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@mmitche
mmitche requested a review from a team as a code owner September 21, 2026 21:15
@nagilson
nagilson enabled auto-merge September 24, 2026 22:31
@nagilson
nagilson merged commit 3e39c39 into release/9.0.1xx Sep 24, 2026
10 checks passed
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.

2 participants