[Gluster-devel] Need suggestion to change out put of volume status command

[Shyam]

On 11/26/2014 08:19 AM, Mohammed Rafi K C wrote:
>
>
> Hi All,
>
> We are planning to change the volume status command to show RDMA port for
> tcp,rdma volumes. We have four output designs in mind , those are,
>
> 1)Modify Port column as TCP,RDMA Ports
> Eg:
>
> Status of volume: xcube
> Gluster process                            TCP,RDMA Port    Online    Pid
> --------------------------------------------------------------------------------
> Brick 192.168.44.108:/home/brick           49157,49158       Y        6030
> NFS Server on localhost                    2049              Y        6532
> Self-heal Daemon on localhost              N/A               N        6234

In this method when RDMA is not present the heading will still read TCP, 
RDMA and entries would be of the <tcp-port> form.

Also if _only_ RDMA is present (if that is possible) this causes a 
presentation problem, at it would read ,<rdma-port> which is not nice. 
so my vote is a no.

>
> 2)Change entry in port as TCP-XXXXX,RDMA-XXXXX
>
> Eg:
>
>
> Status of volume: xcube
> Gluster process                            Port                  Online    Pid
> ---------------------------------------------------------------------------------
> Brick 192.168.44.108:/home/brick           TCP-49157,RDMA-49158  Y         6030
> NFS Server on localhost                    TCP-2049              Y         6532
> Self-heal Daemon on localhost              N/A                   N         6234

This is better as it is extensible in the future as well (in case there 
are other such ports). I would personally like this as 
<portnum>(<protocol>) rather than the current form of <protocol>-<portnum>

>
> 3)Instead of column port add two other port as TCP Port and RDMA Port
>
> Eg:
>
>
> Status of volume: xcube
> Gluster process                          TCP Port  RDMA Port   Online    Pid
> -------------------------------------------------------------------------------
> Brick 192.168.44.108:/home/brick         49157     49158          Y      6030
> NFS Server on localhost                  2049                     Y      6532
> Self-heal Daemon on localhost            N/A                      N      6234

This is cleaner, but later _if_ we need another column gets messy, but 
that is not a possibility in the near future, so a good candidate for 
consideration.

>
> 4) Make an extra row appending ".rdma" with brick name for rdma port
>
> Eg:
>
>
> Status of volume: xcube
> Gluster process                                 Port     Online   Pid
> ------------------------------------------------------------------------
> Brick 192.168.44.108:/home/brick                49157    Y        6030
> Brick 192.168.44.108:/home/brick.rdma           49157    Y        6030
> NFS Server on localhost                         2049     Y        6532
> Self-heal Daemon on localhost                   N/A      N       6234

We really do not have multiple processes, so adding 2 each for a 
protocol port representation, misses the meaning of the header for this 
column, so a no from me.

>
> Please give your comments regarding this designs, or if you have better
> idea in mind , feel free to share it.
>
> Best Regards
> Rafi KC
>
>
>
> _______________________________________________
> Gluster-devel mailing list
> Gluster-devel at gluster.org
> http://supercolony.gluster.org/mailman/listinfo/gluster-devel
>