[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