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

Restart on crash - or add some redundancy. #33

Open
alexmorley opened this issue Jul 24, 2018 · 7 comments
Open

Restart on crash - or add some redundancy. #33

alexmorley opened this issue Jul 24, 2018 · 7 comments

Comments

@alexmorley
Copy link
Collaborator

alexmorley commented Jul 24, 2018

to prevent e.g. #32

@blahah
Copy link
Collaborator

blahah commented Aug 1, 2018

Yeah, I'm not sure why this is happening but I think it's to do with docker container resource management. There is probably a systematic solution out there somewhere but I haven't discovered it yet. I might ping some experts on twitter this weekend.

@alexmorley
Copy link
Collaborator Author

alexmorley commented Aug 26, 2018

Hmm slightly different problem this time, got a bug report on twitter. But same solution might apply. For now just restarting the VM works OK.

Hi, I'm a CS professor trying to use http://rik.smith-unna.com/command_line_bootcamp/ … for a lab tomorrow, but I can't type in the shell area and don't get a prompt (just a flashing cursor). I've tried this on multiple machines and in multiple browsers on Windows (FF, Chrome, IE).

@alexmorley
Copy link
Collaborator Author

I have added an hourly script to clean up empty containers and do some logging to make it easier to work out what's going on. I suspect it's RAM. In which case I will try add some more to the VM ...

@blahah
Copy link
Collaborator

blahah commented Sep 12, 2018

nice @alexmorley - can you add the script to the repo?

@alexmorley
Copy link
Collaborator Author

@blahah Sorry missed this. yes, but I also have a better solution going forward.

@alexmorley
Copy link
Collaborator Author

Proposal: Add a (very) lightweight orchestrating server between the front-end and docker-browser-server. It only needs a single API entrypoint.

Orchestrator

GET /url?container_id

{
  "createdAt": "2011-09-20T01:02:17.931Z",
  "ip_addr": "8.8.8.8"
}

Docker-Browser-Server

Ideally we would also add the another entry-point to the docker-browser-server API that returns some memory/cpu availability statistics (either overall or per container).
GET /stats?container_id
and
GET /stats

{
  "createdAt": "2011-08-20T02:06:57.931Z",
  "cpu_percentage": "88",
  "mem_percentage": "88",
  "mem_free": "1028"
}

@alexmorley
Copy link
Collaborator Author

alexmorley commented Nov 21, 2018

working on branch orchestra

@blahah blahah mentioned this issue Jun 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants