cli/quota: Sort the list output alphabetically by path

This is a backport of http://review.gluster.org/14000

> Change-Id: I0b124e119d167817be2ae3eb52ac6c80fc7db5d1
> BUG: 1320716
> Signed-off-by: vmallika <vmallika@redhat.com>
> Reviewed-on: http://review.gluster.org/14000
> Smoke: Gluster Build System <jenkins@build.gluster.com>
> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
> CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
> Reviewed-by: Kaushal M <kaushal@redhat.com>

Change-Id: I87e12d58c8e267b2af67e287998e7313efc70af4
BUG: 1330018
Signed-off-by: vmallika <vmallika@redhat.com>
Reviewed-on: http://review.gluster.org/14061
Smoke: Gluster Build System <jenkins@build.gluster.com>
CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>

1 files changed, 61 insertions, 5 deletions

diff --git a/cli/src/cli-rpc-ops.c b/cli/src/cli-rpc-ops.c
index a2519e8c0e8..1d507f7a205 100644
--- a/cli/src/cli-rpc-ops.c
+++ b/cli/src/cli-rpc-ops.c
@@ -3474,6 +3474,35 @@ cli_cmd_broadcast_response_detached (void *opaque)
         return NULL;
 }
 
+int32_t
+cli_quota_compare_path (struct list_head *list1,
+                        struct list_head *list2)
+{
+        struct list_node  *node1  = NULL;
+        struct list_node  *node2  = NULL;
+        dict_t            *dict1  = NULL;
+        dict_t            *dict2  = NULL;
+        char              *path1  = NULL;
+        char              *path2  = NULL;
+        int                ret    = 0;
+
+        node1 = list_entry (list1, struct list_node, list);
+        node2 = list_entry (list2, struct list_node, list);
+
+        dict1 = node1->ptr;
+        dict2 = node2->ptr;
+
+        ret = dict_get_str (dict1, GET_ANCESTRY_PATH_KEY, &path1);
+        if (ret < 0)
+                return 0;
+
+        ret = dict_get_str (dict2, GET_ANCESTRY_PATH_KEY, &path2);
+        if (ret < 0)
+                return 0;
+
+        return strcmp (path1, path2);
+}
+
 int
 cli_quotad_getlimit_cbk (struct rpc_req *req, struct iovec *iov,
                           int count, void *myframe)
@@ -3482,11 +3511,14 @@ cli_quotad_getlimit_cbk (struct rpc_req *req, struct iovec *iov,
         gf_cli_rsp         rsp         = {0,};
         int                ret         = -1;
         dict_t            *dict        = NULL;
+        struct list_node  *node        = NULL;
+        struct list_node  *tmpnode     = NULL;
         call_frame_t      *frame       = NULL;
         cli_local_t       *local       = NULL;
         dict_t            *gd_rsp_dict = NULL;
         int32_t            list_count  = 0;
         pthread_t          th_id       = {0, };
+        int32_t            max_count   = 0;
 
         frame = myframe;
         GF_ASSERT (frame);
@@ -3500,8 +3532,10 @@ cli_quotad_getlimit_cbk (struct rpc_req *req, struct iovec *iov,
                                       &list_count);
                 if (ret)
                         list_count = 0;
+
+                list_count++;
                 ret = dict_set_int32 (gd_rsp_dict, "quota-list-count",
-                                      list_count + 1);
+                                      list_count);
         }
         UNLOCK (&local->lock);
 
@@ -3549,7 +3583,32 @@ cli_quotad_getlimit_cbk (struct rpc_req *req, struct iovec *iov,
                         goto out;
                 }
 
-                print_quota_list_from_quotad (frame, dict);
+                ret = dict_get_int32 (local->dict, "max_count",
+                                      &max_count);
+                if (ret < 0) {
+                        gf_log ("cli", GF_LOG_ERROR,
+                                "failed to get max_count");
+                        goto out;
+                }
+
+                node = list_node_add_order (dict, &local->dict_list,
+                                            cli_quota_compare_path);
+                if (node == NULL) {
+                        gf_log ("cli", GF_LOG_ERROR,
+                                "failed to add node to the list");
+                        dict_unref (dict);
+                        goto out;
+                }
+
+                if (list_count == max_count) {
+                        list_for_each_entry_safe (node, tmpnode,
+                                                  &local->dict_list, list) {
+                                dict = node->ptr;
+                                print_quota_list_from_quotad (frame, dict);
+                                list_node_del (node);
+                                dict_unref (dict);
+                        }
+                }
         }
 
 out:
@@ -3579,9 +3638,6 @@ out:
                 cli_cmd_broadcast_response (ret);
         }
 
-        if (dict)
-                dict_unref (dict);
-
         free (rsp.dict.dict_val);
         return ret;
 }