Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Alpine's BusyBox cron instead of go-cron #28

Open
eeshugerman opened this issue Jan 29, 2023 · 0 comments
Open

Use Alpine's BusyBox cron instead of go-cron #28

eeshugerman opened this issue Jan 29, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@eeshugerman
Copy link
Owner

It makes me a little uneasy that the cron we're using is just a binary curled from a hardcoded GitHub release URL of a little-known project.

Instead, we should do something like this: https://stackoverflow.com/a/47960145

This would probably be a breaking change (go-cron offers some special-sauce scheduling syntax, IIRC), so it would be a good opportunity to switch to a new image tagging convention which incorporates a version for postgres-backup-s3 itself, in addition to the PostgreSQL version.

@eeshugerman eeshugerman added the enhancement New feature or request label Jan 29, 2023
@eeshugerman eeshugerman changed the title Use cron from Alpine repos instead of go-cron Use Alpine's BusyBox cron instead of go-cron Jan 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant