Skip to content

rabbitmq_ct_helpers: Detect crashes in RabbitMQ logs#9290

Merged
dumbbell merged 1 commit intomainfrom
search-for-crashes-in-test-nodes
Sep 5, 2023
Merged

rabbitmq_ct_helpers: Detect crashes in RabbitMQ logs#9290
dumbbell merged 1 commit intomainfrom
search-for-crashes-in-test-nodes

Conversation

@dumbbell
Copy link
Copy Markdown
Collaborator

@dumbbell dumbbell commented Sep 4, 2023

Why

Some crashes may go under the radar if a testcase didn't check everything (which is hard to do anyway). It could be useful to be aware of those crashes.

How

This new check will search for any logged crashes in the RabbitMQ log files and throw an exception as part of stopping a node.

V2: This facility can be disabled by setting find_crashes to false in the common_test's Config, or by setting the $FIND_CRASHES environment variable to anything but "1"/"yes"/"true".

@dumbbell dumbbell added this to the 3.13.0 milestone Sep 4, 2023
@dumbbell dumbbell self-assigned this Sep 4, 2023
@dumbbell dumbbell changed the title rabbit_ct_helpers: Detect crashes in RabbitMQ logs rabbitmq_ct_helpers: Detect crashes in RabbitMQ logs Sep 4, 2023
@dumbbell dumbbell marked this pull request as ready for review September 4, 2023 21:16
[Why]
Some crashes may go under the radar if a testcase didn't check
everything (which is hard to do anyway). It could be useful to be aware
of those crashes.

[How]
This new check will search for any logged crashes in the RabbitMQ log
files and throw an exception as part of stopping a node.

V2: This facility can be disabled by setting `find_crashes` to false in
    the common_test's `Config`, or by setting the `$FIND_CRASHES`
    environment variable to anything but "1"/"yes"/"true".
@dumbbell dumbbell force-pushed the search-for-crashes-in-test-nodes branch from bafc38a to cb8b0bf Compare September 5, 2023 10:50
@dumbbell dumbbell merged commit 8b79ede into main Sep 5, 2023
@dumbbell dumbbell deleted the search-for-crashes-in-test-nodes branch September 5, 2023 13:37
Comment thread deps/rabbitmq_ct_helpers/src/rabbit_ct_broker_helpers.erl
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