From e76233a222990ac7575d1a0217ef58e7e20efce8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jean-S=C3=A9bastien=20P=C3=A9dron?= Date: Thu, 13 Feb 2025 10:25:07 +0100 Subject: [PATCH 1/2] clustering_management_SUITE: Use old node as seed node [Why] During mixed-version testing, the old node might not be able to join or rejoin a cluster if the other nodes run a newer Khepri machine version. [How] The old node is used as the cluster seed node and is never touched otherwise. Other nodes are restarted or join the cluster later. --- .../test/clustering_management_SUITE.erl | 30 +++++++++---------- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/deps/rabbit/test/clustering_management_SUITE.erl b/deps/rabbit/test/clustering_management_SUITE.erl index 7e18242ccaea..bfa8959c825a 100644 --- a/deps/rabbit/test/clustering_management_SUITE.erl +++ b/deps/rabbit/test/clustering_management_SUITE.erl @@ -337,7 +337,7 @@ restart_cluster_node(Config) -> assert_clustered([Rabbit, Hare]). join_and_part_cluster_in_khepri(Config) -> - [Rabbit, Hare, Bunny] = cluster_members(Config), + [Rabbit, Bunny, Hare] = cluster_members(Config), assert_not_clustered(Rabbit), assert_not_clustered(Hare), assert_not_clustered(Bunny), @@ -447,38 +447,38 @@ join_to_start_interval(Config) -> assert_clustered([Rabbit, Hare]). join_cluster_in_minority(Config) -> - [Rabbit, Hare, Bunny] = cluster_members(Config), + [Rabbit, Bunny, Hare] = cluster_members(Config), assert_not_clustered(Rabbit), assert_not_clustered(Hare), assert_not_clustered(Bunny), - stop_join_start(Config, Bunny, Rabbit), + stop_join_start(Config, Rabbit, Bunny), assert_clustered([Rabbit, Bunny]), - ok = rabbit_ct_broker_helpers:stop_node(Config, Bunny), + ok = rabbit_ct_broker_helpers:stop_node(Config, Rabbit), ok = stop_app(Config, Hare), - ?assertEqual(ok, join_cluster(Config, Hare, Rabbit, false)), + ?assertEqual(ok, join_cluster(Config, Hare, Bunny, false)), - ok = rabbit_ct_broker_helpers:start_node(Config, Bunny), + ok = rabbit_ct_broker_helpers:start_node(Config, Rabbit), ?assertEqual(ok, join_cluster(Config, Hare, Rabbit, false)), ?assertEqual(ok, start_app(Config, Hare)), assert_clustered([Rabbit, Bunny, Hare]). join_cluster_with_rabbit_stopped(Config) -> - [Rabbit, Hare, Bunny] = cluster_members(Config), + [Rabbit, Bunny, Hare] = cluster_members(Config), assert_not_clustered(Rabbit), assert_not_clustered(Hare), assert_not_clustered(Bunny), - stop_join_start(Config, Bunny, Rabbit), + stop_join_start(Config, Rabbit, Bunny), assert_clustered([Rabbit, Bunny]), - ok = stop_app(Config, Bunny), + ok = stop_app(Config, Rabbit), ok = stop_app(Config, Hare), - ?assertEqual(ok, join_cluster(Config, Hare, Rabbit, false)), + ?assertEqual(ok, join_cluster(Config, Hare, Bunny, false)), - ok = start_app(Config, Bunny), + ok = start_app(Config, Rabbit), ?assertEqual(ok, join_cluster(Config, Hare, Rabbit, false)), ?assertEqual(ok, start_app(Config, Hare)), @@ -1119,7 +1119,7 @@ await_running_count_in_khepri(Config) -> await_running_count, [5, 1000])). start_nodes_in_reverse_order(Config) -> - [Rabbit, Hare, Bunny] = cluster_members(Config), + [Rabbit, Bunny, Hare] = cluster_members(Config), assert_not_clustered(Rabbit), assert_not_clustered(Hare), assert_not_clustered(Bunny), @@ -1142,7 +1142,7 @@ start_nodes_in_reverse_order(Config) -> %% Test booting nodes in the wrong order for Mnesia. Interesting... start_nodes_in_stop_order(Config) -> - [Rabbit, Hare, Bunny] = cluster_members(Config), + [Rabbit, Bunny, Hare] = cluster_members(Config), assert_not_clustered(Rabbit), assert_not_clustered(Hare), assert_not_clustered(Bunny), @@ -1167,7 +1167,7 @@ start_nodes_in_stop_order(Config) -> end. start_nodes_in_stop_order_in_khepri(Config) -> - [Rabbit, Hare, Bunny] = cluster_members(Config), + [Rabbit, Bunny, Hare] = cluster_members(Config), assert_not_clustered(Rabbit), assert_not_clustered(Hare), assert_not_clustered(Bunny), @@ -1190,7 +1190,7 @@ start_nodes_in_stop_order_in_khepri(Config) -> %% TODO test force_boot with Khepri involved start_nodes_in_stop_order_with_force_boot(Config) -> - [Rabbit, Hare, Bunny] = cluster_members(Config), + [Rabbit, Bunny, Hare] = cluster_members(Config), assert_not_clustered(Rabbit), assert_not_clustered(Hare), assert_not_clustered(Bunny), From f088c4f5444f123cdbd8e08fc73cd48390fe0765 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jean-S=C3=A9bastien=20P=C3=A9dron?= Date: Thu, 13 Feb 2025 10:39:54 +0100 Subject: [PATCH 2/2] clustering_management_SUITE: Skip `start_with_invalid_schema_in_path` with Khepri [Why] This test plays with the Mnesia database explicitly. --- deps/rabbit/test/clustering_management_SUITE.erl | 1 - 1 file changed, 1 deletion(-) diff --git a/deps/rabbit/test/clustering_management_SUITE.erl b/deps/rabbit/test/clustering_management_SUITE.erl index bfa8959c825a..426f5e35e950 100644 --- a/deps/rabbit/test/clustering_management_SUITE.erl +++ b/deps/rabbit/test/clustering_management_SUITE.erl @@ -76,7 +76,6 @@ groups() -> status_with_alarm, pid_file_and_await_node_startup_in_khepri, await_running_count_in_khepri, - start_with_invalid_schema_in_path, persistent_cluster_id, stop_start_cluster_node, restart_cluster_node,