ci-azure - Skip the Azure runner matrix on Dependabot PRs (do Get-DbaBuild) - #10674
Closed
potatoqualitee wants to merge 1 commit into
Closed
ci-azure - Skip the Azure runner matrix on Dependabot PRs (do Get-DbaBuild)#10674potatoqualitee wants to merge 1 commit into
potatoqualitee wants to merge 1 commit into
Conversation
Dependabot bumps only touch action pins, and the earlier fixes (#10566, 53ea454) gated azure-migration-tests instead of ci-azure, which is the workflow that actually acquires the Azure VMSS pools and went red on #10663. Gate authorize (every matrix job needs it) and the supersede helper on the actor, the same way azure-migration-tests already does. (do Get-DbaBuild) Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Mye3A1CnBn1pnEQnFEE7Ze
potatoqualitee
requested review from
andreasjordan and
niphlod
as code owners
September 2, 2026 20:15
Member
Author
|
Superseded: the same commit was pushed directly onto the Dependabot branch in #10663. |
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
2 participants
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.
Dependabot PRs such as #10663 still fire the full
ci-azurematrix on the self-hosted Azure VMSS pools. The two earlier attempts (#10566 and 53ea454) both patchedazure-migration-tests.yml, which already skips Dependabot correctly, but never touchedci-azure.yml.This gates the
authorizejob ongithub.actor != 'dependabot[bot]'. Every matrix job depends onauthorize, so the whole workflow completes as skipped in seconds and never touches a runner. Theci-azure-supersedehelper gets the same gate since there is nothing for it to cancel.Dependabot bumps only change action pins; the GitHub-hosted workflows (xplat import, cross-platform tests, security lint) still run on them and are enough coverage.
(do Get-DbaBuild)
馃 Generated with Claude Code
https://claude.ai/code/session_01Mye3A1CnBn1pnEQnFEE7Ze