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 lvm exclude filter #107

Closed pkesavap closed 3 years ago

pkesavap commented 3 years ago

Before RHHI-V Deployment , certain tasks (that might involve brick creation, or Day2 operation like creation of new volume, expanding the cluster or volume,) require removal of existing LVM filter

to avoid errors like " Device /dev/sdf excluded by a filter."

pkesavap commented 3 years ago

This PR contains two independent task

1) Create lvm exclude filter this tasks backups existing filter and removes the current filter 2) Create regenerate_new_lvm_filter_rules.yml this task regenerates new lvm filter rules

The reason why its created as two independent task, is that it can be further called or included in gluster-ansible tasks wherever applicable, so that code need not be repeated.

pkesavap commented 3 years ago

Requesting review :- @gobindadas @satheesaran

sabose commented 3 years ago

@pkesavap "Post RHHI-V Deployment " -> post or before the deployment?

pkesavap commented 3 years ago

@pkesavap "Post RHHI-V Deployment " -> post or before the deployment?

done, set to before

pkesavap commented 3 years ago

@satheesaran @gobindadas have updated the discussed changes,

gobindadas commented 3 years ago

Looks good