Ancor Gonzalez Sosa changed bug 1209725
What Removed Added
Flags needinfo?(igonzalezsosa@suse.com)  

Comment # 15 on bug 1209725 from
(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/autoinst_profile/drive_section.rb#L388

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: