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

[ws-daemon] fails mounting proc #4784

Closed
svenefftinge opened this issue Jul 13, 2021 · 5 comments
Closed

[ws-daemon] fails mounting proc #4784

svenefftinge opened this issue Jul 13, 2021 · 5 comments
Labels
component: ws-daemon type: bug Something isn't working

Comments

@svenefftinge
Copy link
Member

There are regular CRITICAL errors logged from the nsinsider process. Followed by a "cannot mount /proc" error logging.

I am not sure how this is experienced by users, but I guess it means a workspace start failure.

@svenefftinge svenefftinge added the priority: highest (user impact) Directly user impacting label Jul 13, 2021
@csweichel
Copy link
Contributor

Unfortunately the error message isn't very helpful. It's well possible that the mount-point where the user is trying to mount proc no longer exists. It's also just as possible that this is an actual problem.

First step is to get better logging in place.

@csweichel
Copy link
Contributor

csweichel commented Jul 13, 2021

I've raised a PR to improve the logs in this corner of the system. We'll deploy today 13:00 CEST and bring this into prod.
We'll revisit this issue once the logging is in place.

@csweichel csweichel removed their assignment Jul 13, 2021
@csweichel csweichel added component: ws-daemon type: bug Something isn't working labels Jul 13, 2021
@csweichel
Copy link
Contributor

looking at the non-headless cases it seems that the proc mount failures are due to Docker container starts going wrong.
I'd propose we tone down this particular message (i.e. the mount point disappears before we can mount/does not exist).
If this affected workspaces, we'd see workspaces fail.

That said, it would be interesting to reproduce the issue and find out how it comes to be.

For now, I'll reduce the priority because this does not seem to directly impact users.

@csweichel csweichel removed the priority: highest (user impact) Directly user impacting label Jul 15, 2021
@csweichel
Copy link
Contributor

/schedule

@csweichel
Copy link
Contributor

Closing for now - will reopen if the issue persists

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: ws-daemon type: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants