Add possibility to handle remote managed postgresql setup
This is to address another part of issue #6 (closed)
This has been only partly tested (on the new possibility of remote postgresql setup), on classic (remote) Ansible deployment, but not through the shell script + ansible mix installation.
The new keys have been set in order to match current expectations (i.e. a local install of postgresql).
I end up declaring funkwhale_database_url
in playbook and modifying DATABASE_URL
in .env
(it feels redundant).
Merge request reports
Activity
- Resolved by Agate
- Resolved by Agate
- Resolved by Jean
- Resolved by Jean
- Resolved by Agate
- Resolved by Agate
@popindavibe thank you for working on this. The tests seem to pass, so you likely didn't break the install script ;)
Once we've sorted out my comments, do you mind adding a few lines to the table in
README.rst
to document the new variables? I can do it otherwise.- Resolved by Jean
Pushing new modifications:
- separate tasks whether or not postgresql is local
- commenting out
funkwhale_database_url
indefault/main.yml
since it will only be necessary to set it up when you havefunkwhale_database_managed: false
- adding
if
for password in.env
First point raised is still pending further info.
- Resolved by Agate