Project

General

Profile

Bug #23504

tests: unittest_pglog timeout

Added by Nathan Cutler 12 months ago. Updated 12 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
03/29/2018
Due date:
% Done:

0%

Source:
Tags:
Backport:
luminous, jewel
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:

Description

The ASSERT_DEATH test in unittest_pglog (at the very end of merge_log test) seems to cause transient test failure due to timeout. We already removed the EXPECT_DEATH tests in http://tracker.ceph.com/issues/18030 so I'm guessing the solution in this case is similar (remove the test).

----------------------------------------------------------
[==========] Running 42 tests from 4 test cases.
[----------] Global test environment set-up.
[----------] 23 tests from PGLogTest
[ RUN      ] PGLogTest.rewind_divergent_log
[       OK ] PGLogTest.rewind_divergent_log (1 ms)
[ RUN      ] PGLogTest.merge_old_entry
[       OK ] PGLogTest.merge_old_entry (0 ms)
[ RUN      ] PGLogTest.merge_log

[WARNING] /opt/j/ws/mkck/src/googletest/googletest/src/gtest-death-test.cc:836:: Death tests use fork(), which is unsafe particularly in a threaded context. For this test, Google Test detected 3 threads.

Related issues

Related to Ceph - Bug #18030: tests: unittest_pglog timeout Resolved 11/24/2016
Copied to Ceph - Backport #23522: luminous: tests: unittest_pglog timeout Resolved
Copied to Ceph - Backport #23523: jewel: tests: unittest_pglog timeout Resolved

History

#1 Updated by Nathan Cutler 12 months ago

  • Description updated (diff)
  • Status changed from New to In Progress

#2 Updated by Nathan Cutler 12 months ago

  • Related to Bug #18030: tests: unittest_pglog timeout added

#3 Updated by Nathan Cutler 12 months ago

  • Status changed from In Progress to Need Review

#4 Updated by Nathan Cutler 12 months ago

  • Status changed from Need Review to Pending Backport

#5 Updated by Nathan Cutler 12 months ago

#6 Updated by Nathan Cutler 12 months ago

#7 Updated by Nathan Cutler 12 months ago

  • Status changed from Pending Backport to Resolved

Also available in: Atom PDF