From 1bbe80e83dddbd14838f63c2c881627a74a3510b Mon Sep 17 00:00:00 2001 From: Jesse Pretorius Date: Mon, 19 Jun 2017 15:05:44 +0100 Subject: [PATCH] Install lvm2 for cinder-volume LVM hosts Currently the LVM2 package is installed by the openstack_hosts role for the integrated build, and by a test playbook for the role tests. This does not seem sensible to me - it would be better to ensure that the role installs the distro packages it need in order to function properly. Additionally, it is not possible to setup a loopback device inside an LXC container after some recent updates to LXC. The secondary cinder-volume service for LVM2 therefore does not work, so it is removed from the inventory to buy some time to come up with an alternative solution. Related-Bug: 1698831 Change-Id: Ie8a84a0412989c64fd896e8d781b8e9b2c153556 --- tests/inventory | 3 --- vars/redhat-7.yml | 1 + vars/ubuntu-16.04.yml | 1 + 3 files changed, 2 insertions(+), 3 deletions(-) diff --git a/tests/inventory b/tests/inventory index c3e91481..93ed5a0e 100644 --- a/tests/inventory +++ b/tests/inventory @@ -3,13 +3,11 @@ localhost infra1 keystone cinder -cinder_vol [all_containers] infra1 keystone cinder -cinder_vol [rabbitmq_all] infra1 @@ -42,7 +40,6 @@ cinder [cinder_volume] localhost -cinder_vol [cinder_all:children] cinder_api diff --git a/vars/redhat-7.yml b/vars/redhat-7.yml index dffd264b..bc921e5c 100644 --- a/vars/redhat-7.yml +++ b/vars/redhat-7.yml @@ -36,5 +36,6 @@ cinder_volume_distro_packages: cinder_lvm_volume_distro_packages: - device-mapper-event + - lvm2 - parted - scsi-target-utils diff --git a/vars/ubuntu-16.04.yml b/vars/ubuntu-16.04.yml index 86b6aa77..1e4acac8 100644 --- a/vars/ubuntu-16.04.yml +++ b/vars/ubuntu-16.04.yml @@ -43,6 +43,7 @@ cinder_volume_distro_packages: cinder_lvm_volume_distro_packages: - dmeventd + - lvm2 - parted - tgt - thin-provisioning-tools