daandemeyer / rpms / systemd

Forked from rpms/systemd 2 years ago
Clone
1abbee
From 94fec84897ab40bf2bc92f0d395a93ecac1b45be Mon Sep 17 00:00:00 2001
1abbee
From: =?UTF-8?q?Luk=C3=A1=C5=A1=20Nykr=C3=BDn?= <lnykryn@redhat.com>
1abbee
Date: Wed, 3 Aug 2016 17:08:37 +0200
1abbee
Subject: [PATCH] systemctl: consider service running only when it is in active
1abbee
 or reloading state (#3874)
1abbee
1abbee
Otherwise for example services that are failing on start and have Restart=on-failure
1abbee
and bigger RestartSec systemctl status will return 0.
1abbee
1abbee
Fixes: #3864
1abbee
Cherry-picked from: 7f5da8bd4fb1ba49ba40195a74ca76bb5d4d1f81
1abbee
Resolves: #1362461
1abbee
---
1abbee
 src/systemctl/systemctl.c | 2 +-
1abbee
 1 file changed, 1 insertion(+), 1 deletion(-)
1abbee
1abbee
diff --git a/src/systemctl/systemctl.c b/src/systemctl/systemctl.c
181b3f
index 93b7a193b..b7496c006 100644
1abbee
--- a/src/systemctl/systemctl.c
1abbee
+++ b/src/systemctl/systemctl.c
1abbee
@@ -4339,7 +4339,7 @@ static int show_one(
1abbee
         else if (streq(verb, "status")) {
1abbee
                 print_status_info(&info, ellipsized);
1abbee
 
1abbee
-                if (info.active_state && STR_IN_SET(info.active_state, "inactive", "failed"))
1abbee
+                if (info.active_state && !STR_IN_SET(info.active_state, "active", "reloading"))
1abbee
                         r = EXIT_PROGRAM_NOT_RUNNING;
1abbee
                 else
1abbee
                         r = EXIT_PROGRAM_RUNNING_OR_SERVICE_OK;