openstack/openstack-ansible
Code Issues Proposed changes

Fix double conditional bug

Browse Source 
Currently there are two conditional statements
on a single task, so only the last is applied.

Change-Id: Ibc5c65b44005baacc43f7eb433097ec382d60bbd
This commit is contained in:
Jimmy McCrory 2017-07-18 12:16:40 -07:00 committed by Jesse Pretorius (odyssey4me)
parent 54ef0d3300
commit 1c9ff37fff
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
tests/roles/bootstrap-host/tasks/prepare_networking.yml
View File

@ -97,14 +97,15 @@
- name: Shut down the network interfaces
command: "ifdown {{ item.name }}"
when: osa_interfaces | changed or osa_multinode_interfaces | changed or network_interfaces_rhel | changed
with_items:
- { name: br-mgmt }
- { name: br-storage }
- { name: br-vlan }
- { name: br-vxlan }
- { name: br-dbaas, enabled: "{{ (bootstrap_host_scenario == 'translations') | bool }}" }
when: item.enabled | default(True)
when:
- osa_interfaces | changed or osa_multinode_interfaces | changed or network_interfaces_rhel | changed
- item.enabled | default(True)
tags:
- networking-interfaces-stop
@ -128,14 +129,15 @@
- name: Start the network interfaces
command: "ifup {{ item.name }}"
when: osa_interfaces | changed or network_interfaces_rhel | changed
with_items:
- { name: br-mgmt }
- { name: br-storage }
- { name: br-vlan }
- { name: br-vxlan }
- { name: br-dbaas, enabled: "{{ (bootstrap_host_scenario == 'translations') | bool }}" }
when: item.enabled | default(True)
when:
- osa_interfaces | changed or network_interfaces_rhel | changed
- item.enabled | default(True)
tags:
- networking-interfaces-start