Re: [PATCH configure] Start/stop script for warehouse sync

[Mike Orazi]

On 03/23/2011 07:01 AM, [e-mail] wrote:
> From: Jan Provaznik
>
> recipes/aeolus_recipe/manifests/conductor.pp |    5 +++++
>   1 files changed, 5 insertions(+), 0 deletions(-)
>
> diff --git a/recipes/aeolus_recipe/manifests/conductor.pp b/recipes/aeolus_recipe/manifests/conductor.pp
> index b6c4c67..f6d9f90 100644
> --- a/recipes/aeolus_recipe/manifests/conductor.pp
> +++ b/recipes/aeolus_recipe/manifests/conductor.pp
> @@ -41,6 +41,7 @@ class aeolus::conductor inherits aeolus {
>         require =>  File['/var/lib/condor/condor_config.local'] }
>       service { ['aeolus-conductor',
>                  'conductor-condor_refreshd',
>           'conductor-warehouse_sync',
>            'conductor-dbomatic',
>            'conductor-delayed_job']:
>   ensure    =>  'running',
>
> @@ -180,6 +181,7 @@ class aeolus::conductor::disabled {
>                   provider =>  'yum', ensure =>  'absent',
>                   require  =>  Service['aeolus-conductor',
>                                       'conductor-condor_refreshd',
>                                'conductor-warehouse_sync',
>                                 'conductor-dbomatic',
>                                 'conductor-delayed_job']}
>
> @@ -206,10 +208,12 @@ class aeolus::conductor::disabled {
>         enable  =>  false,
>         require =>  Service['aeolus-conductor',
>                            'conductor-condor_refreshd',
>                     'conductor-warehouse_sync',
>                      'conductor-dbomatic',
>                      'conductor-delayed_job'] }
> service { ['aeolus-conductor',
>            'conductor-condor_refreshd',
>           'conductor-warehouse_sync',
>            'conductor-dbomatic',
>            'conductor-delayed_job',
>            'aeolus-connector']:
>
> @@ -223,6 +227,7 @@ class aeolus::conductor::disabled {
>                   rails_env  =>  "production",
>                   require    =>  Service["aeolus-conductor",
>                                         "conductor-condor_refreshd",
>                                  'conductor-warehouse_sync',
>                                   "conductor-dbomatic",
>                                   "conductor-delayed_job"]}
> postgres::user{"aeolus":

ACK & pushed on this portion.

Note:  I don't think Tomas/Jan have push rights to the configure repo. 
I DID NOT verify the broader warehouse is working, but verified that the 
service is registered to auto-start on server start up via chkconfig in 
the configure recipes and the conductor-warehouse_sync service is 
started as expected.

Since the whole patchset was already acked, I am assuming I don't need 
to verify functionality of warehouse_sync itself just that services are 
in the correct state.

Mike