Project

General

Profile

Actions
Copy link

Bug #46267

closed

unittest_lockdep failure

Added by Vladimir Bashkirtsev almost 4 years ago. Updated almost 4 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Kefu Chai
Category:
qa
Target version:
v15.2.4
% Done:

0%

Source:
Community (user)
Tags:
Backport:
octopus
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
v15.0.0, v15.2.1, v15.2.2, v15.2.3
ceph-qa-suite:
Pull request ID:
35856
Crash signature (v1):
Crash signature (v2):

Description

Apparently during refactoring to use gtest unittest_lockdep was broken. In file src/test/common/test_lockdep.cc I can see

#ifdef CEPH_DEBUG_MUTEX

which now causes test to skip if CEPH_DEBUG_MUTEX is set. However it should be otherwise: if CEPH_DEBUG_MUTEX is NOT set skip the test. And in Ceph 14 this statement reads:

#ifndef CEPH_DEBUG_MUTEX

Adding missing "n" fixes the test and gets expected behaviour.

Related issues 1 (0 open — 1 closed)

Copied to Ceph - Backport #46307: octopus: unittest_lockdep failureResolvedNathan CutlerActions
Actions
Copy link
 #1

Updated by Nathan Cutler almost 4 years ago

  • Assignee set to Kefu Chai

Looks like a regression caused by 2dc50b5f91476cf10429dbcdd52b00e32f599d5c

Kefu?

Actions
Copy link
 #2

Updated by Kefu Chai almost 4 years ago

  • Status changed from New to Fix Under Review
  • Backport set to octopus, nautilus
  • Pull request ID set to 35856
Actions
Copy link
 #3

Updated by Kefu Chai almost 4 years ago

  • Backport changed from octopus, nautilus to octopus
Actions
Copy link
 #4

Updated by Kefu Chai almost 4 years ago

  • Status changed from Fix Under Review to Pending Backport
Actions
Copy link
 #5

Updated by Nathan Cutler almost 4 years ago

Actions
Copy link
 #6

Updated by Nathan Cutler almost 4 years ago

  • Status changed from Pending Backport to Resolved

While running with --resolve-parent, the script "backport-create-issue" noticed that all backports of this issue are in status "Resolved" or "Rejected".

Actions
Copy link

Also available in: Atom PDF