helm: expose coding_cli_sidecars in chart (REMOTE-2107)#99
Draft
seemeroland wants to merge 1 commit into
Draft
Conversation
PR #92 added the coding_cli_sidecars worker config (for pointing a harness at a custom/internal Claude Code binary wrapper image) and documented it, but the Helm chart was never updated to expose it. Self-hosted k8s deployments that install via the chart therefore could not configure it declaratively and had to fall back to setup-command workarounds. Add a kubernetesBackend.codingCliSidecars value that renders into the worker config's backend.kubernetes.coding_cli_sidecars map, following the existing extraLabels/extraAnnotations pattern. Defaults to {} so it is omitted when unset. Bump chart version to 0.2.0 and add a README note. Co-Authored-By: Oz <oz-agent@warp.dev>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Exposes the
coding_cli_sidecarsworker config through the Helm chart so self-hosted Kubernetes deployments can point a harness (e.g.claude) at a custom or internal Claude Code binary wrapper image declaratively.Background
PR #92 added the
coding_cli_sidecarsconfig to the worker (internal/config,main.go) and documented it in the README, and it shipped in releasev2026-07-09-15-18-23. However, the Helm chart (charts/oz-agent-worker) was never updated to render this field. Deployments that install via the chart could not set it declaratively and had to fall back to setup-command workarounds (as IMC did with theirimc-claudewrapper).Changes
values.yaml: addkubernetesBackend.codingCliSidecars(defaults to{}) with an explanatory comment.templates/configmap.yaml: rendercoding_cli_sidecarsinto the workerconfig.yamlunderbackend.kubernetes, following the existingextraLabels/extraAnnotations{{- with ... }}pattern. Omitted entirely when unset.Chart.yaml: bump chart version0.1.0→0.2.0.README.md: note the new chart value in the Helm section.Example
renders to:
Validation
helm lint ./charts/oz-agent-workerpasses (matches CI).helm templaterenderscoding_cli_sidecarscorrectly nested underbackend.kubernetes.Resolves REMOTE-2107.
Conversation: https://staging.warp.dev/conversation/4694de2e-a0d9-4b1e-ac35-c6cdff087f87
Run: https://oz.staging.warp.dev/runs/019f47d6-d577-7792-bc76-f13f69015c8a
This PR was generated with Oz.