Skip to content

Commit 67883d8

Browse files
mkuratczykLoisSotoLopez
authored andcommitted
[skip ci] Disable force_shrink_member_to_current_member in mixed-version
1 parent 456e07b commit 67883d8

File tree

1 file changed

+29
-24
lines changed

1 file changed

+29
-24
lines changed

deps/rabbit/test/quorum_queue_SUITE.erl

Lines changed: 29 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -1285,38 +1285,43 @@ single_active_consumer_priority(Config) ->
12851285
ok.
12861286

12871287
force_shrink_member_to_current_member(Config) ->
1288-
[Server0, Server1, Server2] =
1289-
rabbit_ct_broker_helpers:get_node_configs(Config, nodename),
1288+
case rabbit_ct_helpers:is_mixed_versions() of
1289+
true ->
1290+
{skip, "Should not run in mixed version environments"};
1291+
_ ->
1292+
[Server0, Server1, Server2] =
1293+
rabbit_ct_broker_helpers:get_node_configs(Config, nodename),
12901294

1291-
Ch = rabbit_ct_client_helpers:open_channel(Config, Server0),
1292-
QQ = ?config(queue_name, Config),
1293-
?assertEqual({'queue.declare_ok', QQ, 0, 0},
1294-
declare(Ch, QQ, [{<<"x-queue-type">>, longstr, <<"quorum">>}])),
1295+
Ch = rabbit_ct_client_helpers:open_channel(Config, Server0),
1296+
QQ = ?config(queue_name, Config),
1297+
?assertEqual({'queue.declare_ok', QQ, 0, 0},
1298+
declare(Ch, QQ, [{<<"x-queue-type">>, longstr, <<"quorum">>}])),
12951299

1296-
RaName = ra_name(QQ),
1297-
rabbit_ct_client_helpers:publish(Ch, QQ, 3),
1298-
wait_for_messages_ready([Server0], RaName, 3),
1300+
RaName = ra_name(QQ),
1301+
rabbit_ct_client_helpers:publish(Ch, QQ, 3),
1302+
wait_for_messages_ready([Server0], RaName, 3),
12991303

1300-
{ok, Q0} = rpc:call(Server0, rabbit_amqqueue, lookup, [QQ, <<"/">>]),
1301-
#{nodes := Nodes0} = amqqueue:get_type_state(Q0),
1302-
?assertEqual(3, length(Nodes0)),
1304+
{ok, Q0} = rpc:call(Server0, rabbit_amqqueue, lookup, [QQ, <<"/">>]),
1305+
#{nodes := Nodes0} = amqqueue:get_type_state(Q0),
1306+
?assertEqual(3, length(Nodes0)),
13031307

1304-
rabbit_ct_broker_helpers:rpc(Config, 0, rabbit_quorum_queue,
1305-
force_shrink_member_to_current_member, [<<"/">>, QQ]),
1308+
rabbit_ct_broker_helpers:rpc(Config, 0, rabbit_quorum_queue,
1309+
force_shrink_member_to_current_member, [<<"/">>, QQ]),
13061310

1307-
wait_for_messages_ready([Server0], RaName, 3),
1311+
wait_for_messages_ready([Server0], RaName, 3),
13081312

1309-
{ok, Q1} = rpc:call(Server0, rabbit_amqqueue, lookup, [QQ, <<"/">>]),
1310-
#{nodes := Nodes1} = amqqueue:get_type_state(Q1),
1311-
?assertEqual(1, length(Nodes1)),
1313+
{ok, Q1} = rpc:call(Server0, rabbit_amqqueue, lookup, [QQ, <<"/">>]),
1314+
#{nodes := Nodes1} = amqqueue:get_type_state(Q1),
1315+
?assertEqual(1, length(Nodes1)),
13121316

1313-
%% grow queues back to all nodes
1314-
[rpc:call(Server0, rabbit_quorum_queue, grow, [S, <<"/">>, <<".*">>, all]) || S <- [Server1, Server2]],
1317+
%% grow queues back to all nodes
1318+
[rpc:call(Server0, rabbit_quorum_queue, grow, [S, <<"/">>, <<".*">>, all]) || S <- [Server1, Server2]],
13151319

1316-
wait_for_messages_ready([Server0], RaName, 3),
1317-
{ok, Q2} = rpc:call(Server0, rabbit_amqqueue, lookup, [QQ, <<"/">>]),
1318-
#{nodes := Nodes2} = amqqueue:get_type_state(Q2),
1319-
?assertEqual(3, length(Nodes2)).
1320+
wait_for_messages_ready([Server0], RaName, 3),
1321+
{ok, Q2} = rpc:call(Server0, rabbit_amqqueue, lookup, [QQ, <<"/">>]),
1322+
#{nodes := Nodes2} = amqqueue:get_type_state(Q2),
1323+
?assertEqual(3, length(Nodes2))
1324+
end.
13201325

13211326
force_all_queues_shrink_member_to_current_member(Config) ->
13221327
[Server0, Server1, Server2] =

0 commit comments

Comments
 (0)