-
Couldn't load subscription status.
- Fork 16
Use wildcard SSH config Host entries #521
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
Conversation
|
cc @code-asher |
|
Looks good so far 😎 |
This simplifies the written SSH config and avoids the need to make an API request for every workspace the filter returns. This can remove minutes from the "Configuring Coder CLI..." step when the user has access to many workspaces (for example, an admin who wants the option of connecting to anyone's workspace on a large deployment). Depends on coder/coder#16088
d8bbd31 to
ebf86fa
Compare
|
I think this is ready for review. The Coder CLI version number placeholder just needs to be filled in when that release happens. |
|
The necessary changes were released in Coder 2.19.0, so this is ready to go now. |
|
|
||
| ### Changed | ||
|
|
||
| - Simplifies the written SSH config and avoids the need to make an API request for every workspace the filter returns. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This simplifies the written SSH config and avoids the need to make an API request for every workspace the filter returns. This can remove minutes from the "Configuring Coder CLI..." step when the user has access to many workspaces (for example, an admin who wants the option of connecting to anyone's workspace on a large deployment).
Depends on coder/coder#16088