github/dokku
0
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Merge pull request #8664 from dokku/8656-convert-filesystem-migration-markers-to-plugin-properties

Browse Source 
Convert filesystem migration markers to plugin properties
This commit is contained in:
Jose Diaz-Gonzalez
2026-05-24 15:37:02 -04:00
committed by GitHub
parent 82fa8de766 5286612618
commit 0b567ff1b9
9 changed files with 373 additions and 68 deletions
Download Patch File Download Diff File Expand all files Collapse all files

48
tests/unit/storage.bats
View File

@@ -420,6 +420,13 @@ teardown() {
assert_success
assert_output "1"
# Property marker is set after a successful migration with mounts.
run /bin/bash -c "sudo test -f /var/lib/dokku/config/storage/$TEST_APP/legacy-mounts-migrated"
assert_success
run /bin/bash -c "sudo cat /var/lib/dokku/config/storage/$TEST_APP/legacy-mounts-migrated"
assert_success
assert_output "true"
# Cleanup: unmount + destroy the synthesized legacy entry. We need
# the entry name from list-entries since legacy-<hash> is content-derived.
legacy_name=$(dokku storage:list-entries --format json | jq -r '.[] | select(.name | startswith("legacy-")) | .name' | head -1)
@@ -438,3 +445,44 @@ teardown() {
assert_success
assert_output_contains "deprecated"
}
@test "(storage:migrate) leaves legacy-mounts-migrated unset for apps with no legacy -v lines" {
# Fresh app with no -v lines: storage:migrate must NOT write the
# legacy-mounts-migrated property. Distinguishes "never had legacy
# state" from "had legacy state, drained".
sudo rm -f "/var/lib/dokku/config/storage/$TEST_APP/legacy-mounts-migrated"
run /bin/bash -c "dokku storage:migrate $TEST_APP"
echo "output: $output"
echo "status: $status"
assert_success
run /bin/bash -c "sudo test -f /var/lib/dokku/config/storage/$TEST_APP/legacy-mounts-migrated"
assert_failure
}
@test "(storage:migrate) converts legacy migration flag file into property" {
# Stage an upgrade-from-old-code scenario: a leftover flag file under
# data/storage-registry/migrations/<app>. storage:migrate (which runs
# convertLegacyMigrationFlag before migrateApp) must drain it into
# the property and remove the file.
flag_dir="/var/lib/dokku/data/storage-registry/migrations"
sudo mkdir -p "$flag_dir"
sudo touch "$flag_dir/$TEST_APP"
sudo chown dokku:dokku "$flag_dir/$TEST_APP"
sudo rm -f "/var/lib/dokku/config/storage/$TEST_APP/legacy-mounts-migrated"
run /bin/bash -c "dokku storage:migrate $TEST_APP"
echo "output: $output"
echo "status: $status"
assert_success
run /bin/bash -c "sudo test -f /var/lib/dokku/config/storage/$TEST_APP/legacy-mounts-migrated"
assert_success
run /bin/bash -c "sudo cat /var/lib/dokku/config/storage/$TEST_APP/legacy-mounts-migrated"
assert_success
assert_output "true"
run /bin/bash -c "sudo test -e $flag_dir/$TEST_APP"
assert_failure
}