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

[v16] Allow a custom SSH dial timeout to be set cluster wide #45650

Merged
merged 1 commit into from
Aug 22, 2024

Conversation

rosstimothy
Copy link
Contributor

Backport #45507 to branch/v16

Changelog: Permit setting a cluster wide SSH connection dial timeout

@rosstimothy
Copy link
Contributor Author

Friendly ping @tigrato

@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from zmb3 August 22, 2024 12:43
@rosstimothy rosstimothy added this pull request to the merge queue Aug 22, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Aug 22, 2024
@rosstimothy rosstimothy added this pull request to the merge queue Aug 22, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 22, 2024
A new SSHDialTimeout field is added to the ClusterNetworkingConfiguration
to allow cluster admins to override the default dial timeout of 30s.
The dial timeout is made available in /webapi/ping, stored in the tsh
profile, and cached on disk like other similar cluster settings so
that tsh ssh can consume the relevant information without requiring
an additional round trip to the cluster. At the moment, there is no
support for an unlimited dial timeout, as that would cause
complexity with backwards compatibility.
@rosstimothy rosstimothy added this pull request to the merge queue Aug 22, 2024
Merged via the queue into branch/v16 with commit 12c8dac Aug 22, 2024
38 of 39 checks passed
@rosstimothy rosstimothy deleted the tross/backport-45507/v16 branch August 22, 2024 15:10
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.

3 participants