CICD: Derive integration test provider list from profiles.json#4245
Merged
TomOnTime merged 3 commits intoMay 12, 2026
Conversation
- Replace hardcoded PROVIDERS array with dynamic read from `integrationTest/profiles.json`. - Add sparse-checkout step to fetch only the profiles file. - Existing filtering on `*_DOMAIN` env vars remains intact, so providers without configured secrets are skipped automatically.
…n_tests.yml`. - Remove instructions to manually add providers to the PROVIDERS list in `writing-providers.md`, `byo-secrets.md` and `release-engineering.md`. - Update checklist and FAQ to reflect the new automated approach.
…n_tests.yml`. - Use `-AsHashtable` flag because `profiles.json` contains keys with mixed casing (`TYPE` vs `domain`), which PowerShell rejects without this flag.
TomOnTime
approved these changes
May 12, 2026
Collaborator
TomOnTime
left a comment
TomOnTime
left a comment
There was a problem hiding this comment.
Wow! That's awesome!
The fact that this deletes a lot of docs shows how much better this is.
This was referenced May 13, 2026
CICD: Make it easier to refer to the list of providers (GitHub workflows PR integration tests)
#4241
Closed
TomOnTime
pushed a commit
that referenced
this pull request
May 16, 2026
The repository has `provider-*` labels that map provider type names (e.g. `CLOUDFLAREAPI`) to issues and pull requests, but these were assigned manually. For pull requests this can be automated by matching changed files against `providers/` directories. This GitHub pull request adds a `go generate` step that builds `.github/labeler.yml` from the registered provider constants. A `pr_labeler.yml` workflow runs `actions/labeler@v5` on `pull_request_target` to apply the labels automatically. New providers are picked up on the next `go generate` run, similar to #4255 (GoReleaser) and #4245 (integration tests).
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.
Replaces the hardcoded
PROVIDERSarray inpr_integration_tests.ymlwith a dynamic read fromintegrationTest/profiles.json. New providers no longer need to be added to a separate list in the workflow file.The existing filtering on
*_DOMAINenv vars remains intact, so providers without configured secrets are skipped automatically. For reference,profiles.jsoncurrently has 57 providers while the old hardcoded list only had 28. The missing 29 are simply skipped since they have no secrets configured in CI.Documentation in
writing-providers.md,byo-secrets.mdandrelease-engineering.mdupdated to remove instructions about manually adding providers to the PROVIDERS list.Relates to #4241 (GHA workflow part only; the GoReleaser regexp is out of scope for this PR).