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.

Sign up for GitHub

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

Jump to bottom

luminous: cephfs: osdc: objecter ops output does not have useful time information #33294

Merged
merged 3 commits into from Jul 10, 2020
Merged

luminous: cephfs: osdc: objecter ops output does not have useful time information #33294

merged 3 commits into from Jul 10, 2020

Conversation

batrick
Copy link
Member

@batrick batrick commented Feb 14, 2020

@varshar16 @batrick
common/ceph_time: Use fixed floating-point notation for mono_clock
37b9d13 
Fixes: https://tracker.ceph.com/issues/40821
Signed-off-by: Varsha Rao <varao@redhat.com>
(cherry picked from commit 9c57c9f)
@batrick batrick added cephfs Ceph File System needs-qa labels Feb 14, 2020
@batrick batrick added this to the luminous milestone Feb 14, 2020
@smithfarm
Copy link
Contributor

fails to build from source?

@mattbenjamin
Copy link
Contributor

@batrick doesnt build on ceph-3.3-rhel-patches, either

@varshar16 @batrick
osdc/Objecter: Add age to the ops
32ab879 
Fixes: https://tracker.ceph.com/issues/40821
Signed-off-by: Varsha Rao <varao@redhat.com>
(cherry picked from commit ed7ebb1)

Conflicts:
        src/osdc/Objecter.cc: changed coarse_mono_clock to mono_clock to
                              make types compatible.
@varshar16 @batrick
client/MetaRequest: Add age to MetaRequest dump
7f75c71 
Fixes: https://tracker.ceph.com/issues/40821
Signed-off-by: Varsha Rao <varao@redhat.com>
(cherry picked from commit 7af0243)
@smithfarm smithfarm changed the title luminous: osdc: objecter ops output does not have useful time information luminous: cephfs: osdc: objecter ops output does not have useful time information Jun 8, 2020
@smithfarm smithfarm requested a review from varshar16 June 8, 2020 20:20
@sidharthanup sidharthanup merged commit 420be68 into ceph:luminous Jul 10, 2020
@smithfarm
Copy link
Contributor

@sidharthanup Did you do QA run(s) on this PR? If so, could you please post links to those?

@sidharthanup
Copy link

Ack
multimds: http://pulpito.ceph.com/sidharthanup-2020-06-04_14:11:16-multimds-wip-sidharth-test5-luminous-distro-basic-smithi/
fs: http://pulpito.ceph.com/sidharthanup-2020-06-04_14:44:27-fs-wip-sidharth-test5-luminous-distro-basic-smithi/
kcephfs: http://pulpito.ceph.com/sidharthanup-2020-06-02_14:46:52-kcephfs-wip-sidharth-test5-luminous-distro-basic-smithi/

@batrick batrick deleted the i42160 branch July 16, 2020 02:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@varshar16 varshar16 varshar16 approved these changes

@ukernel ukernel Awaiting requested review from ukernel

@adamemerson adamemerson Awaiting requested review from adamemerson

Assignees
No one assigned
Labels
cephfs Ceph File System luminous-batch-1 needs-qa
Projects
None yet
Milestone
luminous
5 participants
@batrick @smithfarm @mattbenjamin @sidharthanup @varshar16