Skip to content

Commit ffdc9e5

Browse files
committed
Update router test
Change-Id: Ib78062a136758a29a3078662f80c336967cf176c
1 parent 618c77f commit ffdc9e5

File tree

1 file changed

+6
-10
lines changed

1 file changed

+6
-10
lines changed

tests/e2e/mysqloperator/cluster/router_t.py

+6-10
Original file line numberDiff line numberDiff line change
@@ -465,8 +465,6 @@ def test_1_verify_provided_options(self):
465465

466466

467467
def test_2_update_bootstrap_config(self):
468-
[old_pod] = kutil.ls_po(self.ns, pattern="mycluster-router-.*")
469-
470468
patch = {
471469
"spec": {
472470
"router": {
@@ -476,18 +474,15 @@ def test_2_update_bootstrap_config(self):
476474
}
477475
}
478476
}
477+
waiter = tutil.get_deploy_rollover_update_waiter(self, "mycluster", timeout=300, delay=5)
479478
kutil.patch_ic(self.ns, "mycluster", patch, type="merge")
480-
481-
self.wait_pod_gone(old_pod["NAME"], ns=self.ns)
482-
self.wait_routers("mycluster-router-.*", 1)
479+
waiter()
483480

484481
[new_pod] = kutil.ls_po(self.ns, pattern="mycluster-router-.*")
485482
router_config = kutil.cat(self.ns, new_pod["NAME"], "/tmp/mysqlrouter/mysqlrouter.conf")
486483
self.assertIn(b"name=othername", router_config, "router config does not contain the name")
487484

488485
def test_3_update_config(self):
489-
[old_pod] = kutil.ls_po(self.ns, pattern="mycluster-router-.*")
490-
491486
patch = {
492487
"spec": {
493488
"router": {
@@ -497,10 +492,9 @@ def test_3_update_config(self):
497492
}
498493
}
499494
}
495+
waiter = tutil.get_deploy_rollover_update_waiter(self, "mycluster", timeout=300, delay=5)
500496
kutil.patch_ic(self.ns, "mycluster", patch, type="merge")
501-
502-
self.wait_pod_gone(old_pod["NAME"], ns=self.ns)
503-
self.wait_routers("mycluster-router-.*", 1)
497+
waiter()
504498

505499
[new_pod] = kutil.ls_po(self.ns, pattern="mycluster-router-.*")
506500
self.assertTrue(kutil.file_exists(self.ns, new_pod["NAME"], "/tmp/with`backtick"))
@@ -524,6 +518,7 @@ def test_5_add_routing_options(self):
524518
}
525519
}
526520
kutil.patch_ic(self.ns, "mycluster", patch, type="merge")
521+
sleep(5)
527522
routing_options = get_routing_options(self.ns, "mycluster-0")
528523
rules = routing_options["configuration"]["routing_rules"]
529524
self.assertEqual(rules["read_only_targets"], "read_replicas")
@@ -538,6 +533,7 @@ def test_6_remove_routing_options(self):
538533
}
539534
}
540535
kutil.patch_ic(self.ns, "mycluster", patch, type="merge")
536+
sleep(5)
541537
routing_options = get_routing_options(self.ns, "mycluster-0")
542538
rules = routing_options["configuration"]["routing_rules"]
543539
self.assertEqual(rules["read_only_targets"], "secondaries")

0 commit comments

Comments
 (0)