Skip to content

Pipelines: exclude in-repo .ps1 scripts from CodeSign post-analysis#40541

Merged
benhillis merged 1 commit into
microsoft:masterfrom
benhillis:ben/sdl-codesign-ps1-exclude-pr
May 15, 2026
Merged

Pipelines: exclude in-repo .ps1 scripts from CodeSign post-analysis#40541
benhillis merged 1 commit into
microsoft:masterfrom
benhillis:ben/sdl-codesign-ps1-exclude-pr

Conversation

@benhillis
Copy link
Copy Markdown
Member

Summary

The Guardian CodeSign post-analysis task has been failing the release build on .ps1 files in the source tree (diagnostic / dev / test helpers) that we don't ship. The OneBranch targetGlob for actual signing already restricts signing to .dll/.exe/.sys/.msi/.msix/.appx/.nupkg, so PowerShell scripts being flagged here is purely a CodeSign validation noise issue.

Change

Set ob_sdl_codeSignValidation_excludes to skip **\*.ps1 on the three OneBranch pipelines (release, nightly, PR), and combine with the existing testbin\** exclusion in build-job.yml so neither rule clobbers the other.

Files

  • .pipelines/build-job.yml — extend existing exclude list with **\*.ps1
  • .pipelines/wsl-build-release-onebranch.yml — add pipeline-level exclude
  • .pipelines/wsl-build-nightly-onebranch.yml — add pipeline-level exclude
  • .pipelines/wsl-build-pr-onebranch.yml — add pipeline-level exclude

Validation

Dry-run pipeline (with test+flight stages skipped) is running against the same change on a sibling branch in parallel; opening this PR now to get review going while that finishes.

Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com

@benhillis benhillis requested a review from a team as a code owner May 14, 2026 20:38
@benhillis benhillis force-pushed the ben/sdl-codesign-ps1-exclude-pr branch from 84b4535 to e019c7d Compare May 14, 2026 20:45
@benhillis
Copy link
Copy Markdown
Member Author

This should fix the pipeline issue we've been seeing, I'm running a private release build to see.

@benhillis benhillis force-pushed the ben/sdl-codesign-ps1-exclude-pr branch 2 times, most recently from 90f52f1 to a01c6f2 Compare May 14, 2026 23:32
@benhillis benhillis enabled auto-merge (squash) May 15, 2026 02:09
The Guardian CodeSign tool was scanning all .ps1 files in the source tree
and failing the release build on diagnostic / dev / test helper scripts that
aren't shipped (the OneBranch codesign targetGlob already restricts signing
to .dll/.exe/.sys/.msi/.msix/.appx/.nupkg).

Set ob_sdl_codeSignValidation_excludes to skip **\*.ps1 at the pipeline
level for the release, nightly, and PR OneBranch pipelines, and combine
with the existing testbin\** exclude in build-job.yml.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@benhillis benhillis force-pushed the ben/sdl-codesign-ps1-exclude-pr branch from a01c6f2 to e33c0b8 Compare May 15, 2026 02:10
@benhillis benhillis merged commit b011cf7 into microsoft:master May 15, 2026
9 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