-
Notifications
You must be signed in to change notification settings - Fork 637
bug: ISSUE-7452: Fix EC2 Workspace Connect for Any OS #7455
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
base: master
Are you sure you want to change the base?
Conversation
|
Might need to change a bit here since now we have an input/prompt when the OS does not match Amazon Linux or Ubuntu. |
That's definitely an important part of the logic.
It looks like we always tell vscode to choose / as the target dir: aws-toolkit-vscode/packages/core/src/awsService/ec2/model.ts Lines 215 to 218 in d8d08e2
I think we depend on some later step to detect $HOME, or something like that. |
Co-authored-by: Justin M. Keyes <jmkeyes@amazon.com>
We might want to double check that logic, as we aren't required to specify any path when connecting vscode to an instance.
I saw the errors reported from the unit test run. I will work on them and push a fix soon.
In the meantime, if anyone wants to explore the option to connect ECS to vscode. |
Problem
Described at #7452
Solution
sed
call to a generic style that works in any platform.Notes
feature/x
branches will not be squash-merged at release time.