diff options
author | Tim Rozet <trozet@redhat.com> | 2017-09-18 21:48:45 -0400 |
---|---|---|
committer | Tim Rozet <trozet@redhat.com> | 2017-09-20 17:43:32 -0400 |
commit | 02dc854a504569da15b0e41353a9f456e02599ad (patch) | |
tree | 8463a45f542e38f904fd1e1a7c4db8700f596e27 /apex/tests/test_apex_clean.py | |
parent | 946251720965cd5da9f6798d3ff931950524dd24 (diff) |
Improves clean to cleanup virsh nets and and other bridges
Currently only the admin and external bridges will be removed from the
jumphost which is only really clean for a baremetal deployment. Virtual
deployments will leave br-tenant, br-api, br-storage. This patch
removes those as well as deletes all Apex virsh networks from libvirt.
Also checks if authorized_keys exists before trying to edit it.
Change-Id: I1affe2733ba1dc38fd3b91fb14398454137f00a4
Signed-off-by: Tim Rozet <trozet@redhat.com>
Diffstat (limited to 'apex/tests/test_apex_clean.py')
-rw-r--r-- | apex/tests/test_apex_clean.py | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/apex/tests/test_apex_clean.py b/apex/tests/test_apex_clean.py index b6b9d428..b3ead6f7 100644 --- a/apex/tests/test_apex_clean.py +++ b/apex/tests/test_apex_clean.py @@ -100,3 +100,15 @@ class TestClean: ml.listDefinedDomains.return_value = ['undercloud'] ml.lookupByName.return_value = dummy_domain() assert clean.clean_vms() is None + + @patch('apex.network.jumphost.detach_interface_from_ovs') + @patch('apex.network.jumphost.remove_ovs_bridge') + @patch('libvirt.open') + def test_clean_networks(self, mock_libvirt, mock_jumphost_ovs_remove, + mock_jumphost_detach): + ml = mock_libvirt.return_value + ml.listNetworks.return_value = ['admin', 'external', 'tenant', 'blah'] + mock_net = ml.networkLookupByName.return_value + mock_net.isActive.return_value = True + clean.clean_networks() + assert_equal(mock_net.destroy.call_count, 3) |