<div dir="ltr"><div>Package: lvm2</div><div>Version: 2.02.168-1</div><div>Severity: normal</div><div><br></div><div><div>Subject: Enabling autpkgtests in lvm2.<br></div><div><br></div><div>Hi,</div><div>we recently ran into issues with lvm2 and kernel 4.10, see [1]..</div><div>Due to that I realized there were no dep8 tests for lvm2 yet.</div><div><br></div><div>The following is a series of commites as a suggestion that would add a few basic checks.</div><div>That shall help to spot issues in LVM (or dependent packages) early on.</div><div><br></div><div>The suggested change tried to poke at the basics of LVM:</div><div>- pvcreate, pvmove, pvdisplay, pvck, pvscan</div><div>- vgcreate, vgremove, vgextend, vgreduce, vgck</div><div>- lvcreate, lvremove, snapshots, lvconvert</div><div><br></div><div>Surely there could be more, but then one always has to limit those tests to have a reasonable execution time as well as to work in the autopkgtest environment.</div><div>Since I use loop mounts as base disks I set isolation-machine and since I'd need sudo on every other command it has needs-root.</div><div><br></div><div>I tested that using the Ubuntu Infrastructure building and testing LVM (as in Debian + these tests) and passed on all archietctures that supported it (e.g. those not able to provide isolation-machine just skip it).</div><div>Here is an example of such a tests [2] log.</div><div><br></div><div>Since it is just one script I saw no reason to post multiple commots showing me to evolve that to work.</div><div>Therefore I'm just attaching the debdiff for latest Debian version.</div><div><br></div><div>Please do note, that the trigger for all of this was a crash on i386 [3] - I just reported that on [1] so you can fix as well.</div><div>So if you apply the test today with a recent kernel it will crash on vgremove, but it works fine as soon as the fix is applied.</div><div>Thereby I'd recommend you stage this change in your repo, but only upload it together with the fix or the next lvm2 version bump.</div><div><br></div><div><br></div><div>Note: I also looked at enabling unit tests on build, but concluded it is not worth it at the moment.</div><div>Mostly due to disfunctional tests:</div><div>- have fingerprint mismatches in some tests (is that a failure in lvm?)</div><div>- show memleak in valgrind tests (actual error?)</div><div>- Ruby processing doesn't mark errors as failure</div><div>- Ruby hard dep on 1.9</div><div>- Will need to be fixed upstream before they can be enabled.</div><div>I developed the fixes in git as well, but since I don't think it is worth here a squashed diff FYI just in case you want to look at it as well [4].</div><div>There was an effort to revive them [5] , but I think it is incomplete at the current version or I didn't check deep enough.</div><div><br></div><div><br></div><div>[1]: <a href="https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=858002">https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=858002</a></div><div>[2]: <a href="http://paste.ubuntu.com/24193814/">http://paste.ubuntu.com/24193814/</a></div><div>[3]: <a href="https://sourceware.org/git/?p=lvm2.git;a=commit;h=46b735c937ce68e72d08997635321bf30240325d">https://sourceware.org/git/?p=lvm2.git;a=commit;h=46b735c937ce68e72d08997635321bf30240325d</a></div></div><div>[4]: <a href="http://paste.ubuntu.com/24175491/">http://paste.ubuntu.com/24175491/</a></div><div>[5]: <a href="https://www.redhat.com/archives/lvm-devel/2015-November/msg00165.html">https://www.redhat.com/archives/lvm-devel/2015-November/msg00165.html</a></div><div><br></div><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><span style="color:rgb(136,136,136);font-size:12.8px">Christian Ehrhardt</span><div style="color:rgb(136,136,136);font-size:12.8px">Software Engineer, Ubuntu Server</div><div style="color:rgb(136,136,136);font-size:12.8px">Canonical Ltd</div></div></div></div></div>
</div>