Repository navigation
π Hub: keep channel target probes off my-hives - #11439
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Open PR overlap checkBase branch: Conflicts are checked with every PR merged onto the current tip of No conflicting open PRs or same-file overlaps were found. |
Signed-off-by: Andrew Anderson <andy@clubanderson.com>
bdc59b6 to
d67ee73
Compare
|
merged by Hive trusted-author auto-merge: author @clubanderson holds role owner and GitHub permission admin on repo; CI green at d67ee73. |
|
Thank you for your contribution! Your PR has been merged. We'd love to hear how your experience was: share feedback |
Follow-up to #11429. Fixes #11427.
Summary
/api/saas/my-hivesrequest-path stall: per-row release-channel target resolution no longer runs synchronous GitHub commit-list + GHCR manifest probes frombehindTargetFor(branch, channel, floating SHA)Evidence
Live hub logs during the slow window were dominated by GitHub commit-list timeouts, e.g.
SHA poll: commit list request failed ... context deadline exceeded. The synchronous path washandleMyHives->behindTargetFor->channelPublishedImageTarget->listChannelBranchCommits/probeSpokeImageTag, once for release-channel hives.Validation
gofmt -l src/pkg/hub/behind_target.go src/pkg/hub/channel_image_target.go src/pkg/hub/saas_hives_api.go src/pkg/hub/server.gogit diff --checkbash src/scripts/check-api-reference-citations.sh --fix(no drift)Per AGENTS.md, CI is the Go test/coverage/vet verdict.