[PATCH configure 1/1] Ensure libvirt is installed before trying to start the service.

[Mike Orazi]

---
 recipes/aeolus_recipe/manifests/image-factory.pp |    8 +++++++-
 1 files changed, 7 insertions(+), 1 deletions(-)

diff --git a/recipes/aeolus_recipe/manifests/image-factory.pp
b/recipes/aeolus_recipe/manifests/image-factory.pp
index 499b6e4..5bcd24f 100644
--- a/recipes/aeolus_recipe/manifests/image-factory.pp
+++ b/recipes/aeolus_recipe/manifests/image-factory.pp
@@ -2,6 +2,11 @@
 
 class aeolus::image-factory inherits aeolus {
     if $enable_packages {
+
+      package { 'libvirt':
+                provider => 'yum',
+                ensure=> 'installed'
+      }
       package { 'imagefactory':
                    provider => 'yum', ensure => 'installed',
                    require => [Yumrepo['aeolus_arch',
'aeolus_noarch']]
@@ -31,7 +36,8 @@ class aeolus::image-factory inherits aeolus {
     service {'libvirtd':
                ensure  => 'running',
                enable  => true,
-               hasstatus => true }
+               hasstatus => true,
+               require => return_if($enable_packages, Package['libvirt'])}
     $requires = [return_if($enable_packages, Package['imagefactory']),
                  File['/var/tmp/imagefactory-mock'],
                  Service[qpidd], Service[libvirtd],
-- 
1.7.4

[Angus Thomas]

ACK

On 31/03/11 06:15, Mike Orazi wrote:
>  ---
>    recipes/aeolus_recipe/manifests/image-factory.pp |    8 +++++++-
>    1 files changed, 7 insertions(+), 1 deletions(-)
>
>  diff --git a/recipes/aeolus_recipe/manifests/image-factory.pp
> b/recipes/aeolus_recipe/manifests/image-factory.pp
>  index 499b6e4..5bcd24f 100644
>  --- a/recipes/aeolus_recipe/manifests/image-factory.pp
>  +++ b/recipes/aeolus_recipe/manifests/image-factory.pp
>  @@ -2,6 +2,11 @@
>
>    class aeolus::image-factory inherits aeolus {
>        if $enable_packages {
>  +
>  +      package { 'libvirt':
>  +                provider =>  'yum',
>  +                ensure=>  'installed'
>  +      }
>          package { 'imagefactory':
>                       provider =>  'yum', ensure =>  'installed',
>                       require =>  [Yumrepo['aeolus_arch',
> 'aeolus_noarch']]
>  @@ -31,7 +36,8 @@ class aeolus::image-factory inherits aeolus {
>        service {'libvirtd':
>                   ensure  =>  'running',
>                   enable  =>  true,
>  -               hasstatus =>  true }
>  +               hasstatus =>  true,
>  +               require =>  return_if($enable_packages, Package['libvirt'])}
>        $requires = [return_if($enable_packages, Package['imagefactory']),
>                     File['/var/tmp/imagefactory-mock'],
>                     Service[qpidd], Service[libvirtd],

[Mike Orazi]

Pushed

----- Original Message -----
>  ACK
>  
>  On 31/03/11 06:15, Mike Orazi wrote:
>  > ---
>  >   recipes/aeolus_recipe/manifests/image-factory.pp | 8 +++++++-
>  >   1 files changed, 7 insertions(+), 1 deletions(-)
>  >
>  > diff --git a/recipes/aeolus_recipe/manifests/image-factory.pp
>  > b/recipes/aeolus_recipe/manifests/image-factory.pp
>  > index 499b6e4..5bcd24f 100644
>  > --- a/recipes/aeolus_recipe/manifests/image-factory.pp
>  > +++ b/recipes/aeolus_recipe/manifests/image-factory.pp
>  > @@ -2,6 +2,11 @@
>  >
>  >   class aeolus::image-factory inherits aeolus {
>  >       if $enable_packages {
>  > +
>  > + package { 'libvirt':
>  > + provider => 'yum',
>  > + ensure=> 'installed'
>  > + }
>  >         package { 'imagefactory':
>  >                      provider => 'yum', ensure =>
> 'installed',
>  >                      require => [Yumrepo['aeolus_arch',
>  >                      'aeolus_noarch']]
>  > @@ -31,7 +36,8 @@ class aeolus::image-factory inherits aeolus {
>  >       service {'libvirtd':
>  >                  ensure => 'running',
>  >                  enable => true,
>  > - hasstatus => true }
>  > + hasstatus => true,
>  > + require => return_if($enable_packages, Package['libvirt'])}
>  >       $requires = [return_if($enable_packages,
>  >       Package['imagefactory']),
>  >                    File['/var/tmp/imagefactory-mock'],
>  >                    Service[qpidd], Service[libvirtd],
>  
>  _______________________________________________
>  aeolus-devel mailing list
>  aeolus-devel(a)lists.fedorahosted.org
>  https://fedorahosted.org/mailman/listinfo/aeolus-devel