Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 12, 2025

Trailing slashes in docker cp source paths cause the command to copy directory contents rather than directories themselves, breaking restore compatibility.

Changes

  • Removed trailing slashes from docker cp source paths in backup.sh:
    • /redis-data//redis-data
    • /caddy//caddy
    • /meilisearch//meilisearch
    • /qdrant//qdrant

Context

The restore script expects:

volumes-monolithic/cache/redis-data/
volumes-monolithic/caddy/caddy/

With trailing slashes, backup creates:

volumes-monolithic/cache/[redis files directly here]
volumes-monolithic/caddy/[caddy files directly here]

This mismatch causes restore to fail looking for non-existent subdirectories.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] WIP address feedback on backup script refactor Fix docker cp trailing slashes breaking backup/restore directory structure Nov 12, 2025
Copilot AI requested a review from bytestream November 12, 2025 13:11
@bytestream bytestream closed this Nov 12, 2025
@bytestream bytestream deleted the copilot/sub-pr-282 branch November 12, 2025 14:42
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.

2 participants