diff options
| author | Greg Farnum <greg@inktank.com> | 2013-02-07 10:25:33 -0800 |
|---|---|---|
| committer | Greg Farnum <greg@inktank.com> | 2013-02-07 10:25:33 -0800 |
| commit | 943235351953c773988f3a44e20e632c7e796c56 (patch) | |
| tree | 5396ab0643e7512a080d1af9e7bfc709164a3691 | |
| parent | 75c40fac603a3d21407d326e9faa8883166ad035 (diff) | |
| download | ceph-943235351953c773988f3a44e20e632c7e796c56.tar.gz | |
mds: rename mds_traceless_replies to mds_inject_traceless_reply_probability
Sage pointed out we should try for consistent naming on these debug
options, and this option is like our other inject_[x] stuff.
While we're here, make the config option comment a bit clearer.
08b82b3ef6b43283e35fd4e56eb5c78651345bea
| -rw-r--r-- | src/common/config_opts.h | 5 | ||||
| -rw-r--r-- | src/mds/Server.cc | 4 |
2 files changed, 5 insertions, 4 deletions
diff --git a/src/common/config_opts.h b/src/common/config_opts.h index fcd2f8f8540..303254e1eb3 100644 --- a/src/common/config_opts.h +++ b/src/common/config_opts.h @@ -276,8 +276,9 @@ OPTION(mds_kill_export_at, OPT_INT, 0) OPTION(mds_kill_import_at, OPT_INT, 0) OPTION(mds_kill_link_at, OPT_INT, 0) OPTION(mds_kill_rename_at, OPT_INT, 0) -OPTION(mds_traceless_replies, OPT_DOUBLE, 0) /* [0-1] amount of time MDS - should skip sending the client a trace */ +OPTION(mds_inject_traceless_reply_probability, OPT_DOUBLE, 0) /* percentage + of MDS modify replies to skip sending the + client a trace on [0-1]*/ OPTION(mds_wipe_sessions, OPT_BOOL, 0) OPTION(mds_wipe_ino_prealloc, OPT_BOOL, 0) OPTION(mds_skip_ino, OPT_INT, 0) diff --git a/src/mds/Server.cc b/src/mds/Server.cc index 8348be7a68a..2129dac6084 100644 --- a/src/mds/Server.cc +++ b/src/mds/Server.cc @@ -980,8 +980,8 @@ void Server::set_trace_dist(Session *session, MClientReply *reply, bool modified) { // skip doing this for debugging purposes? - if (modified && g_conf->mds_traceless_replies && - (rand() % 10000 < g_conf->mds_traceless_replies * 10000.0)) { + if (modified && g_conf->mds_inject_traceless_reply_probability && + (rand() % 10000 < g_conf->mds_inject_traceless_reply_probability * 10000.0)) { dout(5) << "deliberately skipping trace for " << *reply << dendl; return; } |