Skip to content

Check if server dir exists before attempting to migrate

Iain Powrierequested to merge
bugfix/db-migration-fix into dev

What does this MR do and why?

Crafty v4.4.10 Introduced a migration to move the server stats database files from the server files directory to the crafty database directory. However on fresh deployments of Crafty portable that creates a servers dir after migrations, this was causing Crafty not pass initialization successfully.

This merge request fixes this, By adding a small gate at the start of the migration, that causes it to bail out if the servitor does not exist.

Resolves #568 (closed)

How to set up and validate locally

  • Checkout branch
  • Build crafty portable binary locally
  • Ensure servers directory does not exist
  • Try and launch crafty

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

  • Have you checked this doesn't interfere/conflict/duplicate someone elses work?
  • Have you fully tested your changes?
  • Have you resolved any lint issues?
  • Have you assigned a reviewer?
  • Have you applied correct labels?
Edited by Iain Powrie

Merge request reports