deis / postgres

A PostgreSQL database used by Deis Workflow.
https://deis.com
MIT License
36 stars 22 forks source link

WAL-E needs AWS_INSTANCE_PROFILE=1 environment variable #193

Closed jwalters-gpsw closed 7 years ago

jwalters-gpsw commented 7 years ago

WAL-E needs AWS_INSTANCE_PROFILE=1 environment variable if used with AWS instanceRoles. See https://github.com/wal-e/wal-e/pull/289

This is a possible fix for https://github.com/deis/postgres/issues/192

deis-admin commented 7 years ago

Thanks for the contribution! Please ensure your commits follow our style guide. This code will be tested once a Deis maintainer reviews it.

bacongobbler commented 7 years ago

Jenkins, test this please.

bacongobbler commented 7 years ago

Can you check and confirm this works for you by testing with the following image once the Jenkins run finishes?

quay.io/deisci/postgres:git-99500ea

You can change the image by using kubectl -n deis edit deploy deis-database and change the image field. That should help determine if this works in an IAM setting. Thanks!

jwalters-gpsw commented 7 years ago

Yes. How long for the Jenkins run?

jwalters-gpsw commented 7 years ago

Not seeing that image label. Created locally and tried. Same issue. Will work on debugging.

jwalters-gpsw commented 7 years ago

After bumping the version of wal-e it runs on my cluster.

mboersma commented 7 years ago

Jenkins, add to whitelist.

jwalters-gpsw commented 7 years ago

Not sure what the issue is with Jenkins. Seems to be a key issue.

vdice commented 7 years ago

Seems to be a key issue.

Indeed -- thankfully appears to have been temporary. Kicked off a fresh build and it's progressing further...