Skip to content

chore(localmode): sync compose template with volcano-hosting - #156

Open
tkkhq wants to merge 1 commit into
mainfrom
tkkhq/vol-788-sync-compose-asset
Open

chore(localmode): sync compose template with volcano-hosting#156
tkkhq wants to merge 1 commit into
mainfrom
tkkhq/vol-788-sync-compose-asset

Conversation

@tkkhq

@tkkhq tkkhq commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Tracking

Summary

Regenerates the embedded local-mode Compose asset from volcano-hosting, so volcano start produces a server that can launch a v2 agent worker.

  • adds the /var/run/docker.sock mount on the server service, which the local-mode server needs to start the agent worker as a sibling container
  • generated by make volcano-cli-localmode-assets-generate in volcano-hosting; the file is not hand-edited

volcano-hosting owns infrastructure/local-mode/docker-compose.template.yml. This repository embeds a generated copy so the CLI can start the stack without a hosting checkout. Hosting PR #807 changed the template, which left this copy stale: a developer running volcano start got a server that silently could not launch a worker.

The mount is local-mode only. It grants the server container the ability to start containers, which is the local equivalent of the ECS RunTask permission the Fargate launcher holds in cloud.

Merge order

Land volcano-hosting PR #807 first, so the source template is final.

Verification

  • make volcano-cli-localmode-assets-check in volcano-hosting, pointed at this branch -> volcano-cli local-mode assets check passed
  • go build ./... -> pass
  • go vet ./... -> pass
  • go test ./... -count=1 -> all pass
  • git diff --check -> clean
  • the diff is limited to the server service volumes block; no other service, port, or volume changed

@tkkhq
tkkhq requested a review from a team as a code owner August 9, 2026 21:22
Copilot AI balanced review requested due to automatic review settings August 9, 2026 21:22

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Syncs the generated local-mode Compose asset with volcano-hosting to enable v2 agent worker containers.

Changes:

  • Mounts the host Docker socket into the server.
  • Retains the existing local-storage volume.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

# WARNING: This grants the server container permission to run any container
# with the server's privileges. Only safe in local development on your machine.
volumes:
- /var/run/docker.sock:/var/run/docker.sock
Comment on lines +76 to +78
# Mount docker socket to allow agent worker launch (local mode only).
# WARNING: This grants the server container permission to run any container
# with the server's privileges. Only safe in local development on your machine.
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