From 7f2e67d40d1006e88fda86eb04699c15db3440ee Mon Sep 17 00:00:00 2001
From: Amar Tumballi <amarts@redhat.com>
Date: Wed, 11 Oct 2017 15:08:49 +0530
Subject: metrics: provide options to dump metrics from xlators

* Introduce xlator methods to allow dumping of metrics
* Separate options to get the metrics dumped in a path

Updates #168

Change-Id: I7df80df33b71d6f449f03c2332665b4a45f6ddf2
Signed-off-by: Amar Tumballi <amarts@redhat.com>
---
 glusterfsd/src/glusterfsd-mgmt.c | 74 ++++++++++++++++++++++++++++++++++++++++
 glusterfsd/src/glusterfsd.c      | 16 ++++++---
 2 files changed, 86 insertions(+), 4 deletions(-)

(limited to 'glusterfsd')

diff --git a/glusterfsd/src/glusterfsd-mgmt.c b/glusterfsd/src/glusterfsd-mgmt.c
index b8ebdefb876..646b4399019 100644
--- a/glusterfsd/src/glusterfsd-mgmt.c
+++ b/glusterfsd/src/glusterfsd-mgmt.c
@@ -32,6 +32,7 @@
 #include "syncop.h"
 #include "xlator.h"
 #include "syscall.h"
+#include "monitoring.h"
 
 static gf_boolean_t is_mgmt_rpc_reconnect = _gf_false;
 int need_emancipate = 0;
@@ -894,6 +895,74 @@ out:
         return ret;
 }
 
+int
+glusterfs_handle_dump_metrics (rpcsvc_request_t *req)
+{
+        int32_t                 ret             = -1;
+        gd1_mgmt_brick_op_req   xlator_req      = {0,};
+        xlator_t                *this           = NULL;
+        glusterfs_ctx_t         *ctx            = NULL;
+        char  *filepath = NULL;
+        int    fd  = -1;
+        struct stat statbuf = {0,};
+        char *msg = NULL;
+
+        GF_ASSERT (req);
+        this = THIS;
+        GF_ASSERT (this);
+
+        ret = xdr_to_generic (req->msg[0], &xlator_req,
+                             (xdrproc_t)xdr_gd1_mgmt_brick_op_req);
+
+        if (ret < 0) {
+                /*failed to decode msg;*/
+                req->rpc_err = GARBAGE_ARGS;
+                return -1;
+        }
+        ret = -1;
+        ctx = this->ctx;
+
+        /* Infra for monitoring */
+        filepath = gf_monitor_metrics (ctx);
+        if (!filepath)
+                goto out;
+
+        fd = sys_open (filepath, O_RDONLY, 0);
+        if (fd < 0)
+                goto out;
+
+        if (sys_fstat (fd, &statbuf) < 0)
+                goto out;
+
+        if (statbuf.st_size > GF_UNIT_MB) {
+                gf_msg (this->name, GF_LOG_WARNING, ENOMEM,
+                        LG_MSG_NO_MEMORY,
+                        "Allocated size exceeds expectation: "
+                        "reconsider logic (%"GF_PRI_SIZET")",
+                        statbuf.st_size);
+        }
+        msg = GF_CALLOC (1, (statbuf.st_size + 1), gf_common_mt_char);
+        if (!msg)
+                goto out;
+
+        ret = sys_read (fd, msg, statbuf.st_size);
+        if (ret < 0)
+                goto out;
+
+        /* Send all the data in errstr, instead of dictionary for now */
+        glusterfs_translator_info_response_send (req, 0, msg, NULL);
+
+        ret = 0;
+out:
+        if (fd >= 0)
+                sys_close (fd);
+
+        GF_FREE (msg);
+        GF_FREE (filepath);
+
+        return ret;
+}
+
 int
 glusterfs_handle_defrag (rpcsvc_request_t *req)
 {
@@ -1695,6 +1764,11 @@ rpcsvc_actor_t glusterfs_actors[GLUSTERD_BRICK_MAXVALUE] = {
                                           GLUSTERD_BRICK_ATTACH,
                                           glusterfs_handle_attach,
                                           NULL, 0, DRC_NA},
+
+        [GLUSTERD_DUMP_METRICS]        = {"DUMP METRICS",
+                                          GLUSTERD_DUMP_METRICS,
+                                          glusterfs_handle_dump_metrics,
+                                          NULL, 0, DRC_NA},
 };
 
 struct rpcsvc_program glusterfs_mop_prog = {
diff --git a/glusterfsd/src/glusterfsd.c b/glusterfsd/src/glusterfsd.c
index 69383f87b17..921a86bfd65 100644
--- a/glusterfsd/src/glusterfsd.c
+++ b/glusterfsd/src/glusterfsd.c
@@ -76,6 +76,7 @@
 #include "client_t.h"
 #include "netgroups.h"
 #include "exports.h"
+#include "monitoring.h"
 
 #include "daemon.h"
 
@@ -1542,6 +1543,9 @@ glusterfs_ctx_defaults_init (glusterfs_ctx_t *ctx)
          */
         ret = -1;
 
+        /* monitoring should be enabled by default */
+        ctx->measure_latency = true;
+
         ctx->process_uuid = generate_glusterfs_ctx_id ();
         if (!ctx->process_uuid) {
                 gf_msg ("", GF_LOG_CRITICAL, 0, glusterfsd_msg_13);
@@ -2175,14 +2179,14 @@ glusterfs_sigwaiter (void *arg)
         sigset_t  set;
         int       ret = 0;
         int       sig = 0;
-
+        char     *file = NULL;
 
         sigemptyset (&set);
         sigaddset (&set, SIGINT);   /* cleanup_and_exit */
         sigaddset (&set, SIGTERM);  /* cleanup_and_exit */
         sigaddset (&set, SIGHUP);   /* reincarnate */
         sigaddset (&set, SIGUSR1);  /* gf_proc_dump_info */
-        sigaddset (&set, SIGUSR2);  /* gf_latency_toggle */
+        sigaddset (&set, SIGUSR2);
 
         for (;;) {
                 ret = sigwait (&set, &sig);
@@ -2202,7 +2206,11 @@ glusterfs_sigwaiter (void *arg)
                         gf_proc_dump_info (sig, glusterfsd_ctx);
                         break;
                 case SIGUSR2:
-                        gf_latency_toggle (sig, glusterfsd_ctx);
+                        file = gf_monitor_metrics (glusterfsd_ctx);
+
+                        /* Nothing needed to be done here */
+                        GF_FREE (file);
+
                         break;
                 default:
 
@@ -2243,7 +2251,7 @@ glusterfs_signals_setup (glusterfs_ctx_t *ctx)
         sigaddset (&set, SIGTERM);  /* cleanup_and_exit */
         sigaddset (&set, SIGHUP);   /* reincarnate */
         sigaddset (&set, SIGUSR1);  /* gf_proc_dump_info */
-        sigaddset (&set, SIGUSR2);  /* gf_latency_toggle */
+        sigaddset (&set, SIGUSR2);
 
         ret = pthread_sigmask (SIG_BLOCK, &set, NULL);
         if (ret) {
-- 
cgit