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

Postgres-backup readme is misconfigured. #2524

Closed
je-bugshell opened this issue Feb 27, 2023 · 0 comments
Closed

Postgres-backup readme is misconfigured. #2524

je-bugshell opened this issue Feb 27, 2023 · 0 comments

Comments

@je-bugshell
Copy link

je-bugshell commented Feb 27, 2023

Description:

In the readme: configuring-playbook-postgres-backup.md there are multiple options for configuring file paths related to the postgres backups. See excerpt below.

Name Default value Description
devture_postgres_base_path "{{ matrix_base_data_path }}/postgres-backup" Base path for postgres-backup. Also see devture_postgres_data_path

According to the description the variable devture_postgres_base_path sets the location for the postgres-db backups. However, it seems that this variable does not set the backup-path (as stated in the description), but rather the actual databases location.

There seems to be a mix-up with the devture_postgres_backup_base_path variable, which instead produced the results that I was expecting.

Proposed fix:

Update docs to for the following variable name:

Name Default value Description
devture_postgres_backup_base_path "{{ matrix_base_data_path }}/postgres-backup" Base path for postgres-backup. Also see devture_postgres_data_path
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

No branches or pull requests

1 participant