Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Machine ID: Support path-based Kubernetes routing #50898

Open
wants to merge 8 commits into
base: timothyb89/kubernetes-path-routing
Choose a base branch
from

Conversation

timothyb89
Copy link
Contributor

@timothyb89 timothyb89 commented Jan 9, 2025

This adds a new kubernetes/v2 service to support path-based routing, which allows clients to access an arbitrary number of Kubernetes clusters using a single issued identity.

It can be used with tbot start kubernetes/v2 and specifying one or more explicit clusters with --kubernetes-cluster-name or a label selector with --kubernetes-cluster-labels.

Part of #40405. Requires #50567.

Changelog: Machine ID's new kubernetes/v2 service supports access to multiple Kubernetes clusters by name or label without needing to issue new identities.

This adds a new `kubernetes/v2` service to support path-based
routing, which allows clients to access an arbitrary number of
Kubernetes clusters using a single issued identity.

It can be used with `tbot start kubernetes/v2` and specifying one or
more explicit clusters with `--kubernetes-cluster-name` or a label
selector with `--kubernetes-cluster-labels`.
@timothyb89 timothyb89 force-pushed the timothyb89/kubernetes-path-routing-tbot branch from eef0afc to a1b84c0 Compare January 17, 2025 03:17
@timothyb89 timothyb89 marked this pull request as ready for review January 17, 2025 03:28
Copy link
Contributor

@strideynet strideynet left a comment

Choose a reason for hiding this comment

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

Looks good - I'll approve here but let's make sure we come to a decision on the roles field before merging.

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.

2 participants