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

Backend fails to start possibly because of a timeout #2710

Closed
mathansen opened this Issue Aug 30, 2018 · 5 comments

Comments

Projects
None yet
3 participants
@mathansen

mathansen commented Aug 30, 2018

The server worker startup fails for me from time to time with the following output:

time yarn start --hostname=0.0.0.0
yarn run v1.9.4
$ theia start --hostname=0.0.0.0
Starting server worker...
Server worker has been started. [ID: 1 | PID: 1558]
Server worker failed to start.
Server worker is ready to accept messages. [ID: 1 | PID: 1558]
Server worker has been disconnected. [ID: 1 | PID: 1558]
^CCommand terminated by signal 2
real 0m 16.72s
user 0m 3.26s
sys 0m 0.21s

I terminated the command immediately after the "...has been disconnected" message. The host machine is a Raspberry Pi. A timeout seems a probable cause.

@akosyakov

This comment has been minimized.

Show comment
Hide comment
@akosyakov

akosyakov Aug 30, 2018

Member

We should allow to increase timeout or check whether we need it at all on the first startup in the case if the DI configuration is broken.

Member

akosyakov commented Aug 30, 2018

We should allow to increase timeout or check whether we need it at all on the first startup in the case if the DI configuration is broken.

@akosyakov akosyakov self-assigned this Aug 30, 2018

@JonasHelming

This comment has been minimized.

Show comment
Hide comment
@JonasHelming

JonasHelming Aug 30, 2018

Workaround until this is fixed: By appending --no-cluster to the yarn start command, everything will run in the same process rather than separate worker processes.

JonasHelming commented Aug 30, 2018

Workaround until this is fixed: By appending --no-cluster to the yarn start command, everything will run in the same process rather than separate worker processes.

@akosyakov

This comment has been minimized.

Show comment
Hide comment
@akosyakov

akosyakov Aug 30, 2018

Member

@JonasHelming just be aware that it can potentially run heavy file processing in the same process and block the UI, like locating git repositories and watching file system, if you git repo and workspace are small it should be fine

Member

akosyakov commented Aug 30, 2018

@JonasHelming just be aware that it can potentially run heavy file processing in the same process and block the UI, like locating git repositories and watching file system, if you git repo and workspace are small it should be fine

@JonasHelming

This comment has been minimized.

Show comment
Hide comment
@JonasHelming

JonasHelming Aug 30, 2018

@akosyakov : Thanks for the hint. In our current use case, we deal with a very small project, so we are fine for now...

JonasHelming commented Aug 30, 2018

@akosyakov : Thanks for the hint. In our current use case, we deal with a very small project, so we are fine for now...

@mathansen

This comment has been minimized.

Show comment
Hide comment
@mathansen

mathansen Sep 6, 2018

@akosyakov is there an update to when you guys will find the time to look into this? Alternatively, it would be great to know where this timeout is currently specified in Theia, so we can just increase it for the moment.

mathansen commented Sep 6, 2018

@akosyakov is there an update to when you guys will find the time to look into this? Alternatively, it would be great to know where this timeout is currently specified in Theia, so we can just increase it for the moment.

akosyakov added a commit that referenced this issue Sep 7, 2018

fix #2710: add startup-timeout option
Signed-off-by: Anton Kosyakov <anton.kosyakov@typefox.io>

akosyakov added a commit that referenced this issue Sep 7, 2018

fix #2710: add startup-timeout option
Signed-off-by: Anton Kosyakov <anton.kosyakov@typefox.io>

akosyakov added a commit that referenced this issue Sep 7, 2018

fix #2710: add startup-timeout option
Signed-off-by: Anton Kosyakov <anton.kosyakov@typefox.io>

@akosyakov akosyakov closed this in dbb7745 Sep 11, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment