]> review.fuel-infra Code Review - puppet-modules/puppet-ceilometer.git/commitdiff
Fix dependency on nova-common package
authoriberezovskiy <iberezovskiy@mirantis.com>
Fri, 22 May 2015 09:39:48 +0000 (12:39 +0300)
committeriberezovskiy <iberezovskiy@mirantis.com>
Fri, 22 May 2015 09:43:51 +0000 (12:43 +0300)
  Nova module could be not included or imported directly
  and this can lead to failure with missing dependency
  on 'nova-common' package. More safe is to use
  Package <| title == nova-common |>

Change-Id: Idac8d52955ff6f3f1c62a538b197eeac6f281a1f

manifests/agent/compute.pp
manifests/agent/polling.pp

index 1c48c781ce7f5d19bc6e509edd898b075800348c..ec678b8ed06edad8e97c1261bd06f54c80dbe9ad 100644 (file)
@@ -51,7 +51,7 @@ class ceilometer::agent::compute (
     }
   }
 
-  Package['nova-common'] -> Package['ceilometer-common'] -> Service['ceilometer-agent-compute']
+  Package <| title == 'nova-common' |> -> Package['ceilometer-common'] -> Service['ceilometer-agent-compute']
   service { 'ceilometer-agent-compute':
     ensure     => $service_ensure,
     name       => $::ceilometer::params::agent_compute_service_name,
index a4316af21766cbbaaa239680de0c952ffdec3de3..0987420ac5207f0895bec9f61be34f54d21b23c2 100644 (file)
@@ -83,7 +83,7 @@ class ceilometer::agent::polling (
 
     $compute_namespace_name = 'compute'
 
-    Package['nova-common'] -> Package['ceilometer-common']
+    Package <| title == 'nova-common' |> -> Package['ceilometer-common']
   }
 
   if $ipmi_namespace {