Project

General

Profile

Actions

Bug #18128

closed

unittest_ceph_crypto fails ForkDeathTest.MD5 (intermitent)

Added by David Zafman over 7 years ago. Updated over 7 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
-
% Done:

0%

Source:
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

https://jenkins.ceph.com/job/ceph-pull-requests/15152/

 32/154 Test  #39: unittest_ceph_crypto ....................***Failed    0.30 sec
[==========] Running 7 tests from 3 test cases.
[----------] Global test environment set-up.
[----------] 1 test from ForkDeathTest
[ RUN      ] ForkDeathTest.MD5
/home/jenkins-build/build/workspace/ceph-pull-requests/src/test/ceph_crypto.cc:138: Failure
Death test: do_simple_crypto()
    Result: died but not with expected error.
  Expected: ^$
Actual msg:
[  DEATH   ] 2016-12-01 07:59:11.389116 7ff3ca15dd00 -1 did not load config file, using default settings.
[  DEATH   ] 2016-12-01 07:59:11.389116 7ff3ca15dd00 -1 did not load config file, using default settings.
[  DEATH   ]
[  FAILED  ] ForkDeathTest.MD5 (8 ms)
[----------] 1 test from ForkDeathTest (8 ms total)

Related issues 1 (0 open1 closed)

Has duplicate Ceph - Bug #18121: unittest_ceph_crypto fails ForkDeathTest.MD5 (intermitent)Duplicate12/02/2016

Actions
Actions #1

Updated by David Zafman over 7 years ago

  • Description updated (diff)
Actions #4

Updated by Loïc Dachary over 7 years ago

  • Has duplicate Bug #18121: unittest_ceph_crypto fails ForkDeathTest.MD5 (intermitent) added
Actions #5

Updated by Loïc Dachary over 7 years ago

  • Subject changed from unittest_ceph_crypto failed on Jenkins to unittest_ceph_crypto fails ForkDeathTest.MD5 (intermitent)
Actions #6

Updated by Loïc Dachary over 7 years ago

  • Status changed from New to 12
  • Assignee set to Kefu Chai

@Kefu Chai could these intermittent failures be a side effect of the following ? Maybe not directly but it could have amplified an undelying problem ? Wild guess at this point ;-)

commit 5af29540675b674c1985ff98b28a783ed124acf6
Author: Kefu Chai <kchai@redhat.com>
Date:   Fri Nov 25 02:16:40 2016 +0800

    test,cmake: turn unit.h into unit.cc to speed up compilation

    Signed-off-by: Kefu Chai <kchai@redhat.com>

diff --git a/src/test/ceph_crypto.cc b/src/test/ceph_crypto.cc
index 893f332..4782601 100644
--- a/src/test/ceph_crypto.cc
+++ b/src/test/ceph_crypto.cc
@@ -1,6 +1,6 @@
+#include "gtest/gtest.h" 
 #include "common/ceph_crypto.h" 
-
-#include "test/unit.h" 
+#include "global/global_context.h" 

 class CryptoEnvironment: public ::testing::Environment {
 public:

Actions #7

Updated by Loïc Dachary over 7 years ago

It can easily be repeated with

while : ; do ctest -R unittest_ceph_crypto -V >& out || break ; done

Actions #8

Updated by Kefu Chai over 7 years ago

  • Status changed from 12 to Fix Under Review
Actions #9

Updated by Loïc Dachary over 7 years ago

  • Status changed from Fix Under Review to Resolved
Actions

Also available in: Atom PDF