Skip to content

Restore azuredeploy.json for traffic-manager-linked-record (fork-gap #14769 from #14875) - #14876

Open
Alex Frankel (alex-frankel) wants to merge 7 commits into
masterfrom
restore-azuredeploy-14875
Open

Alex Frankel (alex-frankel) wants to merge 7 commits into
masterfrom
restore-azuredeploy-14875

Conversation

@alex-frankel

@alex-frankel Alex Frankel (alex-frankel) commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

PR #14875 (fork) merged without its compiled azuredeploy.json/prereq.azuredeploy.json because commit-generated-on-merge skips fork PRs (#14769), leaving the sample's Deploy-to-Azure button broken on master.

This upstream-branch PR re-touches main.bicep (+ a no-op metadata.json touch for CHECK 0b) so CI regenerates azuredeploy.json via the same bicep-version-pinned path that validated #14875, and commit-generated-on-merge (which works for upstream branches) commits it on merge. testResult is unchanged, so ADX re-validation matches the original deployment records.

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

@alex-frankel

Copy link
Copy Markdown
Contributor Author

/validate

@alex-frankel Alex Frankel (alex-frankel) changed the title Restore azuredeploy.json for traffic-manager-linked-record (fork-gap #14769 from PR #14875) Restore azuredeploy.json for traffic-manager-linked-record (fork-gap #14769 from #14875) Sep 21, 2026
@alex-frankel

Copy link
Copy Markdown
Contributor Author

/validate

@alex-frankel

Copy link
Copy Markdown
Contributor Author

/validate

@asudbring

Copy link
Copy Markdown
Contributor

Alex Frankel (@alex-frankel)
Can you see why this is failing? I can try and fix if I can see the error. I have an article waiting on merge for this one to get merged as it references the file in this PR. Once this is merged I can have that other one merged. Let me know what I can do to speed this along.

Thanks!
allen

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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants