Skip to content

Commit

Permalink
peer_discovery_classic_config_SUITE: Increase timetrap for `successfu…
Browse files Browse the repository at this point in the history
…l_discovery_with_a_subset_of_nodes_coming_online`

[Why]
Now that feature flags compatibility is tested first, before
Mnesia-specific checks, when a peer is not started yet, the feature
flags check lasts the entire timeout, so one minute. This retry
mechanism was added to feature flags in #8411.

Thus, instead of 20 seconds, the testcase takes 10 minutes now (10
retries of one minute each).
  • Loading branch information
dumbbell committed Jun 21, 2023
1 parent 0e7c0f7 commit a989e70
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
2 changes: 1 addition & 1 deletion deps/rabbit/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -553,7 +553,7 @@ rabbitmq_suite(

rabbitmq_integration_suite(
name = "peer_discovery_classic_config_SUITE",
size = "medium",
size = "large",
flaky = True,
)

Expand Down
3 changes: 3 additions & 0 deletions deps/rabbit/test/peer_discovery_classic_config_SUITE.erl
Original file line number Diff line number Diff line change
Expand Up @@ -147,6 +147,9 @@ successful_discovery(Config) ->
cluster_members_online(Config, 1)},
?TIMEOUT).

successful_discovery_with_a_subset_of_nodes_coming_online() ->
[{timetrap, {minutes, 15}}].

successful_discovery_with_a_subset_of_nodes_coming_online(Config) ->
?awaitMatch(
{M1, M2} when length(M1) =:= 2; length(M2) =:= 2,
Expand Down

0 comments on commit a989e70

Please sign in to comment.