--- Begin Message ---
Dear Ray,

while setting up a new Debain box with FAI, I stumbled over an issue with setting up LVM. FAIs setup-storage failed, complaining about uninitialized variables somewhere in the LVM.pm module.

It turned out, that someone else also stumbled upon this and filed a bug report with FAI: https://www.mail-archive.com/linux-fai@uni-koeln.de/msg05423.html

Thomas Lange guessed right, the issue is in the Linux::LVM perl module, version 0.17. I managed to fix the issue, see the attached patch file. After applying the patch, FAI setup-storage works fine again with LVM.

Best regards,
Roland

--
Roland Dieterich
IT-Manager

Max Planck-Genome-centre Cologne
Building B
Carl-von-Linné-Weg 10
50829 Köln

Tel: (+49)221-5062816
Fax: (+49)221-5062851
--- LVM.pm.orig 2014-04-23 15:51:33.000000000 +0200
+++ LVM.pm      2014-04-23 18:15:23.000000000 +0200
@@ -290,15 +290,16 @@
 
         # Parse the logical volume name.
         elsif( m/LV Name\s+(\S+)/ ) { 
-            $lvn = $1; 
+            $lvn = $1 unless $lvn;
             $vghash{$vgn}->{lvols}->{$lvn}->{name} = $1; 
             next VGINF; }
 
         # since version 2.02.89 'LV Name' is no longer the full path, 'LV 
Path' is.
         # LV Path may be bogus or missing in some cases, such as thin pools.
         if( m/LV Path\s+(\S+)/ ) {
+            $lvn = $1;
             $vghash{$vgn}->{lvols}->{$lvn}->{name} = $1;
-            next LVINF; }
+            next VGINF; }
 
         # Parse the logical volume UUID.
         elsif( m/LV UUID\s+(\S+)/ ) { 

--- End Message ---
-- 
regards Thomas

Reply via email to