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

Don't run "activateEnvironment" command in SSH private terminals #9503

Closed
roblourens opened this issue Jan 10, 2020 · 10 comments
Closed

Don't run "activateEnvironment" command in SSH private terminals #9503

roblourens opened this issue Jan 10, 2020 · 10 comments
Labels
area-environments Features relating to handling interpreter environments area-terminal bug Issue identified by VS Code Team member as probable bug important Issue identified as high-priority

Comments

@roblourens
Copy link
Member

The Remote-SSH extension starts a hidden remote terminal at one point to run some commands. In this issue microsoft/vscode-remote-release#1760 people reported that the terminal was being shown to them, when it should never be shown because it sets a flag to be hidden. We realized that it is shown because this extension runs a command in it, and setting this setting fixes it:

"python.terminal.activateEnvironment": false

It sounds like the Python extension should be able to know that these terminals are not terminals in the user's workspace, and should not touch them: microsoft/vscode-remote-release#1760 (comment)

@roblourens roblourens added triage-needed Needs assignment to the proper sub-team bug Issue identified by VS Code Team member as probable bug labels Jan 10, 2020
@Tyriar
Copy link
Member

Tyriar commented Jan 10, 2020

It looks like the Python extension tries to activate the venv regardless of the type of terminal which should certainly not be happening with the SSH connection ones. microsoft/vscode#63052 was done to help the Python extension know when it should and shouldn't be activating the env so maybe that could be used to avoid activating special terminals like this generally (instead of hardcoding for just the "check remote" terminal)

@karthiknadig karthiknadig added area-environments Features relating to handling interpreter environments needs PR labels Jan 13, 2020
@ghost ghost removed the triage-needed Needs assignment to the proper sub-team label Jan 13, 2020
@karthiknadig karthiknadig added area-terminal important Issue identified as high-priority labels Jan 13, 2020
@lihz6
Copy link

lihz6 commented Jan 17, 2020

This works for me:

    "remote.SSH.enableDynamicForwarding": false

@NellyWhads
Copy link

Would a fix like this hit Insiders first? Wondering if I should switch over so I can get this as soon as it's merged/released.

@roblourens
Copy link
Member Author

@karrtikr this seems to be an issue for a lot of people, could this be prioritized?

@karrtikr
Copy link

We've started working on it. #9584

@roblourens
Copy link
Member Author

@karrtikr We published 1.42 last week so that new API is available now.

@DonJayamanne
Copy link

@karrtikr I've mentioned to @karthiknadig I have a PR for this.
Will push today.

@roblourens
Copy link
Member Author

Thanks! When will the extension next be published?

@karthiknadig
Copy link
Member

@roblourens This will be in the next release which will be (like on) March 9th.

@karthiknadig
Copy link
Member

@roblourens You can test this out now using the insiders version of the extension (VSIX here).

@luabud luabud closed this as completed Feb 27, 2020
@ghost ghost removed the needs PR label Feb 27, 2020
@lock lock bot locked as resolved and limited conversation to collaborators Mar 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-environments Features relating to handling interpreter environments area-terminal bug Issue identified by VS Code Team member as probable bug important Issue identified as high-priority
Projects
None yet
Development

No branches or pull requests

9 participants