[PATCH conductor] Bug 852866 - aeolus-upgrade hits "aeolus-services: unrecognized service"

John Eckersberg jeckersb at redhat.com
Wed Sep 5 20:31:00 UTC 2012


Steve Linabery <slinaber at redhat.com> writes:

> aeolus-upgrade script previously called 'service aeolus-services stop'
> to stop services, but aeolus-services is not a system service. This was
> silently failing to stop services on f16/f17, but rhel more sensibly
> reports the failure which uncovered the issue. This patch
> changes the stop method to call aeolus-services directly.
> ---
>  src/script/aeolus-upgrade |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/script/aeolus-upgrade b/src/script/aeolus-upgrade
> index df89c31..2fe8d13 100755
> --- a/src/script/aeolus-upgrade
> +++ b/src/script/aeolus-upgrade
> @@ -296,7 +296,7 @@ class UpgradeProcess
>        else
>          puts "This script can be run in auto-stop mode with the -a / --auto-stop option to"
>          puts "disable services automatically."
> -        puts "You may always use `service aeolus-services stop` to stop all the services"
> +        puts "You may always use `aeolus-services stop` to stop all the services"
>          puts ""
>          exit_service_stop
>        end
> @@ -419,7 +419,7 @@ end
>  # system call fails. non catastrophic errors, such as the service
>  # already being off
>  def stop(service)
> -  return system('/sbin/service %s stop' % service)
> +  return system('%s stop' % service)
>  end
>  
>  # Parse and return script options
> -- 
> 1.7.7.6

ACK and pushed.  Tested on both f17 and el6, works as advertised!



More information about the aeolus-devel mailing list