Project

General

Profile

Bug #14029

"[ FAILED ] LibCephFS.OpenLayout" in smoke-master-distro-basic-openstack

Added by Yuri Weinstein about 3 years ago. Updated about 3 years ago.

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

0%

Source:
Q/A
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
smoke
Component(FS):
Labels (FS):
Pull request ID:

Description

Run: http://pulpito.ovh.sepia.ceph.com:8081/teuthology-2015-12-09_05:00:02-smoke-master-distro-basic-openstack/
Job: 34716
Logs: http://teuthology.ovh.sepia.ceph.com/teuthology/teuthology-2015-12-09_05:00:02-smoke-master-distro-basic-openstack/34716/teuthology.log

2015-12-09T05:56:43.091 INFO:tasks.workunit.client.0.target081236.stdout:test/libcephfs/test.cc:195: Failure
2015-12-09T05:56:43.092 INFO:tasks.workunit.client.0.target081236.stdout:Value of: ceph_setxattr(cmount, test_layout_file, xattrk, (void *)xattrv, 5, 0)
2015-12-09T05:56:43.092 INFO:tasks.workunit.client.0.target081236.stdout:  Actual: 0
2015-12-09T05:56:43.092 INFO:tasks.workunit.client.0.target081236.stdout:Expected: -39
2015-12-09T05:56:43.092 INFO:tasks.workunit.client.0.target081236.stdout:[  FAILED  ] LibCephFS.OpenLayout (90 ms)

Associated revisions

Revision d4658301 (diff)
Added by John Spray about 3 years ago

mds: fix setvxattr (broken in a536d114)

There was an accidental move of this line
when adding the MAY_SET_POOL check, which
was causing setxattr to proceed before
it had the right locks, and thereby apply
its checks on bad data (symptom was failing
to detect that the file had data written to it).

Fixes: #14029
Signed-off-by: John Spray <>

History

#1 Updated by Greg Farnum about 3 years ago

  • Project changed from Ceph to fs

Hopefully we just got a mismatch between qa suite and ceph with regards to the new pool layout cephx flag? :/

#2 Updated by Greg Farnum about 3 years ago

ovh: teuthology-2015-12-07_23:04:02-fs-master---basic-openstack/32792/

Hrm.

#3 Updated by Greg Farnum about 3 years ago

  • Assignee set to John Spray

Can you look at this, John? We got this some more:

http://pulpito.ovh.sepia.ceph.com:8081/gregf-2015-12-11_09:34:05-fs-greg-fs-testing---basic-openstack/37464/
http://pulpito.ovh.sepia.ceph.com:8081/gregf-2015-12-11_09:34:05-fs-greg-fs-testing---basic-openstack/37502/

I presume things just didn't get updated with the new cap syntax for setting layouts or something?

#4 Updated by John Spray about 3 years ago

  • Status changed from New to In Progress

#5 Updated by John Spray about 3 years ago

  • Status changed from In Progress to Need Review

#6 Updated by Zheng Yan about 3 years ago

  • Status changed from Need Review to Resolved

Also available in: Atom PDF