Project

General

Profile

Bug #18128

unittest_ceph_crypto fails ForkDeathTest.MD5 (intermitent)

Added by David Zafman 10 months ago. Updated 10 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
12/02/2016
Due date:
% Done:

0%

Source:
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Release:
Needs Doc:
No

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

Duplicated by Ceph - Bug #18121: unittest_ceph_crypto fails ForkDeathTest.MD5 (intermitent) Duplicate 12/02/2016

History

#1 Updated by David Zafman 10 months ago

  • Description updated (diff)

#4 Updated by Loic Dachary 10 months ago

  • Duplicated by Bug #18121: unittest_ceph_crypto fails ForkDeathTest.MD5 (intermitent) added

#5 Updated by Loic Dachary 10 months ago

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

#6 Updated by Loic Dachary 10 months ago

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

@Kefu 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:

#7 Updated by Loic Dachary 10 months ago

It can easily be repeated with

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

#8 Updated by Kefu Chai 10 months ago

  • Status changed from Verified to Need Review

#9 Updated by Loic Dachary 10 months ago

  • Status changed from Need Review to Resolved

Also available in: Atom PDF