Skip to content

Commit f34d3ee

Browse files
dumbbellmichaelklishin
authored andcommitted
peer_discovery_classic_config_SUITE: Fix successful_discovery_with_a_subset_of_nodes_coming_online
[Why] The testcase was broken as part of the work on Khepri (#7206): all nodes were started, making it an equivalent of the `successful_discovery` testcase. [How] We drop the first entry in the list of nodes given to `rabbit_ct_broker_helpers`. This way, it won't be started at all while still being listed in the classic config parameter.
1 parent a520090 commit f34d3ee

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

deps/rabbit/test/peer_discovery_classic_config_SUITE.erl

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -109,8 +109,10 @@ init_per_testcase(successful_discovery_with_a_subset_of_nodes_coming_online = Te
109109
],
110110
Config2 = rabbit_ct_helpers:set_config(Config1, [
111111
{rmq_nodename_suffix, Testcase},
112+
%% We remove the first node in the list: it will be considered by peer
113+
%% discovery (see `cluster_nodes' below), but it won't be started.
112114
%% note: this must not include the host part
113-
{rmq_nodes_count, NodeNames},
115+
{rmq_nodes_count, tl(NodeNames)},
114116
{rmq_nodes_clustered, false}
115117
]),
116118
NodeNamesWithHostname = [rabbit_nodes:make({Name, "localhost"}) || Name <- NodeNames],

0 commit comments

Comments
 (0)