feat: restart stopped workspaces on ssh command #11050
Conversation
This is opt out by default. VScode ssh does not have this behavior
| } | ||
| // The workspace needs to be stopped before we can start it. | ||
| // It cannot be in any pending or failed state. | ||
| if workspace.LatestBuild.Status != codersdk.WorkspaceStatusStopped { |
There was a problem hiding this comment.
A case that I would personally like for to work is that I can ssh into a stopping workspace and have it wait until the stop is complete and then either issue the start or wait for a start to complete.
No action necessary for this PR, though. Just writing this down as a food for thought.
There was a problem hiding this comment.
Things could be improved to handle more cases, but that felt a bit overkill atm.
cli/ssh_test.go
Outdated
| err := inv.WithContext(ctx).Run() | ||
| assert.NoError(t, err) | ||
| }) | ||
| pty.ExpectMatch("⧗ Running") |
There was a problem hiding this comment.
I wonder if this has the potential to be racy, i.e. workspace started before we end up streaming logs. 🤔
There was a problem hiding this comment.
That is a good point. I am unsure how to prevent that 🤔.
We really should have a controllable provisioner in our unit tests to trigger this stuff 😢
There was a problem hiding this comment.
Actually, we have coderdtest.AwaitWorkspaceAgents(t, client, workspace.ID) before I run exit which is sufficient to wait for the workspace to be ready.
There was a problem hiding this comment.
Yeah, I think workspace being ready shouldn't be an issue. But I'm wondering if the provisioner log missing could be an issue depending on at what stage of the workspace lifecycle we connect to it.
There was a problem hiding this comment.
Yup, I removed it because I check if the agent is ready right below it. So That agent ready will always work, even if we miss the logs.
What this does
On
coder sshorsshif the target workspace is off, the workspace will be started.Feature can be opt out, so by default this is on. To opt out:
$ coder ssh --disable-autostart local-docker # Or $ coder config-ssh --disable-autostartCloses: #4973
VScode extension?
Vscode uses
coder vscodesshand is unaffected:coder/cli/vscodessh.go
Line 34 in f422f47