Skip to content

Conversation

@FeneralGeldmarschall
Copy link
Contributor

Description

Added the -h flag to the tar backup command so it follows symlinks, which is relevant for docker containers as otherwise only the symlink to the serverfiles is backed up.

Fixes #4746

Type of change

  • Bug fix (a change which fixes an issue).
  • New feature (a change which adds functionality).
  • New Server (new server added).
  • Refactor (restructures existing code).
  • Comment update (typo, spelling, explanation, examples, etc).

Checklist

PR will not be merged until all steps are complete.

  • This pull request links to an issue.
  • This pull request uses the develop branch as its base.
  • This pull request subject follows the Conventional Commits standard.
  • This code follows the style guidelines of this project.
  • I have performed a self-review of my code.
  • I have checked that this code is commented where required.
  • I have provided a detailed enough description of this PR.
  • I have checked if documentation needs updating.

Documentation

If documentation does need updating either update it by creating a PR (preferred) or request a documentation update.

Thank you for your Pull Request!

@FeneralGeldmarschall FeneralGeldmarschall changed the base branch from master to develop April 26, 2025 19:15
@dgibbs64 dgibbs64 added this to the v25.2.0 milestone Jun 15, 2025
@dgibbs64 dgibbs64 merged commit 3cd3009 into GameServerManagers:develop Jun 15, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Docker Backups stores symlinks insead server files. Also backups folder is outside of persisted volume by default

2 participants