Nir Soffer has posted comments on this change.
Change subject: guest-lvs: Add lvm bootstrap tests ......................................................................
Patch Set 7:
(1 comment)
https://gerrit.ovirt.org/#/c/64367/7/tests/storage_lvm_test.py File tests/storage_lvm_test.py:
Line 57: Line 58: class TestDeactivation(TestCaseBase): Line 59: Line 60: @ValidateRunningAsRoot Line 61: def test_deactivate_unused_ovirt_lvs(self): This test always fails in the ci, anyone has an idea why? maybe we need to filter out these tests on the ci, running inside mock.
On real system (fedora 24, rhel 7.3 configured as ovirt host), it works. Line 62: with fake_env() as env: Line 63: log.debug("Creating ovirt lvs") Line 64: run("pvcreate", "-ff", env.device) Line 65: run("vgcreate", "ovirt-vg", env.device)