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

Update Docker backup docs #2550

Open
fluid217 opened this issue Apr 19, 2020 · 1 comment
Open

Update Docker backup docs #2550

fluid217 opened this issue Apr 19, 2020 · 1 comment

Comments

@fluid217
Copy link

Description

In docs/docker-postgres-backups.rst it's suggested to use:

docker-compose -f local.yml exec postgres backup

I'd suggest updating that to:

 docker-compose -f local.yml exec -T postgres backup

Rationale

Running the command as documented from a Cron job fails with:

The input device is not a TTY.

This can be fixed by adding the above -T switch.

Use case(s) / visualization(s)

Works fine as documented if backing up by hand. Cron jobs however have potential to fail.

@browniebroke
Copy link
Member

Sounds good. I wouldn't include this option in all commands, but I think it would be great to add a paragraph at the end of the page on how to run backups in a crontab.

We could link to some external documentation for those unfamiliar with cron and maybe suggest an entry for daily backups.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants