Workflows/ci hygiene - #609
Open
MaddyMicrosoft wants to merge 6 commits into
Open
Conversation
The canary and integration-test workflows still pinned azure/login@v1 while the action is on v3, so they were validating a two-major-old release instead of the current one. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The slack-post-result jobs posted to a Slack webhook that is no longer used, and referenced a secret with broken interpolation. Removing them; the test jobs themselves are unchanged. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
pr-check.yml duplicated ci.yml's build+test (ci.yml already runs on master PRs across windows and ubuntu). Removing the redundant workflow. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
These suites perform real Azure logins. Triggering on every push to any branch ran the full matrix (and consumed live credentials) for incidental commits. Switch to pull_request targeting master, keeping workflow_dispatch for manual runs. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
MaddyMicrosoft
requested a deployment
to
Automation test
August 14, 2026 03:10 — with
GitHub Actions
Waiting
MaddyMicrosoft
requested a deployment
to
Automation test
August 14, 2026 03:10 — with
GitHub Actions
Waiting
MaddyMicrosoft
requested a deployment
to
Automation test
August 14, 2026 03:10 — with
GitHub Actions
Waiting
MaddyMicrosoft
requested a deployment
to
Automation test
August 14, 2026 03:10 — with
GitHub Actions
Waiting
MaddyMicrosoft
requested a deployment
to
Automation test
August 14, 2026 03:10 — with
GitHub Actions
Waiting
MaddyMicrosoft
requested a deployment
to
Automation test
August 14, 2026 03:10 — with
GitHub Actions
Waiting
MaddyMicrosoft
requested a deployment
to
Automation test
August 14, 2026 03:10 — with
GitHub Actions
Waiting
MaddyMicrosoft
requested a deployment
to
Automation test
August 14, 2026 03:10 — with
GitHub Actions
Waiting
MaddyMicrosoft
requested a deployment
to
Automation test
August 14, 2026 03:10 — with
GitHub Actions
Waiting
MaddyMicrosoft
requested a deployment
to
Automation test
August 14, 2026 03:10 — with
GitHub Actions
Waiting
MaddyMicrosoft
requested a deployment
to
Automation test
August 14, 2026 03:10 — with
GitHub Actions
Waiting
MaddyMicrosoft
marked this pull request as ready for review
August 14, 2026 06:53
- positive/negative: add setup/teardown jobs that create and delete GitHubAction_CI_Group via azure/login@v3 (stable), gated on all jobs - drop az vm list / Get-AzVM assertions (no VM is provisioned) - negative permission tests keep targeting the never-created GitHubAction_CI_RG so they fail as expected - shared concurrency group so positive/negative don't collide on the RG
MaddyMicrosoft
had a problem deploying
to
Automation test
August 14, 2026 07:29 — with
GitHub Actions
Error
MaddyMicrosoft
had a problem deploying
to
Automation test
August 14, 2026 07:29 — with
GitHub Actions
Failure
MaddyMicrosoft
had a problem deploying
to
Automation test
August 14, 2026 07:39 — with
GitHub Actions
Failure
MaddyMicrosoft
had a problem deploying
to
Automation test
August 14, 2026 07:39 — with
GitHub Actions
Failure
MaddyMicrosoft
had a problem deploying
to
Automation test
August 14, 2026 07:39 — with
GitHub Actions
Failure
MaddyMicrosoft
had a problem deploying
to
Automation test
August 14, 2026 07:39 — with
GitHub Actions
Error
MaddyMicrosoft
had a problem deploying
to
Automation test
August 14, 2026 07:39 — with
GitHub Actions
Error
MaddyMicrosoft
had a problem deploying
to
Automation test
August 14, 2026 07:39 — with
GitHub Actions
Error
MaddyMicrosoft
had a problem deploying
to
Automation test
August 14, 2026 07:39 — with
GitHub Actions
Error
MaddyMicrosoft
had a problem deploying
to
Automation test
August 14, 2026 07:46 — with
GitHub Actions
Error
MaddyMicrosoft
had a problem deploying
to
Automation test
August 14, 2026 07:46 — with
GitHub Actions
Failure
Replace azure-login-positive.yml and azure-login-negative.yml with a single azure-login-live-tests.yml. All 49 positive and 45 negative test steps are preserved unchanged; only structure and provisioning change.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Cleanup of the Azure Login CI/test workflows: removes dead/redundant config, refreshes a stale action pin, and stops running live-credential tests on every push.
Changes