Skip to content

Commit 013d078

Browse files
shivanand-adkyashwin2002
authored andcommitted
Fix AttributeError: replace add_back_node with re_add_node in disk/multi-node failover tests
ClusterRestAPI exposes re_add_node, not add_back_node; the old method only exists on the legacy RestConnection client. Change-Id: Ief96a5a6ad776f5191f69d4202843248958da194 Reviewed-on: https://review.couchbase.org/c/TAF/+/245756 Reviewed-by: Ashwin <ashwin.govindarajulu@couchbase.com> Tested-by: Ashwin <ashwin.govindarajulu@couchbase.com>
1 parent a1664a9 commit 013d078

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

pytests/failover/DiskFailoverTests.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -309,7 +309,7 @@ def test_disk_autofailover_and_addback_of_node(self):
309309
self.log.info(self.server_to_fail[0])
310310
self.nodes = self.cluster_util.get_otp_nodes(self.orchestrator)
311311
self.log.info("Adding back node {0}".format(self.server_to_fail[0].ip))
312-
self.rest.add_back_node("ns_1@{}".format(self.server_to_fail[0].ip))
312+
self.rest.re_add_node("ns_1@{}".format(self.server_to_fail[0].ip))
313313
self.rest.set_recovery_type("ns_1@{}".format(self.server_to_fail[
314314
0].ip),
315315
self.recovery_strategy)

pytests/failover/MultiNodeAutoFailoverTests.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -230,7 +230,7 @@ def test_autofailover_and_addback_of_node(self):
230230
self.sleep(30)
231231
self.nodes = self.cluster_util.get_nodes(self.orchestrator)
232232
for node in self.server_to_fail:
233-
self.rest.add_back_node("ns_1@{}".format(node.ip))
233+
self.rest.re_add_node("ns_1@{}".format(node.ip))
234234
self.rest.set_recovery_type("ns_1@{}".format(node.ip),
235235
self.recovery_strategy)
236236
self.rest.rebalance(known_nodes=[node.id for node in self.nodes])

0 commit comments

Comments
 (0)