Project

General

Profile

Bug #14955

unittest_bufferlist.sh failing

Added by Greg Farnum almost 3 years ago. Updated almost 3 years ago.

Status:
Resolved
Priority:
Immediate
Assignee:
Category:
common
Target version:
-
Start date:
03/03/2016
Due date:
% Done:

0%

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

Description

Like it says, see eg http://gitbuilder.sepia.ceph.com/gitbuilder-ceph-tarball-trusty-amd64-basic/#origin/master

It hasn't bisected down yet but the failure is pretty new and is showing up as failures of newly-rebased PRs on GitHub.

Associated revisions

Revision 1335871f (diff)
Added by Samuel Just almost 3 years ago

buffer: increment history alloc as well in raw_combined

Introduced in 73dcd26fc8798e09d93520c583d63edb28ffc631

Fixes: #14955
Signed-off-by: Samuel Just <>

History

#1 Updated by Samuel Just almost 3 years ago

  • Assignee set to Samuel Just

#2 Updated by Samuel Just almost 3 years ago

Running main() from gmock_main.cc
[==========] Running 94 tests from 7 test cases.
[----------] Global test environment set-up.
[----------] 2 tests from Buffer
[ RUN ] Buffer.constructors
/home/sam/git-checkouts/ceph/src/test/bufferlist.cc:61: Failure
Value of: buffer::get_history_alloc_bytes()
Actual: 0
Expected: history_alloc_bytes
Which is: 17
/home/sam/git-checkouts/ceph/src/test/bufferlist.cc:62: Failure
Value of: buffer::get_history_alloc_num()
Actual: 0
Expected: history_alloc_num
Which is: 1
/home/sam/git-checkouts/ceph/src/test/bufferlist.cc:76: Failure
Value of: buffer::get_history_alloc_bytes()
Actual: 0
Expected: history_alloc_bytes
Which is: 17
/home/sam/git-checkouts/ceph/src/test/bufferlist.cc:77: Failure
Value of: buffer::get_history_alloc_num()
Actual: 0
Expected: history_alloc_num
Which is: 1
/home/sam/git-checkouts/ceph/src/test/bufferlist.cc:96: Failure
Value of: buffer::get_history_alloc_bytes()
Actual: 17
Expected: history_alloc_bytes
Which is: 34
/home/sam/git-checkouts/ceph/src/test/bufferlist.cc:97: Failure
Value of: buffer::get_history_alloc_num()
Actual: 1
Expected: history_alloc_num
Which is: 2
/home/sam/git-checkouts/ceph/src/test/bufferlist.cc:114: Failure
Value of: buffer::get_history_alloc_bytes()
Actual: 34
Expected: history_alloc_bytes
Which is: 51
/home/sam/git-checkouts/ceph/src/test/bufferlist.cc:115: Failure
Value of: buffer::get_history_alloc_num()
Actual: 2
Expected: history_alloc_num
Which is: 3
/home/sam/git-checkouts/ceph/src/test/bufferlist.cc:132: Failure
Value of: buffer::get_history_alloc_bytes()
Actual: 34
Expected: history_alloc_bytes
Which is: 51
/home/sam/git-checkouts/ceph/src/test/bufferlist.cc:133: Failure
Value of: buffer::get_history_alloc_num()
Actual: 2
Expected: history_alloc_num
Which is: 3
/home/sam/git-checkouts/ceph/src/test/bufferlist.cc:154: Failure
Value of: buffer::get_history_alloc_bytes()
Actual: 51
Expected: history_alloc_bytes
Which is: 85
/home/sam/git-checkouts/ceph/src/test/bufferlist.cc:155: Failure
Value of: buffer::get_history_alloc_num()
Actual: 3
Expected: history_alloc_num
Which is: 5
/home/sam/git-checkouts/ceph/src/test/bufferlist.cc:172: Failure
Value of: buffer::get_history_alloc_bytes()
Actual: 68
Expected: history_alloc_bytes
Which is: 102
/home/sam/git-checkouts/ceph/src/test/bufferlist.cc:173: Failure
Value of: buffer::get_history_alloc_num()
Actual: 4
Expected: history_alloc_num
Which is: 6
/home/sam/git-checkouts/ceph/src/test/bufferlist.cc:185: Failure
Value of: buffer::get_history_alloc_bytes()
Actual: 85
Expected: history_alloc_bytes
Which is: 119
/home/sam/git-checkouts/ceph/src/test/bufferlist.cc:186: Failure
Value of: buffer::get_history_alloc_num()
Actual: 5
Expected: history_alloc_num
Which is: 7
[ FAILED ] Buffer.constructors (2 ms)

#3 Updated by Samuel Just almost 3 years ago

Odd, it passes if I run the test directly.

#4 Updated by Samuel Just almost 3 years ago

ah CEPH_BUFFER_TRACK=true

bisect running

#5 Updated by Samuel Just almost 3 years ago

73dcd26fc8798e09d93520c583d63edb28ffc631

#6 Updated by Samuel Just almost 3 years ago

  • Status changed from New to Need Review

#7 Updated by Samuel Just almost 3 years ago

  • Status changed from Need Review to Resolved

Also available in: Atom PDF