Skip to content

feat: add CODER_WORKSPACE_HOSTNAME_SUFFIX #17268

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

Merged
merged 1 commit into from
Apr 7, 2025

Conversation

spikecurtis
Copy link
Contributor

@spikecurtis spikecurtis commented Apr 7, 2025

Adds deployment option CODER_WORKSPACE_HOSTNAME_SUFFIX. This will eventually replace CODER_SSH_HOSTNAME_PREFIX, but we will do this slowly and support both for coder ssh for some time.

Note that the name is changed to "workspace" hostname, since this suffix will also be used for Coder Connect on Coder Desktop, which is not limited to SSH.

Copy link
Contributor Author

spikecurtis commented Apr 7, 2025

Copy link
Member

@mafredri mafredri left a comment

Choose a reason for hiding this comment

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

Nice, LGTM!

Just checking, we're not deprecating prefix yet?

@@ -2549,6 +2550,17 @@ func (c *DeploymentValues) Options() serpent.OptionSet {
Hidden: false,
Default: "coder.",
},
{
Name: "Workspace Hostname Suffix",
Description: "Workspace hostnames use this suffix in SSH config and Coder Connect on Coder Desktop. By default it is coder, resulting in names like myworkspace.coder.",
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Description: "Workspace hostnames use this suffix in SSH config and Coder Connect on Coder Desktop. By default it is coder, resulting in names like myworkspace.coder.",
Description: "Workspace hostnames use this suffix in SSH config and Coder Connect on Coder Desktop. By default it is \"coder\", resulting in names like myworkspace.coder.",

Suggestion: To differentiate the value from the text.

@spikecurtis spikecurtis merged commit 87d9ff0 into main Apr 7, 2025
44 of 46 checks passed
@spikecurtis spikecurtis deleted the spike/host-suffix/deployment-option branch April 7, 2025 07:35
@github-actions github-actions bot locked and limited conversation to collaborators Apr 7, 2025
Copy link
Contributor Author

No, not for a while. I'm marking some internal fields deprecated, but the external behavior isn't deprecated yet.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants