Skip to content

Fix path for post-build template and adjust validation#7615

Merged
nohwnd merged 3 commits intomicrosoft:mainfrom
nohwnd:fix-postbuild-template
Mar 27, 2026
Merged

Fix path for post-build template and adjust validation#7615
nohwnd merged 3 commits intomicrosoft:mainfrom
nohwnd:fix-postbuild-template

Conversation

@nohwnd
Copy link
Copy Markdown
Member

@nohwnd nohwnd commented Mar 27, 2026

Same fix as microsoft/vstest#15592 — fix backslash path for post-build template reference and temporarily skip signing validation until infra issues are resolved, based on advice from Arcade.

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

Copilot AI review requested due to automatic review settings March 27, 2026 08:47
- Fix backslash path to forward slash for template reference
- Temporarily skip signing validation until infra issues are resolved,
  based on advice from Arcade.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@nohwnd nohwnd force-pushed the fix-postbuild-template branch from 690c768 to 15ecc98 Compare March 27, 2026 08:48
Copy link
Copy Markdown
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

Updates the official Azure Pipelines definition to correctly reference the post-build template using a forward-slash path and disables signing validation as a temporary workaround for current infrastructure issues (per Arcade guidance).

Changes:

  • Fix template include path formatting for the post-build template.
  • Pass enableSigningValidation: false to temporarily skip signing validation in post-build.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings March 27, 2026 09:01
@nohwnd nohwnd merged commit fe45711 into microsoft:main Mar 27, 2026
2 checks passed
Copy link
Copy Markdown
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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

LclPackageId: 'LCL-JUNO-PROD-TESTFX'

- template: eng\common\templates-official\post-build\post-build.yml@self
- template: eng/common/templates-official/post-build/post-build.yml@self
Copy link

Copilot AI Mar 27, 2026

Choose a reason for hiding this comment

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

Template includes in this pipeline generally use an absolute repo-root path (leading /), but this one is still relative (eng/common/...). To keep things consistent with the other template references in this file (and avoid any path-resolution ambiguity if the file is ever relocated), consider switching this to /eng/common/templates-official/post-build/post-build.yml@self.

Suggested change
- template: eng/common/templates-official/post-build/post-build.yml@self
- template: /eng/common/templates-official/post-build/post-build.yml@self

Copilot uses AI. Check for mistakes.
Comment on lines +257 to +259
parameters:
# Temporarily disable signing validation until infra issues are resolved, based on advice from Arcade.
enableSigningValidation: False
Copy link

Copilot AI Mar 27, 2026

Choose a reason for hiding this comment

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

Disabling signing validation on the official post-build stage is a significant safety/quality reduction. If this is truly temporary, please add a tracking link (issue/incident) and/or a clear re-enable mechanism (e.g., a pipeline parameter/variable toggle with a default of True) so we don’t accidentally ship long-term with signing validation turned off.

Copilot uses AI. Check for mistakes.
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.

3 participants