Skip to content

Add backend.docker.sidecar_image config override - #133

Merged
captainsafia merged 3 commits into
mainfrom
factory/docker-sidecar-image-override
Aug 28, 2026
Merged

Add backend.docker.sidecar_image config override#133
captainsafia merged 3 commits into
mainfrom
factory/docker-sidecar-image-override

Conversation

@warp-agent-staging

@warp-agent-staging warp-agent-staging Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Docker always used the server-provided warp-agent sidecar image. This adds backend.docker.sidecar_image with the same rules as Kubernetes: override only when the assignment includes a sidecar image, and do not rewrite additional sidecars. Sidecar pulls stay anonymous.

What changed

  • Load and validate backend.docker.sidecar_image (optional, no whitespace).
  • Wire it through mergeConfig into DockerBackendConfig.
  • Apply it in prepareTaskParams before sidecar prep, sharing a helper with the Kubernetes override.

Helm is unchanged. The chart always deploys the Kubernetes backend.

Fixes https://linear.app/warpdotdev/issue/REMOTE-3065/oz-agent-worker-add-backenddockersidecar-image-config-override

Validation

go test ./... -count=1, go vet ./..., and gofmt -s -l . all passed locally. Tests cover YAML load, merge into worker config, Docker and Kubernetes override/empty/nil cases, and that additional sidecars are left alone.

Allow operators to set backend.docker.sidecar_image so managed Docker
workers can pull the warp-agent /agent sidecar from a private registry
mirror, matching the existing Kubernetes override.

REMOTE-3065
@warp-agent-staging

Copy link
Copy Markdown
Contributor Author

This PR was generated with Warp.

Comment @warp-factory on this PR to send it follow-up work.

View run View conversation View on Slack

Private-registry mirrors of warp-agent require the same registry auth
as the task image. Additional sidecars stay unauthenticated.

REMOTE-3065
@warp-agent-staging
warp-agent-staging Bot marked this pull request as ready for review August 28, 2026 21:13
Sidecar images stay public and are pulled without registry credentials.
backend.docker.sidecar_image still overrides the /agent image name.

REMOTE-3065
@captainsafia
captainsafia merged commit 63aa7c5 into main Aug 28, 2026
7 checks passed
@warp-local-for-testing-only

Copy link
Copy Markdown

An unexpected error has occurred: managed MCP server 01a01a21-3ec9-71a1-99f2-f13b1773c920 is not active

@captainsafia
captainsafia deleted the factory/docker-sidecar-image-override branch August 28, 2026 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant