jammsen / docker-palworld-dedicated-server

Docker container to easily provision and manage Palworld Dedicated Server
https://hub.docker.com/r/jammsen/palworld-dedicated-server
MIT License
911 stars 158 forks source link

[Bug Report] Multiple backupmanager entries in supercronic cronlist. #161

Closed SkylerGhostly closed 8 months ago

SkylerGhostly commented 8 months ago

Have you read the Important information text above

Current behavior

Occasionally, the backupmanager.sh script will be called again even if the cronlist file already contains a job. This causes the backup process to be called multiple times.

Desired behavior

Backup operation should only execute once.

Links to screenshots

https://doe.wtf/6OhtWA https://doe.wtf/0m2q_w

To Reproduce

Steps to reproduce the behavior:

  1. Compose & Start the image.
  2. Use docker compose restart
  3. backupmanager.sh adds another backup entry

Software setup

Hardware setup

Additional context

No response

jammsen commented 8 months ago

This bug should be gone when using docker-compose restart, im not touching now the file but writing it new. Feel free to test this and give me feedback on this.

If you like this project, please consider giving this repo and the docker-hub-repo a Star.

SkylerGhostly commented 8 months ago

Thanks for the quick fix. Much appreciated.

jammsen commented 8 months ago

Thanks for the quick fix. Much appreciated.

No problem, please be aware its only on develop right now.

docker pull jammsen/palworld-dedicated-server:develop

I will merge later to prod/master/latest