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

IDE doesn't execute a command after start of workspace #6324

Closed
garagatyi opened this issue Sep 18, 2017 · 9 comments
Closed

IDE doesn't execute a command after start of workspace #6324

garagatyi opened this issue Sep 18, 2017 · 9 comments
Labels
kind/bug Outline of a bug - must adhere to the bug report template. target/branch Indicates that a PR will be merged into a branch other than master.

Comments

@garagatyi
Copy link

Related to SPI branch only.
When I start workspace either in IDE or UD client doesn't connect to exec server.
So when I try to execute any command IDE doesn't show the result of the command since it doesn't send command execution request to exec server.
Also, IDE should not activate RUN panel when there is no connection with exec server.
BTW reloading the page fixes the issue.

Reproduction Steps:

OS and version:

Diagnostics:

@garagatyi garagatyi added kind/bug Outline of a bug - must adhere to the bug report template. target/branch Indicates that a PR will be merged into a branch other than master. labels Sep 18, 2017
@azatsarynnyy
Copy link
Member

Executing commands in IDE fixed by c04314d

@garagatyi
Copy link
Author

@slemeur How do you think should we disable run commands panel until the server is available? Isn't it the purpose of new start flow?

@ghost
Copy link

ghost commented Sep 20, 2017

@garagatyi in my case exec agent is up and OK since F5 fixes it.

@garagatyi
Copy link
Author

It is exactly what I have experienced.

@ghost
Copy link

ghost commented Sep 20, 2017

But exec agent is the first to start, even before ws agent. It is there and accepting requests (I used rest console to check)

@garagatyi
Copy link
Author

yes, I completely agree with you

@garagatyi
Copy link
Author

@eivantsov can you check whether it is fixed or not?

@ghost
Copy link

ghost commented Sep 21, 2017

@garagatyi @azatsarynnyy cannot reproduce it with the latest SPI

@slemeur
Copy link
Contributor

slemeur commented Sep 21, 2017

@garagatyi you are right, this is also something we need to handle as part of the work on the new workspace loading sequence.

BTW - We should not start by the exec server.

See #5231

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template. target/branch Indicates that a PR will be merged into a branch other than master.
Projects
None yet
Development

No branches or pull requests

3 participants