exoframejs / exoframe

Exoframe is a self-hosted tool that allows simple one-command deployments using Docker
https://exoframejs.github.io/exoframe/
1.14k stars 57 forks source link

Exoframe server has no restart policy after update #295

Closed Tiim closed 4 years ago

Tiim commented 4 years ago

First of all, thank you so much for exoframe, it's an amazing project. I saved me a lot of fumbling around with some kind of overkill private docker registry for my single server project.

A few weeks ago I upgraded my exoframe from version around 6.1.0 or 6.0.2 (not sure anymore) to the newest version. Now a few days ago i rebooted the server and apparently exoframe and traefik didn't start up again, all the deployed containers did.

A quick inspection of the stopped container showed:

$ docker inspect exoframe-server | grep RestartPolicy -A 3
"RestartPolicy": {
    "Name": "",
    "MaximumRetryCount": 0
},

When I manually started the docker container with the --restart always flag like shown in the documentation then the RestartPolicy looks like this:

$ docker inspect 0fc736562fb8 | grep RestartPolicy -A 3
"RestartPolicy": {
  "Name": "always",
  "MaximumRetryCount": 0
},

I guess the update didn't properly set the restart flag and therefore the server won't spin up after a crash or a reboot etc.

yamalight commented 4 years ago

Yep, that's currently not transferred. Would be a nice improvement. Here's a relevant bit of code: https://github.com/exoframejs/exoframe-server/blob/master/src/routes/update.js#L77 PRs welcome :)

yamalight commented 4 years ago

Should be fixed now. Thanks for PR! ❤️

Tiim commented 4 years ago

For future reference since the restart policy is only retained for updates from version >=6.2.1 the following command will set the restart policy of the exoframe-server container to 'always':

docker update --restart=always $(docker ps --format {{.Names}} | grep exoframe-server)