matrix-docker-ansible-deploy/roles/custom/matrix-postgres/tasks/util/create_additional_databases.yml

---

- name: Ensure matrix-postgres is started
  ansible.builtin.service:
    name: matrix-postgres
    state: started
    daemon_reload: true
  register: matrix_postgres_service_start_result

- name: Wait a bit, so that Postgres can start
  ansible.builtin.wait_for:
    timeout: "{{ matrix_postgres_additional_databases_postgres_start_wait_timeout_seconds }}"
  delegate_to: 127.0.0.1
  become: false
  when: "matrix_postgres_service_start_result.changed | bool"

- name: Create additional Postgres user and database
  ansible.builtin.include_tasks: "{{ role_path }}/tasks/util/create_additional_database.yml"
  with_items: "{{ matrix_postgres_additional_databases }}"
  loop_control:
    loop_var: additional_db
  # Suppress logging to avoid dumping the credentials to the shell
  no_log: true
Relocate Postgres additional database creation logic Moving it above the "uninstalling" set of tasks is better. Extracting it out to another file at the same time, for readability, especially given that it will probably have to become more complex in the future (potentially installing `jq`, etc.) 2020-12-13 19:40:32 +00:00			`---`
Prevent weird loop error > Invalid data passed to 'loop', it requires a list, got this instead: matrix_postgres_additional_databases. Hint: If you passed a list/dict of just one element, try adding wantlist=True to your lookup invocation or use q/query instead of lookup. Well, or working around it, as I've done in this commit (which seems more sane than `wantlist=True` stuff). 2020-12-13 20:56:56 +00:00
Ensure matrix-postgres.service is started before creating additional users/databases 2020-12-13 22:59:59 +00:00			`- name: Ensure matrix-postgres is started`
Use fully-qualified module names for builtin Ansible modules Related to https://github.com/spantaleev/matrix-docker-ansible-deploy/pull/1939 2022-07-18 07:39:08 +00:00			`ansible.builtin.service:`
Ensure matrix-postgres.service is started before creating additional users/databases 2020-12-13 22:59:59 +00:00			`name: matrix-postgres`
			`state: started`
fix: all praise the allmighty yamllinter 2022-02-05 20:32:54 +00:00			`daemon_reload: true`
Ensure matrix-postgres.service is started before creating additional users/databases 2020-12-13 22:59:59 +00:00			`register: matrix_postgres_service_start_result`
Relocate Postgres additional database creation logic Moving it above the "uninstalling" set of tasks is better. Extracting it out to another file at the same time, for readability, especially given that it will probably have to become more complex in the future (potentially installing `jq`, etc.) 2020-12-13 19:40:32 +00:00
Ensure matrix-postgres.service is started before creating additional users/databases 2020-12-13 22:59:59 +00:00			`- name: Wait a bit, so that Postgres can start`
More ansible-lint fixes 2022-07-18 08:22:05 +00:00			`ansible.builtin.wait_for:`
Ensure matrix-postgres.service is started before creating additional users/databases 2020-12-13 22:59:59 +00:00			`timeout: "{{ matrix_postgres_additional_databases_postgres_start_wait_timeout_seconds }}"`
			`delegate_to: 127.0.0.1`
			`become: false`
More ansible-lint fixes 2022-07-18 08:22:05 +00:00			`when: "matrix_postgres_service_start_result.changed \| bool"`
Ensure matrix-postgres.service is started before creating additional users/databases 2020-12-13 22:59:59 +00:00
			`- name: Create additional Postgres user and database`
More ansible-lint fixes 2022-07-18 08:22:05 +00:00			`ansible.builtin.include_tasks: "{{ role_path }}/tasks/util/create_additional_database.yml"`
Do not use the postgresql_user/postgresql_db modules While these modules are really nice and helpful, we can't use them for at least 2 reasons: - for us, Postgres runs in a container on a private Docker network (`--network=matrix`) without usually being exposed to the host. These modules execute on the host so they won't be able to reach it. - these modules require `psycopg2`, so we need to install it before using it. This might or might not be its own can of worms. 2020-12-13 22:25:13 +00:00			`with_items: "{{ matrix_postgres_additional_databases }}"`
			`loop_control:`
			`loop_var: additional_db`
			`# Suppress logging to avoid dumping the credentials to the shell`
			`no_log: true`