-
Notifications
You must be signed in to change notification settings - Fork 552
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
Added timeout to gtest_raft_rpunit
#16637
Added timeout to gtest_raft_rpunit
#16637
Conversation
/dt |
ducktape was retried in https://buildkite.com/redpanda/redpanda/builds/45124#018dc21f-bbec-4a3f-a6bf-0a8c1e2591f0 |
Signed-off-by: Michal Maslanka <michal@redpanda.com>
7c69966
to
50b4382
Compare
Added timeout to the test to prevent it from hanging and to be able to retrieve the logs from the test that execution stuck. Signed-off-by: Michal Maslanka <michal@redpanda.com>
50b4382
to
2e2c8b7
Compare
gtest_raft_rpunit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice idea to timeout and capture logs.
@@ -173,4 +175,7 @@ class seastar_test : public ::testing::Test { | |||
#define ASSERT_LT_CORO(val1, val2) GTEST_ASSERT_LT_CORO(val1, val2) | |||
#define ASSERT_LE_CORO(val1, val2) GTEST_ASSERT_LE_CORO(val1, val2) | |||
#define ASSERT_NE_CORO(val1, val2) GTEST_ASSERT_NE_CORO(val1, val2) | |||
|
|||
#define GTEST_SKIP_CORO() GTEST_SKIP_CORO_("") | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This macro appears to be unused in this PR. It also should have a test in test_utils/tests/gtest_tests.cc
.
Added timeout to for the
gtest_raft_rpunit
. This will allow us to retrieve logs from failed test.Backports Required
Release Notes