Bug #87358 for Linux-LVM: liblinux-lvm-storage doesn't pass the LV Path field

Fri Jul 26 12:32:24 2013 gregoa [...] cpan.org - Ticket created

From:	gregoa [...] cpan.org
Subject:	liblinux-lvm-storage doesn't pass the LV Path field

This bug has been forwarded from http://bugs.debian.org/717684 -->-->-->-->-->-->-->-->-->-->-->-->-->-->-->-->-->-->-->--> Package: liblinux-lvm-perl Version: 0.16-1 Severity: important Hello, I've been trying to debug issues in fai-setup-storage using lvm partitions, and in the process I found that they use your module. The ultimate problem is that logical volume information is derived from executing the "vgdisplay -v" command within your module. The output from this command has changed between squeeze and wheezy. The older version of this command has fewer fields. snippet of older output: LV Name /dev/vg/var VG Name vg snippit of newer output: LV Path /dev/vg/var LV Name var VG Name vg Ultimately they have a logic problem because between squeeze and wheezy lvm apparently added a field and redefined the meaning for the old field. That said, your module needs to pass through the new field for any user of the module to get accurate information, and I don't believe that the code is there to do so. Thanks for supporting the module, and effort you put towards the fix, -Ken Hahn <--<--<--<--<--<--<--<--<--<--<--<--<--<--<--<--<--<--<--<-- Thanks in advance, gregor herrmann, Debian Perl Group

Wed Oct 16 00:20:23 2013 dam [...] cpan.org - Correspondence added

Subject:

[patch] RT#87358: Linux::LVM doesn't parse the LV Path field

На 26 юли 2013, пт 19:32:24, GREGOA написа: Show quoted text

> This bug has been forwarded from http://bugs.debian.org/717684

Attached is the patch that is applied to the Debian package. It adapts the code to the vgdisplay output in Linux 3.x, dropping support for older kernels.

Subject:

vgdisplay-output.patch

Description: adapt to 'vgdisplay' output in Linux 3.x The patch doesn't support the pre-3.x output anymore Author: Thomas Lange <lange@informatik.uni-koeln.de> Bug: https://rt.cpan.org/Public/Bug/Display.html?id=87358 Bug-Debian: http://bugs.debian.org/717684 Reviewed-By: Damyan Ivanov <dmn@debian.org> Last-Update: 2013-10-16 --- a/LVM.pm +++ b/LVM.pm @@ -288,9 +288,14 @@ sub get_vg_information() { $vghash{$vgn}->{uuid} = $1; next VGINF; } + # Parse the logical path name. + elsif( m/LV Path\s+(\S+)/ ) { + $lvn = $1; + $vghash{$vgn}->{lvols}->{$lvn}->{path} = $1; + next VGINF; } + # Parse the logical volume name. elsif( m/LV Name\s+(\S+)/ ) { - $lvn = $1; $vghash{$vgn}->{lvols}->{$lvn}->{name} = $1; next VGINF; }

Wed Oct 16 00:20:24 2013 The RT System itself - Status changed from 'new' to 'open'

Wed Oct 16 00:24:15 2013 dam [...] cpan.org - Broken in 0.16 added

Wed Oct 16 20:53:36 2013 support [...] bettercgi.com - Correspondence added

RT-Send-CC:

gregoa [...] cpan.org, lange [...] informatik.uni-koeln.de, 717684 [...] bugs.debian.org

We can't break LVM.pm on most distributions. Please either: a) take two minutes to adjust the patch to not break on other latest current distros, such as Red Hat (if "Path is unset, use "Name") or better: b) man lvs, which would make it more robust and efficient long term On Wed Oct 16 00:20:23 2013, DAM wrote: Show quoted text

> На 26 юли 2013, пт 19:32:24, GREGOA написа:

> > This bug has been forwarded from http://bugs.debian.org/717684

> > Attached is the patch that is applied to the Debian package. It adapts > the code to the vgdisplay output in Linux 3.x, dropping support for > older kernels.

Wed Oct 16 21:03:27 2013 717684 [...] bugs.debian.org - Correspondence added

Subject:	Bug#717684: Info received ([rt.cpan.org #87358] liblinux-lvm-storage doesn't pass the LV Path field)
Date:	Thu, 17 Oct 2013 01:03:04 +0000
To:	bug-Linux-LVM [...] rt.cpan.org
From:	owner [...] bugs.debian.org (Debian Bug Tracking System)

Thank you for the additional information you have supplied regarding this Bug report. This is an automatically generated reply to let you know your message has been received. Your message is being forwarded to the package maintainers and other interested parties for their attention; they will reply in due course. Your message has been sent to the package maintainer(s): Debian Perl Group <pkg-perl-maintainers@lists.alioth.debian.org> If you wish to submit further information on this problem, please send it to 717684@bugs.debian.org. Please do not send mail to owner@bugs.debian.org unless you wish to report a problem with the Bug-tracking system. -- 717684: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=717684 Debian Bug Tracking System Contact owner@bugs.debian.org with problems

Thu Nov 07 12:57:18 2013 support [...] bettercgi.com - Correspondence added

RT-Send-CC:

717684 [...] bugs.debian.org, gregoa [...] cpan.org

This has been addressed in 0.17, while remaining compatible with other distributions including Red Hat.

Thu Nov 07 12:58:46 2013 support [...] bettercgi.com - Status changed from 'open' to 'resolved'

Thu Nov 07 12:58:47 2013 support [...] bettercgi.com - Fixed in 0.17 added

Thu Nov 07 13:00:26 2013 717684 [...] bugs.debian.org - Correspondence added

Subject:	Bug#717684: Info received ([rt.cpan.org #87358] liblinux-lvm-storage doesn't pass the LV Path field)
Date:	Thu, 07 Nov 2013 18:00:04 +0000
To:	bug-Linux-LVM [...] rt.cpan.org
From:	owner [...] bugs.debian.org (Debian Bug Tracking System)

Thank you for the additional information you have supplied regarding this Bug report. This is an automatically generated reply to let you know your message has been received. Your message is being forwarded to the package maintainers and other interested parties for their attention; they will reply in due course. Your message has been sent to the package maintainer(s): Debian Perl Group <pkg-perl-maintainers@lists.alioth.debian.org> If you wish to submit further information on this problem, please send it to 717684@bugs.debian.org. Please do not send mail to owner@bugs.debian.org unless you wish to report a problem with the Bug-tracking system. -- 717684: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=717684 Debian Bug Tracking System Contact owner@bugs.debian.org with problems