Project

General

Profile

Actions

Bug #1405

closed

cephfs: shouldn't have to specify all layout options

Added by Greg Farnum over 12 years ago. Updated over 7 years ago.

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

0%

Source:
Tags:
Backport:
Regression:
Severity:
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
Labels (FS):
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

Right now, you need to specify all layout options in cephfs (of the stripe unit, stripe count, and block size, anyway). I'm pretty sure this is unintentional -- it at least makes sense to change the stripe unit and count without changing the block size (ie, to actually turn on striping in the first place).

At least figure out why it's set up that way, and if appropriate, (1) change it, (2) add tests for it to file_layout.sh.

(This may actually be a kclient issue, too.)

Actions #1

Updated by Greg Farnum over 12 years ago

And you also need to specify these even if you only want to set the pool. :(

Actions #2

Updated by Greg Farnum over 12 years ago

  • Assignee set to Greg Farnum

This is going to be related to #1446, obviously. I'll take both.

Actions #3

Updated by Greg Farnum over 12 years ago

The in-kernel code rejects any layout that doesn't set the stripe unit (and if you set the object_size it makes sure they'll match up when striping). Which probably explains the EINVALs.

Actions #4

Updated by Greg Farnum over 12 years ago

  • Status changed from New to Resolved

Fixed in userspace commit:b8267492551f1adc5e0079a670b20f6180de18f0

and kernel client commit:7c296cadd05d28329e595b7f15f3be091c6cf31b

Actions #5

Updated by John Spray over 7 years ago

  • Project changed from Ceph to CephFS
  • Category deleted (1)
  • Target version deleted (v0.35)

Bulk updating project=ceph category=mds bugs so that I can remove the MDS category from the Ceph project to avoid confusion.

Actions

Also available in: Atom PDF