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

backend_setup role fails with invalid argument for `lvcreate` #129

Closed anoopcs9 closed 2 years ago

anoopcs9 commented 2 years ago

Following error is seen while preparing backend for GlusterFS cluster:

TASK [gluster.infra/roles/backend_setup : Create a LV thinpool for similar device types] ***
failed: [storage0] (item={'vgname': 'vg_vdb', 'thinpoolname': 'vg_vdb_thinpool', 'thinpoolsize': '9G', 'poolmetadatasize': '100M'}) => {"ansible_loop_var": "item", "changed": true, "cmd": ["lvcreate", "--type", "thin-pool", "--zero", "n", "-l", "9G", "--chunksize", "256K", "--poolmetadatasize", "100MiB", "-n", "vg_vdb_thinpool", "vg_vdb"], "delta": "0:00:00.012250", "end": "2022-06-02 10:02:10.161117", "item": {"poolmetadatasize": "100M", "thinpoolname": "vg_vdb_thinpool", "thinpoolsize": "9G", "vgname": "vg_vdb"}, "msg": "non-zero return code", "rc": 3, "start": "2022-06-02 10:02:10.148867", "stderr": "  Invalid argument for --extents: 9G\n  Error during parsing of command line.", "stderr_lines": ["  Invalid argument for --extents: 9G", "  Error during parsing of command line."], "stdout": "", "stdout_lines": []}

It looks like wrong option -l is used instead of -L.

anoopcs9 commented 2 years ago

@rchikatw Can we have this fixed?

rchikatw commented 2 years ago

Hello Anoop,

Looks like there is an issue with passing thinpoolsize parameter, As of now don't pass that value it will default available space.

I will commit the required changes within 1-2 days.

anoopcs9 commented 2 years ago

Fixed with #130

gobindadas commented 2 years ago

PR merged, so closing this issue.