matrix-docker-ansible-deploy/roles/matrix-coturn/tasks/validate_config.yml

---

- name: Fail if required Coturn settings not defined
  fail:
    msg: >
      You need to define a required configuration setting (`{{ item }}`) for using Coturn.
  when: "vars[item] == ''"
  with_items:
    - "matrix_coturn_turn_external_ip_address"
    - "matrix_coturn_turn_static_auth_secret"
Make roles more independent of one another With this change, the following roles are now only dependent on the minimal `matrix-base` role: - `matrix-corporal` - `matrix-coturn` - `matrix-mailer` - `matrix-mxisd` - `matrix-postgres` - `matrix-riot-web` - `matrix-synapse` The `matrix-nginx-proxy` role still does too much and remains dependent on the others. Wiring up the various (now-independent) roles happens via a glue variables file (`group_vars/matrix-servers`). It's triggered for all hosts in the `matrix-servers` group. According to Ansible's rules of priority, we have the following chain of inclusion/overriding now: - role defaults (mostly empty or good for independent usage) - playbook glue variables (`group_vars/matrix-servers`) - inventory host variables (`inventory/host_vars/matrix.<your-domain>`) All roles default to enabling their main component (e.g. `matrix_mxisd_enabled: true`, `matrix_riot_web_enabled: true`). Reasoning: if a role is included in a playbook (especially separately, in another playbook), it should "work" by default. Our playbook disables some of those if they are not generally useful (e.g. `matrix_corporal_enabled: false`). 2019-01-16 16:05:48 +00:00			`---`

			`- name: Fail if required Coturn settings not defined`
			`fail:`
			`msg: >`
			You need to define a required configuration setting (`{{ item }}`) for using Coturn.
			`when: "vars[item] == ''"`
			`with_items:`
Define matrix_coturn_turn_external_ip_address in the playbook group vars This is more explicit than hiding it in the role defaults. People who reuse the roles in their own playbook (and not only) may incorrectly define `ansible_host` to be a hostname or some local address. Making it more explicit is more likely to prevent such mistakes. 2019-03-18 15:04:40 +00:00			`- "matrix_coturn_turn_external_ip_address"`
			`- "matrix_coturn_turn_static_auth_secret"`