Skip to content

feat(kubernetes): add combined topology config surface#2074

Merged
TaylorMutch merged 2 commits into
mainfrom
feat/kubernetes-topology-base-v2
Jun 30, 2026
Merged

feat(kubernetes): add combined topology config surface#2074
TaylorMutch merged 2 commits into
mainfrom
feat/kubernetes-topology-base-v2

Conversation

@TaylorMutch

@TaylorMutch TaylorMutch commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

Summary

Add the Kubernetes supervisor topology configuration surface with the existing combined topology as the only supported value. This keeps current sandbox behavior unchanged while creating the config, Helm, and documentation shape needed for future topology work.

Related Issue

References #1973.
References #1827.
References #981.
References #899.
References #1305.

Related PR: #2016.

Changes

  • Add SupervisorTopology to the Kubernetes driver config and CLI/env parsing.
  • Render supervisor_topology into Helm-managed gateway.toml with chart tests.
  • Add supervisor.topology to Helm values and generated chart README.
  • Add Kubernetes topology documentation for the supported combined mode and note that additional topologies are still being designed.
  • Update gateway config and Kubernetes setup docs for the new field.

Testing

  • mise run pre-commit passes
  • cargo check -p openshell-driver-kubernetes passes
  • cargo test -p openshell-driver-kubernetes --lib passes
  • mise run helm:test passes
  • E2E tests added/updated (not applicable; this PR preserves the existing combined topology behavior)

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)

@TaylorMutch TaylorMutch requested a review from mrunalp as a code owner June 30, 2026 21:22
@TaylorMutch TaylorMutch added the test:e2e Requires end-to-end coverage label Jun 30, 2026
@github-actions

Copy link
Copy Markdown

Label test:e2e applied for e7e3504. Open Branch E2E Checks, find the run for commit e7e3504, and click Re-run all jobs to execute with the label set. The run will execute the standard E2E suite after building the required gateway and supervisor images once. The matching required CI gate status on this PR will flip green automatically once the run finishes.

@github-actions

Copy link
Copy Markdown

Signed-off-by: Taylor Mutch <taylormutch@gmail.com>
@TaylorMutch TaylorMutch force-pushed the feat/kubernetes-topology-base-v2 branch from e7e3504 to 1f1aef5 Compare June 30, 2026 21:24
Comment thread crates/openshell-driver-kubernetes/src/config.rs Outdated
Comment thread crates/openshell-driver-kubernetes/src/config.rs Outdated
Comment thread crates/openshell-driver-kubernetes/src/config.rs Outdated
Comment thread docs/kubernetes/setup.mdx Outdated
Signed-off-by: Taylor Mutch <taylormutch@gmail.com>
@TaylorMutch TaylorMutch requested a review from krishicks June 30, 2026 21:37
@TaylorMutch TaylorMutch merged commit 914da33 into main Jun 30, 2026
47 checks passed
@TaylorMutch TaylorMutch deleted the feat/kubernetes-topology-base-v2 branch June 30, 2026 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:e2e Requires end-to-end coverage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants