Fix creating a VG with multiple PVs

nilsmeyer / ansible-debootstrap

An ansible role for bootstrapping new Debian based systems, including setting up partitions, file systems, encryption (luks), RAID and LVM

GNU Lesser General Public License v3.0

35 stars 19 forks source link

Fix creating a VG with multiple PVs #38

Closed mvk15 closed 3 years ago

mvk15 commented 3 years ago

Found the issue with Ansible >= 2.8 (tested 2.8 and 2.9) and creating VG with multiple PVs. For fixing the issue decided to join PVs that belong to the same VG to a list and use them in one loop iteration.

nilsmeyer commented 3 years ago

Thank you @mvk for your PR, unfortunately I haven't had the time to test your changes, only to review the code.

mvk15 commented 3 years ago

Hi, It does not hurry up. Please test it when you will have time.