Skip to content

Default the Sourcebot image to docker.sourcebot.dev#118

Merged
msukkari merged 1 commit into
mainfrom
michael/docker-registry-domain
Jul 10, 2026
Merged

Default the Sourcebot image to docker.sourcebot.dev#118
msukkari merged 1 commit into
mainfrom
michael/docker-registry-domain

Conversation

@msukkari

@msukkari msukkari commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Points the chart's default image.repository at docker.sourcebot.dev/sourcebot-dev/sourcebot instead of ghcr.io/sourcebot-dev/sourcebot, and updates the generated README.md row and the two rendered-image test assertions in tests/basic_test.yaml to match.

The chart-publish workflow (.github/workflows/release.yaml) still pushes the chart to ghcr.io OCI and is intentionally unchanged (that's the chart registry, not the app image).

Verified the new host resolves the image: GET https://docker.sourcebot.dev/v2/sourcebot-dev/sourcebot/manifests/latest returns 200 (307 reroute in front of ghcr.io).

🤖 Generated with Claude Code

Point the chart's default image.repository (and the README/test assertions
derived from it) at docker.sourcebot.dev/sourcebot-dev/sourcebot instead of
ghcr.io. The chart-publish workflow continues to push the chart to ghcr.io OCI.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jul 10, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

The Helm chart’s default Sourcebot image repository changed from GitHub Container Registry to docker.sourcebot.dev. README documentation and tag/digest template assertions were updated to match.

Changes

Sourcebot image registry

Layer / File(s) Summary
Update image registry references
charts/sourcebot/values.yaml, charts/sourcebot/README.md, charts/sourcebot/tests/basic_test.yaml
The chart default, documented repository value, and tag/digest image assertions now reference docker.sourcebot.dev/sourcebot-dev/sourcebot.

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

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
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.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: defaulting the Sourcebot image to docker.sourcebot.dev.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch michael/docker-registry-domain

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.

@github-actions

Copy link
Copy Markdown
Contributor

Helm Unit Test Results

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

Results for commit 3ec7f36. ± Comparison against base commit f712ec1.

@msukkari msukkari requested a review from brendan-kellam July 10, 2026 03:21
@msukkari msukkari merged commit 3d46add into main Jul 10, 2026
6 checks passed
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.

2 participants