gluster / gluster-ansible-infra

Ansible role to enable and deploy the backend of a Gluster cluster.
GNU General Public License v3.0
28 stars 36 forks source link

Create thick LV is broken #139

Open stobbsm opened 8 months ago

stobbsm commented 8 months ago

The current implementation has a default value of 100%FREE, but is using the -L|--size argument, which only allows physical sizes like 100G.

I would like to suggest changing the default -L 100%FREE to the more acceptable -l 100%FREE, which won't fail on run. I'm not an ansible genius, but it should be possible to have different settings, ie. lvextentsize and lvbytesize, and to favour one over another.

Maybe set the default of lvextentsize, but if lvbytesize is defined, use that, which will be -L {SIZE}[M|G|T]