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

Configured default shell not used when reconnecting to Codespace #175107

Closed
lumaxis opened this issue Feb 22, 2023 · 6 comments · Fixed by #175844 or #181832
Closed

Configured default shell not used when reconnecting to Codespace #175107

lumaxis opened this issue Feb 22, 2023 · 6 comments · Fixed by #175844 or #181832
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders remote Remote system operations issues terminal-profiles verified Verification succeeded
Milestone

Comments

@lumaxis
Copy link
Contributor

lumaxis commented Feb 22, 2023

Does this issue occur when all extensions are disabled?: Yes

Version: 1.75.1 (Universal)
Commit: 441438a
Date: 2023-02-08T21:34:59.000Z
Electron: 19.1.9
Chromium: 102.0.5005.194
Node.js: 16.14.2
V8: 10.2.154.23-electron.0
OS: Darwin arm64 22.3.0
Sandboxed: Yes

When reconnecting to an existing Codespace, my "terminal.integrated.defaultProfile.linux" configuration is ignored. Instead of zsh, the terminal always starts bash.

The same/a similar issue used to exist in early 2022 but was presumably fixed with #154290, however this appears to have regressed over the last weeks/months.

Steps to Reproduce:

  1. Configure "terminal.integrated.defaultProfile.linux": "zsh"
  2. Create a Codespace & stop it
  3. Reconnect to the Codespace
  4. See that the terminal opens using bash
@meganrogge
Copy link
Contributor

can reproduce

Screenshot 2023-02-22 at 2 35 56 PM

@meganrogge meganrogge added bug Issue identified by VS Code Team member as probable bug terminal-profiles labels Feb 22, 2023
@meganrogge meganrogge added this to the Backlog milestone Feb 22, 2023
jeanp413 added a commit to jeanp413/vscode that referenced this issue Mar 2, 2023
@jeanp413
Copy link
Contributor

jeanp413 commented Mar 2, 2023

PR fixing it #175844

@Tyriar
Copy link
Member

Tyriar commented Mar 29, 2023

Repro in OSS:

  1. Set "terminal.integrated.defaultProfile.windows": "Command Prompt" (or whatever the platform is)
  2. Open a test resolver window
  3. Open the panel
  4. Reload, 🐛 it's not the shell defined in step 1

@Tyriar Tyriar modified the milestones: Backlog, April 2023 Mar 29, 2023
trbot86 pushed a commit to trbot86/vscode that referenced this issue Mar 29, 2023
@lumaxis
Copy link
Contributor Author

lumaxis commented Mar 31, 2023

Thanks everyone! So glad this is fixed now ❤️

@alexr00 alexr00 added verified Verification succeeded verification-found Issue verification failed and removed verified Verification succeeded labels Apr 27, 2023
@alexr00
Copy link
Member

alexr00 commented Apr 27, 2023

I can still repro this issue:

  1. Set "terminal.integrated.defaultProfile.linux": "zsh"
  2. From within VS Code desktop (on Windows, if it matters), create a Codespace
  3. Open a terminal in the new Codespace and run ps. See that zsh is one of the processes ✅
  4. Command "Codespaces: Stop Current Codespace"
  5. Open the Remote Explorer and connect to the Codespace from steop 2
  6. On a terminal in the Codespace and run ps. No zsh processes are listed.

@alexr00 alexr00 reopened this Apr 27, 2023
@jeanp413
Copy link
Contributor

Seems it's only for the above case only now, using codespaces extension which I cannot test from sources 😅, before I could repro easily with my SSH extension too which is working properly now

@meganrogge meganrogge removed the verification-found Issue verification failed label Apr 27, 2023
@meganrogge meganrogge modified the milestones: April 2023, May 2023, Backlog Apr 27, 2023
jeanp413 added a commit to jeanp413/vscode that referenced this issue May 8, 2023
jeanp413 added a commit to jeanp413/vscode that referenced this issue May 8, 2023
@Tyriar Tyriar added the remote Remote system operations issues label May 24, 2023
@Tyriar Tyriar modified the milestones: Backlog, August 2023 Aug 4, 2023
@vscodenpa vscodenpa added unreleased Patch has not yet been released in VS Code Insiders insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Aug 4, 2023
@alexr00 alexr00 added the verified Verification succeeded label Aug 30, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Sep 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders remote Remote system operations issues terminal-profiles verified Verification succeeded
Projects
None yet
6 participants