Skip to content

Conversation

@bloever
Copy link
Contributor

@bloever bloever commented Nov 14, 2021

Current documentation in the postgres-operator fails to mention that WALG_AZ_PREFIX enables Azure Backups. This is a simple fix for the documentation so others won't be stranded.

Spilo image env variables: (https://github.com/zalando/spilo/blob/master/ENVIRONMENT.rst)

@FxKu
Copy link
Member

FxKu commented Nov 16, 2021

WALG_AZ_PREFIX is generated by spilo if wal_az_storage_account is configured. I think, that's why @apeschel left this variable out in #1537. You can overwrite it, but not sure if everything still works then.

@bloever
Copy link
Contributor Author

bloever commented Nov 16, 2021

My experience was that there was nothing in /run/etc/wal-e.d/env before i manually configured WALG_AZ_PREFIX. wal_az_storage_account was set from the start.

I guess this is the relevant part in the Spilo image?
(https://github.com/zalando/spilo/blob/b2b1c6988c81ed4705c08b1902712251b86a4f75/postgres-appliance/scripts/configure_spilo.py#L819)

@FxKu
Copy link
Member

FxKu commented Nov 16, 2021

Ok, just learned that I mixed this variable up with the scope prefix I guess. So it might indeed be needed.

@FxKu
Copy link
Member

FxKu commented Nov 16, 2021

👍

1 similar comment
@Jan-M
Copy link
Member

Jan-M commented Nov 18, 2021

👍

@FxKu FxKu merged commit a55639d into zalando:master Nov 19, 2021
@FxKu
Copy link
Member

FxKu commented Nov 19, 2021

Thanks again @bloever

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.

3 participants