Repository navigation
Allow Docker sandboxes to set container labels #4562
Description
Please read this first
- Have you read the docs? Yes. I reviewed the sandbox client documentation and the current Docker sandbox implementation.
- Have you searched for related issues? Yes. I searched open and closed issues and pull requests for Docker sandbox labels, container labels,
DockerSandboxClientOptionslabels, and Docker container metadata. I did not find an existing request covering this capability.
Describe the feature
I would like DockerSandboxClientOptions to support optional Docker container labels.
Docker labels are standard container metadata for identifying and managing containers with existing Docker tooling. The Docker Python SDK supports labels when creating a container, but DockerSandboxClient does not currently expose this option.
Proposed API:
DockerSandboxClientOptions(
image="python:3.14-slim",
labels={"com.example.owner": "worker-123"},
)The option would be additive and optional. When omitted, Docker container creation would remain unchanged.
Labels would also be preserved in DockerSandboxSessionState so replacement containers created during resume() retain the configured metadata.
Activity
seanxuu commentedon Aug 21, 2026
I’d like to work on this issue. I plan to add an optional
labelsmapping toDockerSandboxClientOptions, persist it inDockerSandboxSessionState, and forward it only when creating new or replacement containers. I’ll add focused coverage for Docker SDK keyword forwarding, options/state round-trips, and resume replacement behavior while keeping unlabeled containers unchanged.koadegno commentedon Aug 21, 2026
Thanks for offering to work on this! I’ve already been working on an implementation for this issue and have the changes and test coverage ready on my fork. I’m doing the final review now and plan to open the PR shortly, so I’d like to avoid duplicating your work. I’ll link the PR here once it’s opened.seanxuu commentedon Aug 21, 2026
Could you help me to check my code? @koadegno
[-][Sandbox] Allow Docker sandboxes to set container labels[/-][+]Allow Docker sandboxes to set container labels[/+]