Steve Linabery <[email protected]> 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!
