Skip to content
Snippets Groups Projects

Add possibility to handle remote managed postgresql setup

Merged Jean requested to merge popindavibe/ansible:mr_psql_remote into master

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).

Edited by Jean

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Agate
  • Agate
  • Agate
  • Agate
  • 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.

  • Agate
  • Jean added 1 commit

    added 1 commit

    Compare with previous version

  • Author Contributor

    Pushing new modifications:

    • separate tasks whether or not postgresql is local
    • commenting out funkwhale_database_url in default/main.yml since it will only be necessary to set it up when you have funkwhale_database_managed: false
    • adding if for password in .env

    First point raised is still pending further info.

  • Author Contributor

    oops, I broke the install script...

  • Jean added 1 commit

    added 1 commit

    Compare with previous version

  • Jean added 1 commit

    added 1 commit

    Compare with previous version

  • Author Contributor
    • removing the funwhale_database_host_app bit in .env if it's localhost
  • Agate
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading