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

mds: add perf counters for MDLog replay and SessionMap #10539

Merged
merged 2 commits into from Aug 5, 2016

Conversation

jcsp
Copy link
Contributor

@jcsp jcsp commented Aug 2, 2016

No description provided.

@jcsp jcsp added feature cephfs Ceph File System labels Aug 2, 2016
@@ -75,6 +75,8 @@ void MDLog::create_logger()
plb.add_u64(l_mdl_rdpos, "rdpos", "Journaler read position");
plb.add_u64(l_mdl_jlat, "jlat", "Journaler flush latency");

plb.add_u64_counter(l_mdl_replay, "replay", "Events replayed");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure "replay" is descriptive enough; why not "events_replayed" or "replayed" (past tense) or something?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changed to "replayed"

@gregsfortytwo
Copy link
Member

👍 otherwise

@jcsp jcsp changed the title MDS: addition perf counters for MDLog replay and SessionMap mds: add perf counters for MDLog replay and SessionMap Aug 2, 2016
John Spray added 2 commits August 5, 2016 12:22
So that we can see how many sessions exist via
the counters rather than only by looking at status.

Signed-off-by: John Spray <john.spray@redhat.com>
Useful for watching the progress of a daemon
in standby-replay or replay.

Signed-off-by: John Spray <john.spray@redhat.com>
@jcsp jcsp merged commit e688f5f into ceph:master Aug 5, 2016
@jcsp jcsp deleted the wip-mds-counters branch August 5, 2016 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cephfs Ceph File System feature
Projects
None yet
2 participants