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

Add cluster_name to proxy kubernetes config #4354

Merged
merged 2 commits into from
Sep 30, 2020

Conversation

awly
Copy link
Contributor

@awly awly commented Sep 21, 2020

Cluster name from this field plug all clusters from kubeconfig are
stored on the auth server via heartbeats.
This info will later be used to route k8s requests back to proxies.

Updates #3952

@awly awly force-pushed the andrew/k8s-cluster-name-config branch 4 times, most recently from 6f85e20 to bd89569 Compare September 24, 2020 22:55
lib/srv/regular/sshserver.go Outdated Show resolved Hide resolved
lib/srv/regular/sshserver.go Outdated Show resolved Hide resolved
lib/srv/regular/sshserver.go Outdated Show resolved Hide resolved
@awly awly requested a review from webvictim September 28, 2020 23:16
@webvictim
Copy link
Contributor

LGTM (once the conflicts are fixed) - I'll leave it for the others as it's core functionality though.

@awly awly force-pushed the andrew/k8s-cluster-name-config branch from f13afb7 to 88e2838 Compare September 29, 2020 17:01
@awly
Copy link
Contributor Author

awly commented Sep 29, 2020

retest this please

Cluster name from this field plug all clusters from kubeconfig are
stored on the auth server via heartbeats.
This info will later be used to route k8s requests back to proxies.

Updates #3952
@awly awly force-pushed the andrew/k8s-cluster-name-config branch from 88e2838 to 8d65f6b Compare September 29, 2020 21:28
@awly awly force-pushed the andrew/k8s-cluster-name-config branch from 8d65f6b to cbef815 Compare September 29, 2020 22:10
@awly awly merged commit 8306082 into master Sep 30, 2020
@awly awly deleted the andrew/k8s-cluster-name-config branch September 30, 2020 15:56
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.

3 participants