merge queue: embarking main (1caca40) and #1382 together#1454
Closed
mergify[bot] wants to merge 2 commits into
Closed
merge queue: embarking main (1caca40) and #1382 together#1454mergify[bot] wants to merge 2 commits into
mergify[bot] wants to merge 2 commits into
Conversation
Wraps two CI Insights endpoints into a single batch command:
- `GET /v1/ci/{owner}/repositories/{repo}/search/tests` resolves test
identities by name (glob-aware) on the default branch. Filters
(`test_name`, `pipeline_name`, `job_name`, …) travel as repeated
query parameters; page size travels as `per_page`.
- `GET /v1/ci/{owner}/repositories/{repo}/tests/{test_id}` returns the
full health/metrics payload for one identity.
The search is a true batch API — pass one or more `<NAME>` positionals
(globs allowed) and one block per match is rendered. `--json` emits a
single `{"tests": [...]}` document; the human renderer hides metadata
lines for absent fields rather than printing placeholders. Exit code
reflects the worst health observed across results (0 = healthy or
unknown, 1 = any flaky, 6 = any broken).
`split_owner_repo` lives in `detector.rs` next to the existing
`owner/repo` validators so callers can interpolate the segments into
request paths without re-escaping. A new `HttpClient::get_with_query`
helper percent-encodes values and preserves repeated keys in caller
order, which the search endpoint relies on for `test_name` repetition.
Also adds a live-smoke case that exercises the search endpoint with a
guaranteed-nonexistent name so the round-trip stays independent of
canary repository state.
Fixes: MRGFY-7166
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Change-Id: Iafe286495b7842079e9c63b437a6305926fc22a5
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
1 participant
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.
🎉 This pull request has been checked successfully and will be merged soon. 🎉
Branch main (1caca40) and #1382 are embarked together for merge.
This pull request has been created by Mergify to speculatively check the mergeability of #1382.
You don't need to do anything. Mergify will close this pull request automatically when it is complete.
Required conditions of queue rule
defaultfor merge:title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert|ui)(?:\(.+\))?:mergify tests showcommand #1382#approved-reviews-by>=2mergify tests showcommand #1382author = dependabot[bot]mergify tests showcommand #1382author = mergify-ci-botmergify tests showcommand #1382author = renovate[bot]mergify tests showcommand #1382body ~= (?ms:.{48,})mergify tests showcommand #1382#changes-requested-reviews-by = 0mergify tests showcommand #1382#review-requested = 0mergify tests showcommand #1382#review-threads-unresolved = 0mergify tests showcommand #1382check-success=ci-gateRequired conditions to stay in the queue:
base=mainmergify tests showcommand #1382label!=manual mergemergify tests showcommand #1382title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert|ui)(?:\(.+\))?:mergify tests showcommand #1382#approved-reviews-by>=2mergify tests showcommand #1382author = dependabot[bot]mergify tests showcommand #1382author = mergify-ci-botmergify tests showcommand #1382author = renovate[bot]mergify tests showcommand #1382body ~= (?ms:.{48,})mergify tests showcommand #1382#changes-requested-reviews-by = 0mergify tests showcommand #1382#review-requested = 0mergify tests showcommand #1382#review-threads-unresolved = 0mergify tests showcommand #1382check-success=ci-gatemergify tests showcommand #1382