mirror of
https://github.com/GameServerManagers/LinuxGSM.git
synced 2026-04-24 21:56:01 +03:00
[GH-ISSUE #4496] [Bug]: Race condition when running backup and monitor at ~ the same time will result in failed backup #2807
Labels
No labels
Atomic
Epic
cannot reproduce
command: backup
command: console
command: debug
command: details
command: fast-dl
command: install
command: mods
command: monitor
command: post-details
command: restart
command: send
command: start
command: stop
command: update
command: update-lgsm
command: validate
command: wipe
distro: AlmaLinux
distro: Arch Linux
distro: CentOS
distro: Debian
distro: Fedora
distro: RedHat
distro: Rocky Linux
distro: Ubuntu
distro: openSUSE
engine: goldsrc
engine: source
game: 7 Days to Die
game: ARMA 3
game: Ark: Survival Evolved
game: Assetto Corsa
game: Avorion
game: BATTALION: Legacy
game: Barotrauma
game: Battalion 1944
game: Battlefield 1942
game: Black Mesa: Deathmatch
game: Blade Symphony
game: Call of Duty 2
game: Call of Duty 4
game: Call of Duty: United Offensive
game: Counter-Strike 1.6
game: Counter-Strike 2
game: Counter-Strike: Global Offensive
game: Counter-Strike: Source
game: Day of Infamy
game: Dayz
game: Death Match Classic
game: Don't Starve Together
game: ET: Legacy
game: Eco
game: Factorio
game: Factorio
game: Garry's Mod
game: Half-Life
game: Hurtword
game: Insurgecy
game: Insurgecy
game: Insurgency: Sandstorm
game: Just Cause 3
game: Killing Floor
game: Killing Floor 2
game: Left 4 Dead 2
game: Minecraft
game: Minecraft Bedrock
game: Mordhau
game: Multi Theft Auto
game: Mumble
game: Natural Selection 2
game: No More Room in Hell
game: Pavlov VR
game: Post Scriptum
game: Project Zomboid
game: Quake 3
game: QuakeWorld
game: Red Orchestra: Ostfront 41-45
game: Return to Castle Wolfenstein
game: Rising World
game: Rust
game: San Andreas Multiplayer
game: Satisfactory
game: Soldat
game: Soldier of Fortune 2
game: Squad
game: Squad 44
game: Starbound
game: Stationeers
game: Sven Co-op
game: Team Fortress 2
game: Teamspeak 3
game: Teeworlds
game: Terraria
game: The Front
game: Unreal Tournament 2004
game: Unreal Tournament 3
game: Unreal Tournament 99
game: Unturned
game: Valheim
game: Wurm Unlimited
game: Zombie Master Reborn
game: label missing
good first issue
help wanted
info: alerts
info: dependency
info: docker
info: docs
info: email
info: query
info: steamcmd
info: systemd
info: tmux
info: website
info: website
needs more info
outcome: duplicate
outcome: issue resolved
outcome: issue resolved
outcome: issue unresolved
outcome: pr accepted
outcome: pr rejected
outcome: unconfirmed
outcome: wontfix
outcome: wrong forum
potential-duplicate
priority
pull-request
type: bug
type: feature
type: feature
type: feature request
type: game server request
type: refactor
waiting response
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/LinuxGSM#2807
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @MicLieg on GitHub (Feb 12, 2024).
Original GitHub issue: https://github.com/GameServerManagers/LinuxGSM/issues/4496
User story
As a server admin, i want the monitor command to not interrupt currently running backups.
Game
All ?!
Linux distro
Ubuntu 22.04
Command
command: backup
Further information
I noticed a bug when my Minecraft server got mistakenly restarted during a backup because of an automated cronjob running the monitor command (while the server was creating a backup) but essentially at the same time. This occured becasuse i have the monitor command set to run every 5 minutes and the backup command to run every day at 4:00 am.
After looking into what's happening i think i found the issue:
If you start a backup and immediatley run the monitor command after the server has already been stopped by the backup command but before the
backup.lockfile was created, the monitor command will pass thebackup.lockfilecheck and run regularly (start querying the gameserver) and restart the server because thebackup.lockfile didn't exist. The monitor command will send out the appropriate alertUnable to query mcserver. Game server has been restarted.and restart the server. This will possibly result in the backup not finishing becasue of atarerror:Backing up mcserver: Backup (9,7G) mcserver-2024-02-12-015407.tar.gz, in progress...tar: ././serverfiles/world: file changed as we read it FAILThis will also lead to thebackup.lockfile not being deleted.Note that this will not be noticed if the backup finishes within 60 seconds of the monitor command being called, in which case the monitor command will simply assume that the server is online and thus won't restart the server.
If creating the backup takes a little longer the monitor command will continue querying and spew out errors depending how far the backup command got:
Another error that can occur depending on timing:
Possible Solutions:
backup.lockfile earlier in the backup flow.gameserver-monitor.lockfile that forces commands that affect uptime to wait until the lock file is removed.Relevant log output
Steps to reproduce
stoponbackup./gameserver backup./gameserver monitorbackup.lockfile, you were too slow and the backup command has gotten too far alreadyKeep in mind to manually delete the
backup.lockfile in between trying to reproduce this bug becasue it won't be deleted sometimes.@MicLieg commented on GitHub (Feb 24, 2024):
Possibly related to #3972
@MicLieg commented on GitHub (Feb 25, 2024):
link #2106
@dgibbs64 commented on GitHub (Jun 27, 2025):
I have moved fn_backup_create_lockfile higher up so the lock file will be created before the server is stopped. Hopefully this should deal with the edge case
d472d66221