[Bug 1209725] autoinst.xml gen by "yast2 clone_system" mess thin pool LVM partitions?
https://bugzilla.suse.com/show_bug.cgi?id=1209725 https://bugzilla.suse.com/show_bug.cgi?id=1209725#c15 Ancor Gonzalez Sosa <ancor@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|needinfo?(igonzalezsosa@sus | |e.com) | --- Comment #15 from Ancor Gonzalez Sosa <ancor@suse.com> --- (In reply to Stefan Hundhammer from comment #14)
That much is clear. But the question was: Does the cloning part do that extra iteration over each LVM LV because it might have sub-elements itself; which only happens in the case of a thin pool LV?
No. And that's exactly the root of the problem. Here it only iterates through the first level https://github.com/yast/yast-storage-ng/blob/master/src/lib/y2storage/autoin... If it would use "vg.all_lvm_lvs" instead of the current "vg.lvm_lvs", then it would cover the thin volumes as well and it would likely work. I can write a unit test to verify that. -- You are receiving this mail because: You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply@suse.com