Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tools/cephfs: set dir_layout when injecting inodes #14234

Merged
merged 1 commit into from Apr 13, 2017

Conversation

Projects
None yet
2 participants
@jcsp
Copy link
Contributor

jcsp commented Mar 29, 2017

When we left this as zero, the MDS would interpret it was HASH_LINUX
rather than the default HASH_RJENKINS. Potentially that
could cause problems if there perhaps were already dirfrags in
the metadata pool that were set up using rjenkins. Mainly
it just seems more appropriate to explicitly set this field
rather than hit the fallback behaviour.

Related: http://tracker.ceph.com/issues/19406
Signed-off-by: John Spray john.spray@redhat.com

@jcsp jcsp requested a review from ukernel Mar 29, 2017

@jcsp jcsp force-pushed the jcsp:wip-19406-dirhash branch from b3302f6 to 5b9a22d Apr 4, 2017

@jcsp

This comment has been minimized.

Copy link
Contributor Author

jcsp commented Apr 12, 2017

retest this please

John Spray
tools/cephfs: set dir_layout when injecting inodes
When we left this as zero, the MDS would interpret it was HASH_LINUX
rather than the default HASH_RJENKINS.  Potentially that
could cause problems if there perhaps were already dirfrags in
the metadata pool that were set up using rjenkins.  Mainly
it just seems more appropriate to explicitly set this field
rather than hit the fallback behaviour.

Related: http://tracker.ceph.com/issues/19406
Signed-off-by: John Spray <john.spray@redhat.com>

@jcsp jcsp force-pushed the jcsp:wip-19406-dirhash branch from 5b9a22d to 7d6d542 Apr 12, 2017

@jcsp jcsp merged commit b81e4b7 into ceph:master Apr 13, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@jcsp jcsp deleted the jcsp:wip-19406-dirhash branch Apr 13, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.