Skip to content

Commit 5cc73d8

Browse files
committed
[skip ci] Disable force_shrink_member_to_current_member in mixed-version
1 parent 3033154 commit 5cc73d8

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
@@ -1217,38 +1217,43 @@ single_active_consumer_priority(Config) ->
12171217
ok.
12181218

12191219
force_shrink_member_to_current_member(Config) ->
1220-
[Server0, Server1, Server2] =
1221-
rabbit_ct_broker_helpers:get_node_configs(Config, nodename),
1220+
case rabbit_ct_helpers:is_mixed_versions() of
1221+
true ->
1222+
{skip, "Should not run in mixed version environments"};
1223+
_ ->
1224+
[Server0, Server1, Server2] =
1225+
rabbit_ct_broker_helpers:get_node_configs(Config, nodename),
12221226

1223-
Ch = rabbit_ct_client_helpers:open_channel(Config, Server0),
1224-
QQ = ?config(queue_name, Config),
1225-
?assertEqual({'queue.declare_ok', QQ, 0, 0},
1226-
declare(Ch, QQ, [{<<"x-queue-type">>, longstr, <<"quorum">>}])),
1227+
Ch = rabbit_ct_client_helpers:open_channel(Config, Server0),
1228+
QQ = ?config(queue_name, Config),
1229+
?assertEqual({'queue.declare_ok', QQ, 0, 0},
1230+
declare(Ch, QQ, [{<<"x-queue-type">>, longstr, <<"quorum">>}])),
12271231

1228-
RaName = ra_name(QQ),
1229-
rabbit_ct_client_helpers:publish(Ch, QQ, 3),
1230-
wait_for_messages_ready([Server0], RaName, 3),
1232+
RaName = ra_name(QQ),
1233+
rabbit_ct_client_helpers:publish(Ch, QQ, 3),
1234+
wait_for_messages_ready([Server0], RaName, 3),
12311235

1232-
{ok, Q0} = rpc:call(Server0, rabbit_amqqueue, lookup, [QQ, <<"/">>]),
1233-
#{nodes := Nodes0} = amqqueue:get_type_state(Q0),
1234-
?assertEqual(3, length(Nodes0)),
1236+
{ok, Q0} = rpc:call(Server0, rabbit_amqqueue, lookup, [QQ, <<"/">>]),
1237+
#{nodes := Nodes0} = amqqueue:get_type_state(Q0),
1238+
?assertEqual(3, length(Nodes0)),
12351239

1236-
rabbit_ct_broker_helpers:rpc(Config, 0, rabbit_quorum_queue,
1237-
force_shrink_member_to_current_member, [<<"/">>, QQ]),
1240+
rabbit_ct_broker_helpers:rpc(Config, 0, rabbit_quorum_queue,
1241+
force_shrink_member_to_current_member, [<<"/">>, QQ]),
12381242

1239-
wait_for_messages_ready([Server0], RaName, 3),
1243+
wait_for_messages_ready([Server0], RaName, 3),
12401244

1241-
{ok, Q1} = rpc:call(Server0, rabbit_amqqueue, lookup, [QQ, <<"/">>]),
1242-
#{nodes := Nodes1} = amqqueue:get_type_state(Q1),
1243-
?assertEqual(1, length(Nodes1)),
1245+
{ok, Q1} = rpc:call(Server0, rabbit_amqqueue, lookup, [QQ, <<"/">>]),
1246+
#{nodes := Nodes1} = amqqueue:get_type_state(Q1),
1247+
?assertEqual(1, length(Nodes1)),
12441248

1245-
%% grow queues back to all nodes
1246-
[rpc:call(Server0, rabbit_quorum_queue, grow, [S, <<"/">>, <<".*">>, all]) || S <- [Server1, Server2]],
1249+
%% grow queues back to all nodes
1250+
[rpc:call(Server0, rabbit_quorum_queue, grow, [S, <<"/">>, <<".*">>, all]) || S <- [Server1, Server2]],
12471251

1248-
wait_for_messages_ready([Server0], RaName, 3),
1249-
{ok, Q2} = rpc:call(Server0, rabbit_amqqueue, lookup, [QQ, <<"/">>]),
1250-
#{nodes := Nodes2} = amqqueue:get_type_state(Q2),
1251-
?assertEqual(3, length(Nodes2)).
1252+
wait_for_messages_ready([Server0], RaName, 3),
1253+
{ok, Q2} = rpc:call(Server0, rabbit_amqqueue, lookup, [QQ, <<"/">>]),
1254+
#{nodes := Nodes2} = amqqueue:get_type_state(Q2),
1255+
?assertEqual(3, length(Nodes2))
1256+
end.
12521257

12531258
force_all_queues_shrink_member_to_current_member(Config) ->
12541259
[Server0, Server1, Server2] =

0 commit comments

Comments
 (0)