search

gluster / gluster-block

A framework for gluster block storage
GNU General Public License v2.0
74 stars 32 forks source link

genconfig: switch to string instead of fixed 255 for the volume #261

Closed lxbsz closed 4 years ago

lxbsz commented 4 years ago

What does this PR achieve? Why do we need it?

If we specify more than 2 volumes in one genconfig command line, the genconfig will truncate the volumes.

Does this PR fix issues?

Yeah

Fixes: #1787371

Notes for the reviewer

pkalever commented 4 years ago

Tested and it works as expected.

pkalever commented 4 years ago

@lxbsz update the tags, this is ready for merge Reviewed-by: Prasanna Kumar Kalever \prasanna.kalever@redhat.com\