openstack/kolla-ansible
Code Issues Proposed changes

Merge "host_os: Use assert on checks for readability"

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

18
ansible/roles/prechecks/tasks/host_os_checks.yml
View File

@ -1,22 +1,22 @@
--- ---
- name: Checking host OS distribution - name: Checking host OS distribution
fail: assert:
msg: >- that: ansible_facts.distribution in host_os_distributions
fail_msg: >-
Host OS distribution {{ ansible_facts.distribution }} is not supported. Host OS distribution {{ ansible_facts.distribution }} is not supported.
Supported distributions are: {{ host_os_distributions.keys() | join(', ') }} Supported distributions are: {{ host_os_distributions.keys() | join(', ') }}
when: ansible_facts.distribution not in host_os_distributions
- name: Checking host OS release or version - name: Checking host OS release or version
fail: assert:
msg: >- that:
- ansible_facts.distribution_release in host_os_distributions[ansible_facts.distribution] or
ansible_facts.distribution_version in host_os_distributions[ansible_facts.distribution] or
ansible_facts.distribution_major_version in host_os_distributions[ansible_facts.distribution]
fail_msg: >-
{{ ansible_facts.distribution }} release {{ ansible_facts.distribution_release }} {{ ansible_facts.distribution }} release {{ ansible_facts.distribution_release }}
version {{ ansible_facts.distribution_version }} is not supported. version {{ ansible_facts.distribution_version }} is not supported.
Supported releases are: Supported releases are:
{{ host_os_distributions[ansible_facts.distribution] | join(', ') }} {{ host_os_distributions[ansible_facts.distribution] | join(', ') }}
when:
- ansible_facts.distribution_release not in host_os_distributions[ansible_facts.distribution]
- ansible_facts.distribution_version not in host_os_distributions[ansible_facts.distribution]
- ansible_facts.distribution_major_version not in host_os_distributions[ansible_facts.distribution]
- name: Checking if CentOS is Stream - name: Checking if CentOS is Stream
become: true become: true