ADD periodic restart

[EdTheLoon]

I amended my docker-compose.yml file to enable restart of server at 6am every day.

  windrose:
    container_name: windrose_server
    image: indifferentbroccoli/windrose-server-docker
    platform: linux/amd64
    restart: unless-stopped
    stop_grace_period: 30s
    env_file:
      - .env
    volumes:
      - ./server-files:/home/steam/server-files
    ports:
      - '7777:7777/tcp'
      - '7777:7777/udp'
      - '8780:8780/tcp'

  restarter:
    container_name: windrose_restarter
    image: docker:cli
    restart: unless-stopped
    volumes: ["/var/run/docker.sock:/var/run/docker.sock"]
    entrypoint: ["/bin/sh", "-c"]
    command:
      - |
        while true; do
          if [ "$$(date +'%H:%M')" = '06:00' ]; then
            docker restart windrose_server
          fi
          sleep 60
        done

[RotasOpera]

Why not just a cronjob or systemd timer?

[EdTheLoon]

Because the whole point of docker is to have everything in one self contained system. This allows people with less experience to have a working system without having to mess with things they don't know about.

[RotasOpera]

Completely agree that it should be in contained in a file.
But I don't consider it best practice to mount the docker.sock from a security perspective. I might be a bit nitpicky here, but especially when people with less experience mess with this stuff, things tend to go wrong.
Maybe we can instead issue a shutdown command in a restart container via RCON? Or provide a restart env variable and directly restart inside the container? If I find the time I'll look at the dockerfile later

[EdTheLoon]

Completely agree that it should be in contained in a file. But I don't consider it best practice to mount the docker.sock from a security perspective. I might be a bit nitpicky here, but especially when people with less experience mess with this stuff, things tend to go wrong. Maybe we can instead issue a shutdown command in a restart container via RCON? Or provide a restart env variable and directly restart inside the container? If I find the time I'll look at the dockerfile later

I understand your concern. I've just realised that I may have misunderstood earlier. A cron job could be added within the container itself rather than using the host system's crontab, allowing it to still be fully containerised and would be part of the dockerfile set up.