Closed bacongobbler closed 8 years ago
I'm gonna add integration tests for this so we can verify backups have been shipped to minio. That way nobody needs to test this manually!
found a bug with this: https://github.com/deis/wal-e/issues/2
going to mark as in-progress until that is fixed upstream.
relevant PR: https://github.com/deis/wal-e/pull/3
k, integration tests should fail and the logs will likely show the issue at deis/wal-e#3.
also removing LGTMs and manual testing labels because we discovered an issue, so this PR needs re-evaluation. No manual testing required any more though :8ball:
now it's going green! :tada:
back and ready for review.
backup retention and frequency of backups are configurable through environment variables.
Note that this is the exact same script we used in v1, bumping the backup frequency to every 12 hours. Yay portability!
closes #53 closes #32