From: David Galloway <dgallowa@redhat.com>
Date: Thu, 20 May 2021 16:57:34 +0000 (-0400)
Subject: testnode: Always ifup the secondary NIC
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F627%2Fhead;p=ceph-cm-ansible.git

testnode: Always ifup the secondary NIC

If the NIC was already down, `ifdown` causes the playbook to fail.  We want to always (at least attempt to) take the NIC down, then bring it back up.

Signed-off-by: David Galloway <dgallowa@redhat.com>
---

diff --git a/roles/testnode/tasks/secondary_nic.yml b/roles/testnode/tasks/secondary_nic.yml
index 8416862..8eb2ef6 100644
--- a/roles/testnode/tasks/secondary_nic.yml
+++ b/roles/testnode/tasks/secondary_nic.yml
@@ -74,7 +74,7 @@
     - ansible_os_family == 'RedHat'
 
 - name: "Bounce {{ nic_to_configure }}"
-  shell: "ifdown {{ nic_to_configure }} && ifup {{ nic_to_configure }}"
+  shell: "ifdown {{ nic_to_configure }}; ifup {{ nic_to_configure }}"
   when:
     - wrote_network_config is changed
     - ansible_os_family == 'RedHat'