openstack/kolla-ansible
Code Issues Proposed changes

Merge "rabbitmq: Use assert on checks for readability"

Browse Source
This commit is contained in:
Zuul 2023-02-17 08:40:25 +00:00 committed by Gerrit Code Review
commit 4a0f058c54
1 changed files with 12 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
ansible/roles/rabbitmq/tasks/precheck.yml
View File

@ -78,30 +78,30 @@
- not item.1 is match('^'+('api' | kolla_address(item.0.item))+'\\b') - not item.1 is match('^'+('api' | kolla_address(item.0.item))+'\\b')
- name: Check if TLS certificate exists for RabbitMQ - name: Check if TLS certificate exists for RabbitMQ
assert:
that: cert | length > 0
fail_msg: No TLS certificate provided for RabbitMQ.
vars: vars:
cert: "{{ query('first_found', paths, errors='ignore') }}" cert: "{{ query('first_found', paths, errors='ignore') }}"
paths: paths:
- "{{ kolla_certificates_dir }}/{{ inventory_hostname }}/rabbitmq-cert.pem" - "{{ kolla_certificates_dir }}/{{ inventory_hostname }}/rabbitmq-cert.pem"
- "{{ kolla_certificates_dir }}/{{ inventory_hostname }}-cert.pem" - "{{ kolla_certificates_dir }}/{{ inventory_hostname }}-cert.pem"
- "{{ kolla_certificates_dir }}/rabbitmq-cert.pem" - "{{ kolla_certificates_dir }}/rabbitmq-cert.pem"
fail:
msg: No TLS certificate provided for RabbitMQ.
when: when:
- rabbitmq_enable_tls | bool - rabbitmq_enable_tls | bool
- cert | length == 0
- name: Check if TLS key exists for RabbitMQ - name: Check if TLS key exists for RabbitMQ
assert:
that: key | length > 0
fail_msg: No TLS key provided for RabbitMQ.
vars: vars:
key: "{{ query('first_found', paths, errors='ignore') }}" key: "{{ query('first_found', paths, errors='ignore') }}"
paths: paths:
- "{{ kolla_certificates_dir }}/{{ inventory_hostname }}/rabbitmq-key.pem" - "{{ kolla_certificates_dir }}/{{ inventory_hostname }}/rabbitmq-key.pem"
- "{{ kolla_certificates_dir }}/{{ inventory_hostname }}-key.pem" - "{{ kolla_certificates_dir }}/{{ inventory_hostname }}-key.pem"
- "{{ kolla_certificates_dir }}/rabbitmq-key.pem" - "{{ kolla_certificates_dir }}/rabbitmq-key.pem"
fail:
msg: No TLS key provided for RabbitMQ.
when: when:
- rabbitmq_enable_tls | bool - rabbitmq_enable_tls | bool
- key | length == 0
- name: Checking free port for outward RabbitMQ - name: Checking free port for outward RabbitMQ
wait_for: wait_for:
@ -169,29 +169,29 @@
- not item.1 is match('^'+('api' | kolla_address(item.0.item))+'\\b') - not item.1 is match('^'+('api' | kolla_address(item.0.item))+'\\b')
- name: Check if TLS certificate exists for outward RabbitMQ - name: Check if TLS certificate exists for outward RabbitMQ
assert:
that: cert | length > 0
fail_msg: No TLS certificate provided for outward RabbitMQ.
vars: vars:
cert: "{{ query('first_found', paths, errors='ignore') }}" cert: "{{ query('first_found', paths, errors='ignore') }}"
paths: paths:
- "{{ kolla_certificates_dir }}/{{ inventory_hostname }}/outward_rabbitmq-cert.pem" - "{{ kolla_certificates_dir }}/{{ inventory_hostname }}/outward_rabbitmq-cert.pem"
- "{{ kolla_certificates_dir }}/{{ inventory_hostname }}-cert.pem" - "{{ kolla_certificates_dir }}/{{ inventory_hostname }}-cert.pem"
- "{{ kolla_certificates_dir }}/outward_rabbitmq-cert.pem" - "{{ kolla_certificates_dir }}/outward_rabbitmq-cert.pem"
fail:
msg: No TLS certificate provided for outward RabbitMQ.
when: when:
- enable_outward_rabbitmq | bool - enable_outward_rabbitmq | bool
- rabbitmq_enable_tls | bool - rabbitmq_enable_tls | bool
- cert | length == 0
- name: Check if TLS key exists for outward RabbitMQ - name: Check if TLS key exists for outward RabbitMQ
assert:
that: key | length > 0
msg: No TLS key provided for outward RabbitMQ.
vars: vars:
key: "{{ query('first_found', paths, errors='ignore') }}" key: "{{ query('first_found', paths, errors='ignore') }}"
paths: paths:
- "{{ kolla_certificates_dir }}/{{ inventory_hostname }}/outward_rabbitmq-key.pem" - "{{ kolla_certificates_dir }}/{{ inventory_hostname }}/outward_rabbitmq-key.pem"
- "{{ kolla_certificates_dir }}/{{ inventory_hostname }}-key.pem" - "{{ kolla_certificates_dir }}/{{ inventory_hostname }}-key.pem"
- "{{ kolla_certificates_dir }}/outward_rabbitmq-key.pem" - "{{ kolla_certificates_dir }}/outward_rabbitmq-key.pem"
fail:
msg: No TLS key provided for outward RabbitMQ.
when: when:
- enable_outward_rabbitmq | bool - enable_outward_rabbitmq | bool
- rabbitmq_enable_tls | bool - rabbitmq_enable_tls | bool
- key | length == 0