summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlexandru Avadanii <Alexandru.Avadanii@enea.com>2016-04-18 18:17:57 +0000
committerGerrit Code Review <gerrit@172.30.200.206>2016-04-18 18:17:58 +0000
commit98b1206261c4901d436d8019701ae28e544c065e (patch)
treeaf855ce5ccf346f1eec256559a8480b054fbdf64
parentd531461404a59e574abf97b756ebefd4562fe6d5 (diff)
parentc3b6de80481baba29fed8eb35259e28e2022bc7c (diff)
Merge "ceilometer: Fix libvirt-bin group name for armband."
-rw-r--r--patches/fuel-library/0012-ceilometer-Fix-libvirt-bin-group-name-for-armband.patch46
1 files changed, 46 insertions, 0 deletions
diff --git a/patches/fuel-library/0012-ceilometer-Fix-libvirt-bin-group-name-for-armband.patch b/patches/fuel-library/0012-ceilometer-Fix-libvirt-bin-group-name-for-armband.patch
new file mode 100644
index 00000000..b1d44501
--- /dev/null
+++ b/patches/fuel-library/0012-ceilometer-Fix-libvirt-bin-group-name-for-armband.patch
@@ -0,0 +1,46 @@
+From: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
+Date: Mon, 18 Apr 2016 18:44:27 +0200
+Subject: [PATCH] ceilometer: Fix libvirt-bin group name for armband.
+
+Upstream mirrors use a custom version of libvirt, ported from
+Debian, while keeping the Ubuntu ceilometer package, leading to a
+libvirt group name mismatch between the two.
+
+Fuel-library hardcodes a deb-version test for libvirt-bin, which
+enforces the use of "libvirt" instead of "libvirtd" for 1.2.9 or
+newer libvirt-bin [1].
+
+Armband brings its own 1.3.2 libvirt-bin package, which respects
+Ubuntu standard group naming ("libvirtd"), but since 1.3.2 > 1.2.9,
+the old group name ("libvirt") is still used.
+
+This patch extends the version checking introduced in [1]:
+ - libvirt-bin 1.3.2 or newer will use "libvirtd" naming;
+ - libvirt-bin 1.2.9 ... <1.3.2 will use "libvirt".
+
+[1] https://review.openstack.org/#/c/200602/8/deployment/puppet/
+ openstack/manifests/ceilometer.pp
+---
+ deployment/puppet/openstack/manifests/ceilometer.pp | 8 +++++++-
+ 1 file changed, 7 insertions(+), 1 deletion(-)
+
+diff --git a/deployment/puppet/openstack/manifests/ceilometer.pp b/deployment/puppet/openstack/manifests/ceilometer.pp
+index 8311731..1d7af2b 100644
+--- a/deployment/puppet/openstack/manifests/ceilometer.pp
++++ b/deployment/puppet/openstack/manifests/ceilometer.pp
+@@ -195,8 +195,14 @@ class openstack::ceilometer (
+
+ if ($on_compute) {
+ if $::operatingsystem == 'Ubuntu' and $::ceilometer::params::libvirt_group {
++ # Armband libvirt-bin deb package (1.3.2 version) creates 'libvirtd' group on Ubuntu
++ if (versioncmp($::libvirt_package_version, '1.3.2') >= 0) {
++ User<| name == 'ceilometer' |> {
++ groups => ['nova', 'libvirtd'],
++ }
++ }
+ # Our libvirt-bin deb package (1.2.9 version) creates 'libvirt' group on Ubuntu
+- if (versioncmp($::libvirt_package_version, '1.2.9') >= 0) {
++ elsif (versioncmp($::libvirt_package_version, '1.2.9') >= 0) {
+ User<| name == 'ceilometer' |> {
+ groups => ['nova', 'libvirt'],
+ }