[Bugs] [Bug 1724021] New: volume set <VOLNAME> group <GROUP> description missing space leading to words being merged in help output

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1724021

            Bug ID: 1724021
           Summary: volume set <VOLNAME> group <GROUP> description missing
                    space leading to words being merged in help output
           Product: Red Hat Gluster Storage
           Version: rhgs-3.5
          Hardware: x86_64
                OS: Linux
            Status: NEW
         Component: cli
          Severity: low
          Assignee: srakonde at redhat.com
          Reporter: kiyer at redhat.com
        QA Contact: kiyer at redhat.com
                CC: bugs at gluster.org, rhs-bugs at redhat.com,
                    sankarshan at redhat.com, storage-qa-internal at redhat.com
        Depends On: 1723455
  Target Milestone: ---
    Classification: Red Hat



+++ This bug was initially created as a clone of Bug #1723455 +++

Description of problem:
volume set <VOLNAME> group <GROUP> description missing space leading to words
being merged. This can be seen easily in # gluster v help output.

################################################################################
# gluster v help

gluster volume commands
========================

volume add-brick <VOLNAME> [<stripe|replica> <COUNT> [arbiter <COUNT>]]
<NEW-BRICK> ... [force] - add brick to volume <VOLNAME>
volume barrier <VOLNAME> {enable|disable} - Barrier/unbarrier file operations
on a volume
volume clear-locks <VOLNAME> <path> kind {blocked|granted|all}{inode
[range]|entry [basename]|posix [range]} - Clear locks held on path
volume create <NEW-VOLNAME> [stripe <COUNT>] [replica <COUNT> [arbiter
<COUNT>]] [disperse [<COUNT>]] [disperse-data <COUNT>] [redundancy <COUNT>]
[transport <tcp|rdma|tcp,rdma>] <NEW-BRICK>... [force] - create a new volume of
specified type with mentioned bricks
volume delete <VOLNAME> - delete volume specified by <VOLNAME>
volume geo-replication [<MASTER-VOLNAME>] [<SLAVE-IP>]::[<SLAVE-VOLNAME>] {\
 create [[ssh-port n] [[no-verify] \
 | [push-pem]]] [force] \
 | start [force] \
 | stop [force] \
 | pause [force] \
 | resume [force] \
 | config [[[\!]<option>] [<value>]] \
 | status [detail] \
 | delete [reset-sync-time]}  - Geo-sync operations
volume get <VOLNAME|all> <key|all> - Get the value of the all options or given
option for volume <VOLNAME> or all option. gluster volume get all all is to get
all global options
volume heal <VOLNAME> [enable | disable | full |statistics [heal-count [replica
<HOSTNAME:BRICKNAME>]] |info [summary | split-brain] |split-brain {bigger-file
<FILE> | latest-mtime <FILE> |source-brick <HOSTNAME:BRICKNAME> [<FILE>]}
|granular-entry-heal {enable | disable}] - self-heal commands on volume
specified by <VOLNAME>
volume help - display help for volume commands
volume info [all|<VOLNAME>] - list information of all volumes
volume list - list all volumes in cluster
volume log <VOLNAME> rotate [BRICK] - rotate the log file for corresponding
volume/brick
volume log rotate <VOLNAME> [BRICK] - rotate the log file for corresponding
volume/brick NOTE: This is an old syntax, will be deprecated from next release.
volume profile <VOLNAME> {start|info [peek|incremental
[peek]|cumulative|clear]|stop} [nfs] - volume profile operations
volume rebalance <VOLNAME> {{fix-layout start} | {start [force]|stop|status}} -
rebalance operations
volume remove-brick <VOLNAME> [replica <COUNT>] <BRICK> ...
<start|stop|status|commit|force> - remove brick from volume <VOLNAME>
volume replace-brick <VOLNAME> <SOURCE-BRICK> <NEW-BRICK> {commit force} -
replace-brick operations
volume reset <VOLNAME> [option] [force] - reset all the reconfigured options
volume reset-brick <VOLNAME> <SOURCE-BRICK> {{start} | {<NEW-BRICK> commit}} -
reset-brick operations
volume set <VOLNAME> <KEY> <VALUE> - set options for volume <VOLNAME>
volume set <VOLNAME> group  <GROUP> - This option can be used for setting
multiple pre-defined volume optionswhere group_name is a file under
/var/lib/glusterd/groups containing onekey, value pair per line
volume start <VOLNAME> [force] - start volume specified by <VOLNAME>
volume statedump <VOLNAME> [[nfs|quotad]
[all|mem|iobuf|callpool|priv|fd|inode|history]... | [client
<hostname:process-id>]] - perform statedump on bricks
volume status [all | <VOLNAME> [nfs|shd|<BRICK>|quotad|tierd]]
[detail|clients|mem|inode|fd|callpool|tasks|client-list] - display status of
all or specified volume(s)/brick
volume stop <VOLNAME> [force] - stop volume specified by <VOLNAME>
volume sync <HOSTNAME> [all|<VOLNAME>] - sync the volume information from a
peer
volume top <VOLNAME> {open|read|write|opendir|readdir|clear} [nfs|brick
<brick>] [list-cnt <value>] |
volume top <VOLNAME> {read-perf|write-perf} [bs <size> count <count>] [brick
<brick>] [list-cnt <value>] - volume top operations
################################################################################

Version-Release number of selected component (if applicable):
Whatever is the upstream version.

How reproducible:
constantly 

Steps to Reproduce:
1.Issue # gluster v help command 

Actual results:
volume set <VOLNAME> group  <GROUP> - This option can be used for setting
multiple pre-defined volume optionswhere group_name is a file under
/var/lib/glusterd/groups containing onekey, value pair per line

Expected results:
volume set <VOLNAME> group <GROUP> - This option can be used for setting
multiple pre-defined volume options where group_name is a file under
/var/lib/glusterd/groups containing one key value pair per line

Additional info:

--- Additional comment from Worker Ant on 2019-06-24 14:54:18 UTC ---

REVIEW: https://review.gluster.org/22934 (Adding white spaces to description of
set group.) posted (#1) for review on master by Kshithij Iyer

--- Additional comment from Worker Ant on 2019-06-25 15:24:05 UTC ---

REVIEW: https://review.gluster.org/22934 (Adding white spaces to description of
set group.) merged (#5) on master by Atin Mukherjee


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1723455
[Bug 1723455] volume set <VOLNAME> group <GROUP> description missing space
leading to words being merged in help output
-- 
You are receiving this mail because:
You are on the CC list for the bug.