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

feat(postgres-backup-s3): update AWS CLI to v2; allow support for mul… #135

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

opichon
Copy link

@opichon opichon commented May 2, 2021

…tiple databases; allow support for defining sensitive env vars in files (eg Docker secrets)

AWS CLI updated to v2; python removed; glibc added
Added support for multiple databases (note POSTGRES_DATABASE env var is deprecated and replaced by POSTGRES_DATABASES) -- this means this version of the docker image is not backward-compatible with previosu releases.
Added support for _FILE env vars for sensitive values. This allows using the image with Docker secrets.

For example,

service:
  postgres-backup
    environment:
      POSTGRES_PASSWORD_FILE=/run/secrets/postgres-password
      ...
    secrets:
      - postgres-password

…tiple databases; allow support for defining sensitive env vars in files (eg Docker secrets)

AWS CLI updated to v2; python removed; glibc added
Added support for multiple databases (note POSTGRES_DATABASE env var is deprecated and replaced by POSTGRES_DATABASES) -- this means this version of the docker image is not backward-compatible with previosu releases.
Added support for _FILE env vars for sensitive values. This allows using the image with Docker secrets.

For example,

```
service:
  postgres-backup
    environment:
      POSTGRES_PASSWORD_FILE=/run/secrets/postgres-password
      ...
    secrets:
      - postgres-password
```
@iloveitaly
Copy link
Collaborator

@opichon could you rebase this on master so I could take a look?

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