[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],

[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
> [e-mail]
> https://fedorahosted.org/mailman/listinfo/aeolus-devel