Skip to content

fix(gateways): bump helm chart version to 0.0.110 (CLI lockstep) - #104

Merged
robbycochran merged 1 commit into
mainfrom
rc-helm-chart-version-bump
Aug 27, 2026
Merged

fix(gateways): bump helm chart version to 0.0.110 (CLI lockstep)#104
robbycochran merged 1 commit into
mainfrom
rc-helm-chart-version-bump

Conversation

@robbycochran

@robbycochran robbycochran commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

Summary

profiles/gateways/helm.yaml pinned the Helm chart at 0.0.85 while the CLI floor (.openshell-version / internal/gateway.MinOpenShellVersion) and profiles/gateways/openshift.yaml are at 0.0.110. The chart's own comment says to keep the version in lockstep with the openshell CLI — a lagging chart ships an older supervisor image and can break sandbox create with "supervisor session not found" during the ssh/tar upload.

Changes

  • profiles/gateways/helm.yaml — chart version 0.0.850.0.110 (now matches openshift.yaml and the CLI pin)
  • profiles/gateways/README.md — example chart version 0.0.590.0.110, with a note about the lockstep expectation

Why this is safe

Chart 0.0.110 is already published and exercised by the OCP flow (openshift.yaml). The kind CI job deploys helm.yaml (test-flow.sh helm), so the kind check on this PR validates the helm flow end-to-end at 0.0.110.

Verification

go build ./..., CGO_ENABLED=0 go test ./..., and make test-suite (33/33) pass locally. Full validation is the PR's kind integration job.

Summary by CodeRabbit

  • Documentation
    • Updated the documented Helm chart version to 0.0.110.
    • Clarified that the Helm chart version must stay synchronized with the OpenShell CLI version.

profiles/gateways/helm.yaml pinned the Helm chart at 0.0.85 while the CLI
floor and openshift.yaml are at 0.0.110. The chart's own comment says to keep
it in lockstep with the openshell CLI, since a lagging chart ships an older
supervisor image and can break sandbox create ("supervisor session not found")
during the ssh/tar upload.

- profiles/gateways/helm.yaml: chart version 0.0.85 -> 0.0.110 (matches
  openshift.yaml and .openshell-version / MinOpenShellVersion)
- profiles/gateways/README.md: bump the example chart version 0.0.59 -> 0.0.110
  and note the lockstep expectation

Chart 0.0.110 is already published and exercised by the OCP flow
(openshift.yaml). The kind CI job deploys helm.yaml, so it validates the helm
flow at 0.0.110.
@coderabbitai

coderabbitai Bot commented Aug 27, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: a39576fa-9829-45dd-adc2-c0933b9a77ea

📥 Commits

Reviewing files that changed from the base of the PR and between 3b1f537 and 4abb7a5.

📒 Files selected for processing (2)
  • profiles/gateways/README.md
  • profiles/gateways/helm.yaml

Included review availability: Your plan provides up to 12 included reviews per hour; 10 remain after this review.


Walkthrough

The gateway profile updates its documented and declared Helm chart version to 0.0.110. The README also states that the chart version must remain synchronized with the OpenShell CLI version.

Changes

Gateway chart version

Layer / File(s) Summary
Align chart version references
profiles/gateways/README.md, profiles/gateways/helm.yaml
The documented and declared Helm chart versions change to 0.0.110. The README specifies lockstep alignment with the OpenShell CLI.

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

Merge Risk: ⚪ Minimal · up to 4abb7

This localized change aligns the Helm chart and documentation with the CLI version; no actionable merge-blocking risk remains beyond normal checks and review.

🚥 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 identifies the Helm chart version update to 0.0.110 and the CLI lockstep requirement. It accurately summarizes the main change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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 functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch rc-helm-chart-version-bump

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

@robbycochran
robbycochran merged commit e2fba55 into main Aug 27, 2026
7 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.

1 participant