[PATCH configure] add libvirtd service dependency to imagefactory

[Mo Morsi]

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

diff --git a/recipes/aeolus_recipe/manifests/image-factory.pp b/recipes/aeolus_recipe/manifests/image-factory.pp
index cb05c59..499b6e4 100644
--- a/recipes/aeolus_recipe/manifests/image-factory.pp
+++ b/recipes/aeolus_recipe/manifests/image-factory.pp
@@ -28,9 +28,13 @@ class aeolus::image-factory inherits aeolus {
     file { "/var/tmp/imagefactory-mock":
                ensure => "directory",
                mode   => 755 }
+    service {'libvirtd':
+               ensure  => 'running',
+               enable  => true,
+               hasstatus => true }
     $requires = [return_if($enable_packages, Package['imagefactory']),
                  File['/var/tmp/imagefactory-mock'],
-                 Service[qpidd],
+                 Service[qpidd], Service[libvirtd],
                  Rails::Seed::Db[seed_aeolus_database]]
     service { 'imagefactory':
       ensure  => 'running',

[Richard Su]

On 03/24/2011 02:05 PM, Mohammed Morsi wrote:
> recipes/aeolus_recipe/manifests/image-factory.pp |    6 +++++-
>   1 files changed, 5 insertions(+), 1 deletions(-)
>
> diff --git a/recipes/aeolus_recipe/manifests/image-factory.pp b/recipes/aeolus_recipe/manifests/image-factory.pp
> index cb05c59..499b6e4 100644
> --- a/recipes/aeolus_recipe/manifests/image-factory.pp
> +++ b/recipes/aeolus_recipe/manifests/image-factory.pp
> @@ -28,9 +28,13 @@ class aeolus::image-factory inherits aeolus {
>       file { "/var/tmp/imagefactory-mock":
>                  ensure =>  "directory",
>                  mode   =>  755 }
> service {'libvirtd':
>           ensure  =>  'running',
>           enable  =>  true,
>           hasstatus =>  true }
> $requires = [return_if($enable_packages, Package['imagefactory']),
>              File['/var/tmp/imagefactory-mock'],
>             Service[qpidd],
>             Service[qpidd], Service[libvirtd],
>              Rails::Seed::Db[seed_aeolus_database]]
> service { 'imagefactory':
>   ensure  =>  'running',

Ack. I see libvirtd is brought up before imagefactory if it was down.

On a side note, imagefactory sometimes does not come up cleanly even 
though configure says its state was changed to running.