Skip to content

clustering_recovery_SUITE: Add recover_after_partition_with_leader testcase #14384

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 14, 2025

Conversation

dumbbell
Copy link
Collaborator

Why

The testcase tries to replicate the steps described in issue #12934.

How

It uses intermediate Erlang nodes between the common_test control node and the RabbitMQ nodes, using peer standard_io communication. The goal is to make sure the common_test control node doesn't interfere with the nodes the RabbitMQ nodes can see, despite the blocking of the Erlang distribution connection.

So far, I couldn't reproduce the problem reported in #12934. @mkuratczyk couldn't either, so it might have been fixed as a side effect of another change...

References #12934.

…testcase

[Why]
The testcase tries to replicate the steps described in issue #12934.

[How]
It uses intermediate Erlang nodes between the common_test control node
and the RabbitMQ nodes, using `peer` standard_io communication. The goal
is to make sure the common_test control node doesn't interfere with the
nodes the RabbitMQ nodes can see, despite the blocking of the Erlang
distribution connection.

So far, I couldn't reproduce the problem reported in #12934. @mkuratczyk
couldn't either, so it might have been fixed as a side effect of another
change...

References #12934.
@dumbbell dumbbell marked this pull request as ready for review August 14, 2025 12:19
@dumbbell dumbbell merged commit e329881 into main Aug 14, 2025
560 of 561 checks passed
@dumbbell dumbbell deleted the test-issue-12934 branch August 14, 2025 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants