summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPranith Kumar K <pkarampu@redhat.com>2015-12-21 17:55:49 +0530
committerPranith Kumar Karampuri <pkarampu@redhat.com>2016-02-02 02:25:14 -0800
commitac3183a5012bfed26fa0aead7f359f5d5b00e23e (patch)
tree2dd9c37cfe00d631990e4e90157eb0f9317798c0
parent959785a885528f66c56e744641c71d1e7e2fadc8 (diff)
heal: Have fixed number of fields in heal info output
Sample output: ⚡ gluster v heal r2 info Brick localhost.localdomain:/home/gfs/r2_0 /a / Status: Connected Number of entries: 2 Brick localhost.localdomain:/home/gfs/r2_1 Status: Transport endpoint is not connected Number of entries: - Brick localhost.localdomain:/home/gfs/r2_2 Status: Connected Number of entries: 0 Brick localhost.localdomain:/home/gfs/r2_3 Status: Connected Number of entries: 0 Partial failure will print: ... <gfid:3efb51cf-5fa5-4bed-817b-afd600e9449e> Status: Failed to process entries completely. (Transport endpoint is not connected) Number of entries: 372 ... BUG: 1297897 Change-Id: I79131cbf255acd98f11eedafb53246c465f6665d Signed-off-by: Pranith Kumar K <pkarampu@redhat.com> Reviewed-on: http://review.gluster.org/13231 Smoke: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Krutika Dhananjay <kdhananj@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
-rw-r--r--heal/src/glfs-heal.c31
1 files changed, 20 insertions, 11 deletions
diff --git a/heal/src/glfs-heal.c b/heal/src/glfs-heal.c
index 960fd20bb96..0be21bb57a4 100644
--- a/heal/src/glfs-heal.c
+++ b/heal/src/glfs-heal.c
@@ -54,23 +54,32 @@ void
glfsh_print_heal_op_status (int ret, uint64_t num_entries,
gf_xl_afr_op_t heal_op)
{
+ char *fmt_str = NULL;
- if (ret == -ENOTCONN && num_entries == 0) {
+ if (heal_op == GF_SHD_OP_INDEX_SUMMARY)
+ fmt_str = "Number of entries:";
+ else if (heal_op == GF_SHD_OP_SPLIT_BRAIN_FILES)
+ fmt_str = "Number of entries in split-brain:";
+ else if (heal_op == GF_SHD_OP_SBRAIN_HEAL_FROM_BRICK)
+ fmt_str = "Number of healed entries:";
+
+ if (ret < 0 && num_entries == 0) {
printf ("Status: %s\n", strerror (-ret));
+ if (fmt_str)
+ printf ("%s -\n", fmt_str);
return;
+ } else if (ret == 0) {
+ printf ("Status: Connected\n");
}
+
if (ret < 0) {
- printf ("Failed to process entries completely. "
- "Number of entries so far: %"PRIu64"\n", num_entries);
+ if (fmt_str)
+ printf ("Status: Failed to process entries completely. "
+ "(%s)\n%s: %"PRIu64"\n",
+ strerror (-ret), fmt_str, num_entries);
} else {
- if (heal_op == GF_SHD_OP_INDEX_SUMMARY)
- printf ("Number of entries: %"PRIu64"\n", num_entries);
- else if (heal_op == GF_SHD_OP_SPLIT_BRAIN_FILES)
- printf ("Number of entries in split-brain: %"PRIu64"\n"
- , num_entries);
- else if (heal_op == GF_SHD_OP_SBRAIN_HEAL_FROM_BRICK)
- printf ("Number of healed entries: %"PRIu64"\n",
- num_entries);
+ if (fmt_str)
+ printf ("%s %"PRIu64"\n", fmt_str, num_entries);
}
return;
}