search

alinmear / docker-conanexiles

A docker container to easily provision and manage a conanexiles dedicated server using wine
MIT License
105 stars 75 forks source link

Autoupdate issue #3

Closed kijdam closed 7 years ago

kijdam commented 7 years ago

Hi,

First let me thank you for your work on this docker container. It rocks.

I just noticed that whenever an update is pushed on steam the server is not automatically updated.

To have it updating the server I need to remove lastUpdate file in /conanexiles and restart the container.

Anyone else having this issue ?

Thanks !

alinmear commented 7 years ago

@kijdam i am facing this problem too. I don't know why but the lastUpdate check is not working. Maybe anybody has an idea what we could check here. I tried to check the 'lastUpdate' value from public branch via steamcmd and also the 'buildid', but none of the them worked so far.

When you want to trigger the update you only need to clear the file lastUpdate, because the controllerscript is checking for a change of the lastUpdate (= buildid) and a new buildid, if they are not the same, an update will be triggered.

So only do something like: echo > /var/lib/docker/volumes//_data/lastUpdate or echo > /lastUpdate. After this the Update will be triggered without restarting the container.

kijdam commented 7 years ago

@alinmear Thanks for your answer. Okay so I'm quite "happy" you have the issue too.

I'll take a bit of time to read your script and try to find a way to have the check/update works automatically.

I don't know what you think about it but I would also add a little piece of code to cp -a the Saved dir to Saved.timestamp before the update, because I had some surprises the db being wiped/crashed after an update (including the xxxx_backup.db !)

Sounds good to add this ?

alinmear commented 7 years ago

@kijdam you can find the update logic here conanexiles_controller.sh. If you find a good Solution maybe can you make a pull request for this?

Regarding the backup logic before an Update: Sounds good to me. I will open a new issue for this and will implement it.

alinmear commented 7 years ago

New approach merged with #5. Let's see, hopefuly it works now.

alinmear commented 7 years ago

Seems to work. Thanks alot. Closing this issue ...