Project

General

Profile

Actions

Bug #6130

closed

SharedPtrRegistry_all.wait_lookup_or_create fails inconsistently

Added by Greg Farnum over 10 years ago. Updated over 10 years ago.

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

100%

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

Description

pull request

I noticed via the gitbuilders that this unit test is inconsistently failing. I can run it repeatedly on my dev box and get intermittent failures with the same binary. I assume there's a race in the test code of some kind rather than the SharedPtrRegistry actually being broken.


Related issues 1 (0 open1 closed)

Related to Ceph - Bug #6679: throttle: transient unit test failureResolvedLoïc Dachary10/29/2013

Actions
Actions #1

Updated by Loïc Dachary over 10 years ago

  • Status changed from New to 12

while : ; do ./unittest_sharedptr_registry --gtest_filter=SharedPtrRegistry_all.wait_lookup_or_create || break ; done

Note: Google Test filter = SharedPtrRegistry_all.wait_lookup_or_create
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from SharedPtrRegistry_all
[ RUN      ] SharedPtrRegistry_all.wait_lookup_or_create
test/common/test_sharedptr_registry.cc:140: Failure
Value of: t.ptr
  Actual: false
Expected: true
[  FAILED  ] SharedPtrRegistry_all.wait_lookup_or_create (1 ms)
[----------] 1 test from SharedPtrRegistry_all (1 ms total)

[----------] Global test environment tear-down
[==========] 1 test from 1 test case ran. (1 ms total)
[  PASSED  ] 0 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] SharedPtrRegistry_all.wait_lookup_or_create

 1 FAILED TEST
Actions #2

Updated by Loïc Dachary over 10 years ago

  • Status changed from 12 to In Progress
Actions #3

Updated by Loïc Dachary over 10 years ago

  • Status changed from In Progress to Fix Under Review
  • % Done changed from 0 to 90
Actions #4

Updated by Loïc Dachary over 10 years ago

  • Description updated (diff)
Actions #5

Updated by Greg Farnum over 10 years ago

Sam reviewed and merged into master, commit: 7cc2eb246df14925ca27b8dee19b32e0bdb505a8

Actions #6

Updated by Loïc Dachary over 10 years ago

  • Status changed from Fix Under Review to Resolved
  • % Done changed from 90 to 100

cool :-)

Actions

Also available in: Atom PDF