Skip to content

Commit 949e0cb

Browse files
author
Dean Troyer
committed
Attempt to work around chronically failing server issues with aggregates and qos
So yeah, this is not kosher for functional tests, but we're testing the client interaction, not the raciness of Nova or Neutron. Failure to delete is not our problem. Change-Id: I21043f1de0fbacee1aec63110fb12a7cff42e0a0
1 parent f63bb18 commit 949e0cb

2 files changed

Lines changed: 15 additions & 4 deletions

File tree

openstackclient/tests/functional/compute/v2/test_aggregate.py

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,11 @@ def test_aggregate_list(self):
6767
'--property a=b ' +
6868
name1
6969
)
70-
self.addCleanup(self.openstack, 'aggregate delete ' + name1)
70+
self.addCleanup(
71+
self.openstack,
72+
'aggregate delete ' + name1,
73+
fail_ok=True,
74+
)
7175

7276
name2 = uuid.uuid4().hex
7377
self.openstack(
@@ -76,7 +80,11 @@ def test_aggregate_list(self):
7680
'--property c=d ' +
7781
name2
7882
)
79-
self.addCleanup(self.openstack, 'aggregate delete ' + name2)
83+
self.addCleanup(
84+
self.openstack,
85+
'aggregate delete ' + name2,
86+
fail_ok=True,
87+
)
8088

8189
cmd_output = json.loads(self.openstack(
8290
'aggregate list -f json'

openstackclient/tests/functional/network/v2/test_network_qos_policy.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,11 @@ def setUp(self):
3333
'network qos policy create -f json ' +
3434
self.NAME
3535
))
36-
self.addCleanup(self.openstack,
37-
'network qos policy delete ' + self.NAME)
36+
self.addCleanup(
37+
self.openstack,
38+
'network qos policy delete ' + self.NAME,
39+
fail_ok=True,
40+
)
3841
self.assertEqual(self.NAME, cmd_output['name'])
3942

4043
def test_qos_rule_create_delete(self):

0 commit comments

Comments
 (0)