Project

General

Profile

Bug #22221

qa: src/test/libcephfs/test.cc:376: Expected: (len) > (0), actual: -34 vs 0

Added by Patrick Donnelly about 1 year ago. Updated 9 months ago.

Status:
Resolved
Priority:
Normal
Category:
Testing
Target version:
-
Start date:
11/22/2017
Due date:
% Done:

0%

Source:
Development
Tags:
Backport:
luminous,jewel
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
Labels (FS):
Pull request ID:

Description

2017-11-21T23:35:00.610 INFO:tasks.workunit.client.0.smithi185.stdout:[ RUN      ] LibCephFS.DirLs
2017-11-21T23:35:00.633 INFO:tasks.workunit.client.0.smithi185.stdout:rand: 0
2017-11-21T23:35:00.633 INFO:tasks.workunit.client.0.smithi185.stdout:/build/ceph-13.0.0-3427-g52ad5af/src/test/libcephfs/test.cc:376: Failure
2017-11-21T23:35:00.633 INFO:tasks.workunit.client.0.smithi185.stdout:Expected: (len) > (0), actual: -34 vs 0
2017-11-21T23:35:00.634 INFO:tasks.workunit.client.0.smithi185.stdout:[  FAILED  ] LibCephFS.DirLs (23 ms)

Problem: https://github.com/ceph/ceph/blob/eba596dfc87176b2236b77f39a0ee3c80f37ba37/src/test/libcephfs/test.cc#L373

If the randvar r is 0, then the buffer length told to ceph_getdents is 0. The buffer is sized properly here:

https://github.com/ceph/ceph/blob/eba596dfc87176b2236b77f39a0ee3c80f37ba37/src/test/libcephfs/test.cc#L368

but we didn't pass the right length.


Related issues

Copied to fs - Backport #22383: luminous: qa: src/test/libcephfs/test.cc:376: Expected: (len) > (0), actual: -34 vs 0 Resolved
Copied to fs - Backport #22384: jewel: qa: src/test/libcephfs/test.cc:376: Expected: (len) > (0), actual: -34 vs 0 Resolved

History

#1 Updated by Patrick Donnelly about 1 year ago

  • Status changed from New to Need Review

#2 Updated by Patrick Donnelly about 1 year ago

  • Status changed from Need Review to Pending Backport
  • Backport changed from luminous to luminous,jewel

#3 Updated by Nathan Cutler about 1 year ago

  • Copied to Backport #22383: luminous: qa: src/test/libcephfs/test.cc:376: Expected: (len) > (0), actual: -34 vs 0 added

#4 Updated by Nathan Cutler about 1 year ago

  • Copied to Backport #22384: jewel: qa: src/test/libcephfs/test.cc:376: Expected: (len) > (0), actual: -34 vs 0 added

#5 Updated by Patrick Donnelly 9 months ago

  • Status changed from Pending Backport to Resolved

Also available in: Atom PDF