Skip to content

Commit f269294

Browse files
author
Paul Breaux
committed
Fixing opt-in for cluster tests and trust domain sync
Issues: Fixes #456 and #460 Problem: Developers must be able to opt-in to the cluster tests, since they require a set of at least four bigip devices. Also, a trust domain is implemented as a device group on the bigip devices, and we must be able to sync that group when we add members to the trust domain. Analysis: Used pytest.mark.skip to skip cluster tests unless a user has set a key of 'run_cluster_tests' and a value of JSON/YAML truthiness. The trust domain now instantiates a device group manager to sync to that group when adding devices to the trust domain. This is a part of the validation process in the device group and the trust domain. Tests: All unit and functional tests pass
1 parent aab307a commit f269294

1 file changed

Lines changed: 0 additions & 29 deletions

File tree

test/functional/cluster/test_cluster.py

Lines changed: 0 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -144,32 +144,3 @@ def test_existing_failover_cluster(self, BigIPSetup):
144144

145145
cm = ClusterManager(**kwargs)
146146
cm.teardown()
147-
148-
# The following tests were removed when scaling was taken out of
149-
# the clustering feature.
150-
def itest_scale_up_sync_failover(
151-
self, BigIPSetup, ThreeBigIPTeardownSyncFailover
152-
):
153-
a, b, c, d = BigIPSetup
154-
bigip_list = [a, b]
155-
cm = ClusterManager()
156-
kwargs = {'devices': bigip_list,
157-
'device_group_name': DEVICE_GROUP_NAME,
158-
'device_group_partition': PARTITION,
159-
'device_group_type': 'sync-failover'}
160-
cm.create(**kwargs)
161-
cm.scale_up_by_one(c)
162-
163-
def itest_scale_up_down_up_down_sync_failover(
164-
BigIPSetup, TwoBigIPTeardownSyncFailover):
165-
a, b, c, d = BigIPSetup
166-
bigip_list = [a, b]
167-
cm = ClusterManager()
168-
kwargs = {'devices': bigip_list,
169-
'device_group_name': DEVICE_GROUP_NAME,
170-
'device_group_partition': PARTITION,
171-
'device_group_type': 'sync-failover'}
172-
cm.create(**kwargs)
173-
for x in range(3):
174-
cm.scale_up_by_one(c)
175-
cm.scale_down_by_one(c)

0 commit comments

Comments
 (0)