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

mimic: cephfs: client: ceph.dir.rctime xattr value incorrectly prefixes 09 to the nanoseconds component #28501

Merged
merged 1 commit into from Jul 17, 2019

Conversation

@smithfarm
Copy link
Contributor

smithfarm commented Jun 11, 2019

The vxattr value incorrectly places a "09" prefix to the nanoseconds
field, instead of providing it as a zero-pad width specifier after '%'.

Fixes: https://tracker.ceph.com/issues/39943
Signed-off-by: David Disseldorp <ddiss@suse.de>
(cherry picked from commit 8980378)
@smithfarm smithfarm self-assigned this Jun 11, 2019
@smithfarm smithfarm added this to the mimic milestone Jun 11, 2019
@smithfarm smithfarm requested review from batrick and ddiss Jun 11, 2019
@smithfarm smithfarm added cephfs and removed core labels Jun 11, 2019
@smithfarm smithfarm changed the title mimic: client: ceph.dir.rctime xattr value incorrectly prefixes 09 to the nanoseconds component mimic: cephfs: client: ceph.dir.rctime xattr value incorrectly prefixes 09 to the nanoseconds component Jun 11, 2019
@ddiss
ddiss approved these changes Jun 11, 2019
@yuriw

This comment has been minimized.

Copy link
Contributor

yuriw commented Jul 9, 2019

Copy link
Member

batrick left a comment

Noting for posterity this backport omits 0293598 because https://tracker.ceph.com/issues/38838 is not backported to mimic.

@yuriw yuriw merged commit dfa6339 into ceph:mimic Jul 17, 2019
4 checks passed
4 checks passed
Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
@smithfarm smithfarm deleted the smithfarm:wip-40168-mimic branch Jul 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.