From: Adam C. Emerson <aemerson@redhat.com>
Date: Thu, 5 Oct 2017 22:34:06 +0000 (-0400)
Subject: log: Add log_coarse_timestamps option
X-Git-Tag: v13.0.1~638^2~1
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=0f8606b4e082c87f99b493131b3a8d5c68b1d1d3;p=ceph-ci.git

log: Add log_coarse_timestamps option

And set up an observer in CephContext to coarsen/refine time stamps as
requested.

Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
---

diff --git a/src/common/ceph_context.cc b/src/common/ceph_context.cc
index 3a8d992c614..e82a9b2b122 100644
--- a/src/common/ceph_context.cc
+++ b/src/common/ceph_context.cc
@@ -208,6 +208,7 @@ public:
       "err_to_graylog",
       "log_graylog_host",
       "log_graylog_port",
+      "log_coarse_timestamps",
       "fsid",
       "host",
       NULL
@@ -260,6 +261,10 @@ public:
       log->graylog()->set_destination(conf->log_graylog_host, conf->log_graylog_port);
     }
 
+    if (changed.find("log_coarse_timestamps") != changed.end()) {
+      log->set_coarse_timestamps(conf->_get_val<bool>("log_coarse_timestamps"));
+    }
+
     // metadata
     if (log->graylog() && changed.count("host")) {
       log->graylog()->set_hostname(conf->host);
diff --git a/src/common/options.cc b/src/common/options.cc
index 6c916c8aec8..5b8a4da6817 100644
--- a/src/common/options.cc
+++ b/src/common/options.cc
@@ -389,6 +389,13 @@ std::vector<Option> get_global_options() {
     .set_description("port number for the remote graylog server")
     .add_see_also("log_graylog_host"),
 
+    Option("log_coarse_timestamps", Option::TYPE_BOOL, Option::LEVEL_ADVANCED)
+    .set_default(true)
+    .set_description("timestamp log entries from coarse system clock "
+		     "to improve performance")
+    .add_service("common")
+    .add_tag("performance")
+    .add_tag("service"),
 
 
     // unmodified