Skip to content

Record Depot registry canary results - #1159

Merged
ndizazzo merged 1 commit into
mainfrom
agent/record-depot-registry-canaries
Aug 3, 2026
Merged

ndizazzo merged 1 commit into
mainfrom
agent/record-depot-registry-canaries

Conversation

@ndizazzo

@ndizazzo ndizazzo commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • record all seven valid five-versus-five registry cohorts
  • retain exact digests and workflow run evidence
  • document that no mapping met the gate and both rollout variables remain disabled

Validation

  • git diff --check

Summary by CodeRabbit

  • Documentation
    • Added initial measurement results for registry image pull-through performance.
    • Results cover seven images tested across five fresh runners per source, including digest verification.
    • No tested image met the adoption threshold.
    • Cache readiness checks remain unmet.
    • Alpine results are not included, and runs affected by authentication failures were excluded.

@coderabbitai

coderabbitai Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: ab6b0e92-fd07-43fc-b8d3-39f0fdff0ab9

📥 Commits

Reviewing files that changed from the base of the PR and between 30fa060 and b205629.

📒 Files selected for processing (1)
  • ci/METRICS.md

📝 Walkthrough

Walkthrough

The change documents seven Depot Registry pull-through measurement cohorts. It records median results, gate outcomes, cache status, and excluded Alpine and authentication-failure runs.

Changes

Depot Registry measurements

Layer / File(s) Summary
Measurement results and gate status
ci/METRICS.md
Documents seven valid cohorts with upstream and Depot medians, relative results, run links, failed adoption gates, false cache gates, and excluded runs.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the PR's main change: recording Depot registry canary results.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch agent/record-depot-registry-canaries

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ndizazzo
ndizazzo merged commit 2e6748a into main Aug 3, 2026
38 checks passed
@ndizazzo
ndizazzo deleted the agent/record-depot-registry-canaries branch August 3, 2026 01:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant