Skip to content

chore: pull the Sourcebot image from ghcr.io instead of docker.sourcebot.dev - #153

Merged
brendan-kellam merged 2 commits into
mainfrom
brendan/sou-2804-update-docs-and-setup-sourcebot-to-use-ghcr-image-urls
Nov 3, 2026
Merged

brendan-kellam merged 2 commits into
mainfrom
brendan/sou-2804-update-docs-and-setup-sourcebot-to-use-ghcr-image-urls

Conversation

@brendan-kellam

@brendan-kellam brendan-kellam commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

Refs SOU-2804

docker.sourcebot.dev is a third-party pull-tracking proxy. When it falls back to its CloudFront stack, it answers registry requests with a 307 redirect to ghcr.io instead of proxying them. Podman, skopeo, Buildah and CRI-O (OpenShift) drop the bearer token on that cross-host redirect, so pulls fail with unauthorized. Docker happens to work because it carries headers across redirects. Pulling from ghcr.io directly works for every client.

Existing installs that pin docker.sourcebot.dev keep working with Docker; the DNS record is unchanged.

Changes

  • sourcebot.image.repository default is now ghcr.io/sourcebot-dev/sourcebot (values.yaml, regenerated README.md).
  • Updated the two image assertions in tests/basic_test.yaml.

Users who set sourcebot.image.repository explicitly are unaffected.

🤖 Generated with Claude Code


Note

Low Risk
Default-only registry change with no application or auth logic changes; explicit repository overrides behave as before.

Overview
Changes the Helm chart’s default sourcebot.image.repository from docker.sourcebot.dev/sourcebot-dev/sourcebot to ghcr.io/sourcebot-dev/sourcebot, so new installs pull the image directly from GitHub Container Registry instead of the proxy that can redirect to ghcr.io and break auth on CRI-O (OpenShift), Podman, and similar clients.

Docs and tests are aligned: values.yaml, regenerated README.md, basic_test.yaml image regex assertions, and an [Unreleased] changelog entry. Deployments that override sourcebot.image.repository are unchanged.

Reviewed by Cursor Bugbot for commit 9064488. Bugbot is set up for automated code reviews on this repo. Configure here.

Summary by CodeRabbit

  • Changes
    • The default Sourcebot image repository is now ghcr.io/sourcebot-dev/sourcebot. This may help avoid image-pull errors with CRI-O (OpenShift) and Podman.

brendan-kellam and others added 2 commits November 3, 2026 13:12
…bot.dev

docker.sourcebot.dev is a third-party pull-tracking proxy. Its fallback
mode answers with a 307 redirect to ghcr.io, and Podman, skopeo, Buildah
and CRI-O (OpenShift) drop the bearer token on that cross-host redirect,
so pulls fail with "unauthorized". Pulling from ghcr.io directly works.

Refs SOU-2804

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Refs SOU-2804

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Walkthrough

The chart’s default image repository changed to ghcr.io/sourcebot-dev/sourcebot. The README, image-tag and image-digest test expectations, and changelog reflect the change.

Changes

Image repository update

Layer / File(s) Summary
Update chart image repository
charts/sourcebot/values.yaml, charts/sourcebot/README.md, charts/sourcebot/tests/basic_test.yaml, CHANGELOG.md
The chart default and documented repository changed to ghcr.io/sourcebot-dev/sourcebot. The image-tag and image-digest tests expect the new registry. The changelog notes the change and mentions unauthorized pull errors on CRI-O (OpenShift) and Podman.

Estimated code review effort: 2 (Simple) | ~8 minutes

Merge Risk: Minimal · up to 90644

The chart and tests consistently reference the new repository; no merge-blocking issue was identified.

Pre-merge checks: Passed 5
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 main change: updating the default Sourcebot image registry from docker.sourcebot.dev to ghcr.io.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
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.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (4 skipped: 4 unsupported.)

Finishing Touches
Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

CodeRabbit Verify

  • Run CodeRabbit Verify to test the changes in this pull request.
Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: ced14ac5-5da9-41fe-93a9-a93e3e57378b
Commits

Reviewing files that changed from the base of the PR and between e11df97 and 9064488.

Files selected for processing: 4
  • CHANGELOG.md
  • charts/sourcebot/README.md
  • charts/sourcebot/tests/basic_test.yaml
  • charts/sourcebot/values.yaml

Included review availability: This review used your included allowance. 2 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.

@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Helm Unit Test Results

20 tests  ±0   20 ✅ ±0   0s ⏱️ ±0s
 3 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 9064488. ± Comparison against base commit e11df97.

@brendan-kellam
brendan-kellam merged commit 60a6828 into main Nov 3, 2026
7 checks passed
@brendan-kellam
brendan-kellam deleted the brendan/sou-2804-update-docs-and-setup-sourcebot-to-use-ghcr-image-urls branch November 3, 2026 21:19
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