Skip to content

Commit

Permalink
peer_discovery_classic_config_SUITE: Fix `successful_discovery_with_a…
Browse files Browse the repository at this point in the history
…_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.
  • Loading branch information
dumbbell committed Dec 4, 2023
1 parent ce8fa86 commit 8a59ae3
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion deps/rabbit/test/peer_discovery_classic_config_SUITE.erl
Original file line number Diff line number Diff line change
Expand Up @@ -109,8 +109,10 @@ init_per_testcase(successful_discovery_with_a_subset_of_nodes_coming_online = Te
],
Config2 = rabbit_ct_helpers:set_config(Config1, [
{rmq_nodename_suffix, Testcase},
%% We remove the first node in the list: it will be considered by peer
%% discovery (see `cluster_nodes' below), but it won't be started.
%% note: this must not include the host part
{rmq_nodes_count, NodeNames},
{rmq_nodes_count, tl(NodeNames)},
{rmq_nodes_clustered, false}
]),
NodeNamesWithHostname = [rabbit_nodes:make({Name, "localhost"}) || Name <- NodeNames],
Expand Down

0 comments on commit 8a59ae3

Please sign in to comment.