Skip to content

Fix test case quorum_unaffected_after_vhost_failure#15215

Merged
ansd merged 1 commit intomainfrom
da-quorum_unaffected_after_vhost_failure
Jan 6, 2026
Merged

Fix test case quorum_unaffected_after_vhost_failure#15215
ansd merged 1 commit intomainfrom
da-quorum_unaffected_after_vhost_failure

Conversation

@ansd
Copy link
Copy Markdown
Member

@ansd ansd commented Jan 6, 2026

Make the vhost actually fail in test case
quorum_unaffected_after_vhost_failure.

Previously, the logs didn't indicate any vhost failure. Atom kill showed various errors in the logs providing evidence that the vhost actually fails.

@ansd
Fix test case quorum_unaffected_after_vhost_failure
28f5ba3 
Make the vhost actually fail in test case
`quorum_unaffected_after_vhost_failure`.

Previously, the logs didn't indicate any vhost failure.
Atom `kill` showed various errors in the logs providing evidence that
the vhost actually fails.
@ansd ansd changed the title Fix test case quorum_unaffected_after_vhost_failure Fix test case quorum_unaffected_after_vhost_failure Jan 6, 2026
@ansd ansd marked this pull request as ready for review January 6, 2026 14:24
@ansd
Copy link
Copy Markdown
Member Author

ansd commented Jan 6, 2026

Same unrelated failing test as in #15206 (comment)

@ansd ansd merged commit 8def0a0 into main Jan 6, 2026
844 of 863 checks passed
@ansd ansd deleted the da-quorum_unaffected_after_vhost_failure branch January 6, 2026 15:47
@mergify mergify bot mentioned this pull request Jan 6, 2026
Merged
@ansd
Copy link
Copy Markdown
Member Author

ansd commented Jan 6, 2026

@Ayanda-D @michaelklishin test case force_shrink_member_to_current_member is flaky and fails in this PR as well as in #15206 (comment) and #15208 (comment).

The following assertion fails:

*** CT Error Notification 2026-01-06 15:29:33.318 ***🔗
queue_utils:'-assert_number_of_replicas/5-AwaitMatchFilter/1-0-' failed on line 214
Reason: {awaitMatch,[{module,...},{...}|...]}

Full error description and stacktrace



=== Ended at 2026-01-06 15:29:33
=== Location: [{queue_utils,'-assert_number_of_replicas/5-AwaitMatchFilter/1-0-',214},
              {test_server,ts_tc,1796},
              {test_server,run_test_case_eval1,1305},
              {test_server,run_test_case_eval,1237}]
=== === Reason: {awaitMatch,
                     [{module,queue_utils},
                      {line,214},
                      {expression,
                          "begin { ok , Q } = rabbit_ct_broker_helpers : rpc ( Config , Server , rabbit_amqqueue , lookup , [ QQ , VHost ] ) , # { nodes := Nodes } = amqqueue : get_type_state ( Q ) , length ( Nodes ) end"},
                      {pattern,"Count"},
                      {value,2},
                      {timeout,30000},
                      {polling_interval,50}]}
  in function  queue_utils:'-assert_number_of_replicas/5-AwaitMatchFilter/1-0-'/6 (src/queue_utils.erl:214)
  in call from test_server:ts_tc/3 (test_server.erl:1796)
  in call from test_server:run_test_case_eval1/6 (test_server.erl:1305)
  in call from test_server:run_test_case_eval/9 (test_server.erl:1237)

Can you please take a look? The test case got added in #12468

michaelklishin added a commit that referenced this pull request Jan 6, 2026
@michaelklishin
Fix a test #12468 #15215
6b8fcd6
michaelklishin added a commit that referenced this pull request Jan 6, 2026
@michaelklishin
Fix a test #12468 #15215
f9c98a0 
(cherry picked from commit 6b8fcd6)
michaelklishin added a commit that referenced this pull request Feb 24, 2026
@michaelklishin
Fix a test #12468 #15215
527500d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

backport-v4.2.x

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@ansd