[prev in list] [next in list] [prev in thread] [next in thread] 

List:       fedora-directory-commits
Subject:    [389-commits] wrappers/initscript.in
From:       Mark Reynolds <mreynolds () fedoraproject ! org>
Date:       2012-09-24 17:29:44
Message-ID: 20120924172944.97089A1605 () fedorahosted ! org
[Download RAW message or body]

 wrappers/initscript.in |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit ef48c93ded0f766d8dab679b976ca032d6297c32
Author: Mark Reynolds <mreynolds@redhat.com>
Date:   Mon Sep 24 12:34:59 2012 -0400

    Ticket 457 - dirsrv init script returns 0 even when few or all instances fail to start
    
    Bug Description:  We don't return an error code when one or more instances fails to start.
    
    Fix Description:  Return error 1 when an instance fails to start.
    
    https://fedorahosted.org/389/ticket/457
    
    Reviewed by: richm(Thanks!)

diff --git a/wrappers/initscript.in b/wrappers/initscript.in
index da5f6bb..7601784 100644
--- a/wrappers/initscript.in
+++ b/wrappers/initscript.in
@@ -264,7 +264,8 @@ start() {
         [ -x /sbin/restorecon ] && /sbin/restorecon $lockfile
     fi
     if [ $errors -ge 1 ]; then
-        echo "  *** Warning: $errors instance(s) failed to start"
+        echo "  *** Error: $errors instance(s) failed to start"
+        exit 1
     fi
 }
 


--
389 commits mailing list
389-commits@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/389-commits
[prev in list] [next in list] [prev in thread] [next in thread]