← Back to team overview

curtin-dev team mailing list archive

Re: [Merge] ~alexsander-souza/curtin:deploy_raw_lvm_images into curtin:master

 

Review: Needs Information

Hi, the basic issue seems clear and the fix seems reasonable. I do have have some questions though:

1) Is it possible to add a vmtest for this? I guess it would be a bit annoying as the test would have to construct a disk image with LVM stuff in it.
2) Would udevadm trigger --settle $dev achieve the same result and be more general?
-- 
https://code.launchpad.net/~alexsander-souza/curtin/+git/curtin/+merge/408913
Your team curtin developers is subscribed to branch curtin:master.



Follow ups

References