Skip to content

Commit

Permalink
Add checks for cifmw_devscripts_create_logical_volume is defined
Browse files Browse the repository at this point in the history
  • Loading branch information
frenzyfriday authored and openshift-merge-bot[bot] committed Dec 17, 2024
1 parent 7c36a5a commit ff9f827
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion roles/reproducer/tasks/ocp_layout_assertions.yml
Original file line number Diff line number Diff line change
Expand Up @@ -84,6 +84,7 @@
- name: Ensure we don't set some parameters when no extra disks
when:
- cifmw_devscripts_create_logical_volume is defined
- _cifmw_libvirt_manager_layout.vms.ocp.extra_disks_num is undefined or
_cifmw_libvirt_manager_layout.vms.ocp.extra_disks_num == 0
ansible.builtin.assert:
Expand Down Expand Up @@ -141,7 +142,8 @@
vars:
_cinder_vols: >-
{{
(cifmw_devscripts_create_logical_volume | bool) |
(cifmw_devscripts_create_logical_volume is defined and
cifmw_devscripts_create_logical_volume | bool) |
ternary(cifmw_devscripts_cinder_volume_pvs, [])
}}
_lvms_vols: >-
Expand Down Expand Up @@ -178,6 +180,7 @@
block:
- name: Ensure no allocation overlap
when:
- cifmw_devscripts_create_logical_volume is defined
- cifmw_devscripts_create_logical_volume | bool
- cifmw_use_lvms | default(false) | bool
ansible.builtin.assert:
Expand All @@ -202,6 +205,7 @@
- name: Ensure Cinder PVs allocated disks are available
when:
- cifmw_devscripts_create_logical_volume is defined
- cifmw_devscripts_create_logical_volume | bool
ansible.builtin.assert:
that:
Expand Down

0 comments on commit ff9f827

Please sign in to comment.