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

Migrating to devture-prefixed variables disables postgres-backup? #2305

Closed
cxr6548 opened this issue Nov 30, 2022 · 1 comment
Closed

Migrating to devture-prefixed variables disables postgres-backup? #2305

cxr6548 opened this issue Nov 30, 2022 · 1 comment

Comments

@cxr6548
Copy link

cxr6548 commented Nov 30, 2022

Describe the bug
The first time I ran git pull && make roles && ansible-playbook -i inventory/hosts setup.yml --tags=setup-all,ensure-matrix-users-created,start -K after the Nov 28th update I got a message saying that I have data in /matrix/postgres-backup but the backup service isn't being used and I can freely delete the data. I foolishly closed the terminal and re-running the setup doesn't produce the message again.

To Reproduce
My vars.yml file looks like this:

matrix_domain: mycomain.com
matrix_homeserver_implementation: synapse

matrix_homeserver_generic_secret_key: 'no'

matrix_ssl_lets_encrypt_support_email: 'myemail'

devture_postgres_connection_password: 'no'

matrix_nginx_proxy_base_domain_serving_enabled: true
matrix_synapse_max_upload_size_mb: 50
matrix_client_element_enabled: false
matrix_synapse_presence_enabled: false

devture_postgres_backup_enabled: true

The final line above is what produces the behavior.

Expected behavior
I expect to backups to be enabled with new devture prefixes.

Matrix Server:

  • OS: Ubuntu 22.04
  • Architecture amd64

Additional context
Add any other context about the problem here.

@spantaleev
Copy link
Owner

You shouldn't have renamed matrix_postgres_backup_* variables. Only matrix_postgres_* variables had to be renamed (except for matrix_postgres_backup_*).

This issue brings up a few good points though:

  • the changelog entry was confusing and not making it clear that matrix_postgres_backup variables were unaffected.. The Ansible task that reported the variables to be renamed was much better at explaining this
  • it makes sense for the matrix-postgres-backup role to be extracted out just the same, which I've done now - https://github.com/devture/com.devture.ansible.role.postgres_backup . The Matrix playbook has been updated to use this external role. You'll need to rename matrix_postgres_backup variables to devture_postgres_backup just the same.

In short, from now on, all Postgres-related roles are external and all matrix_postgres-prefixed variables can be renamed to devture_postgres.

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

2 participants