Fix py3 comparison issue(None with integers) in pid method and

Fix split issue which creates empty string value due to newline character at
end of command response.

Signed-off-by: sunilkumarn417 <sunnagar@redhat.com>

diff --git a/teuthology/orchestra/daemon/systemd.py b/teuthology/orchestra/daemon/systemd.py
index 03d00eca825f5725ea065645d0e0f58ec1c62449..204f9422c698fee01eca1a72fc50ec3574f6507e 100644 (file)
--- a/teuthology/orchestra/daemon/systemd.py
+++ b/teuthology/orchestra/daemon/systemd.py
@@ -85,7 +85,7 @@ class SystemDState(DaemonState):
             #    Main process exited, code=exited, status=1/FAILURE
             self.status_cmd + " | grep 'Main.*code=exited'",
         )
-        line = out.split('\n')[-1]
+        line = out.strip().split('\n')[-1]
         exit_code = int(re.match('.*status=(\d+).*', line).groups()[0])
         if exit_code:
             self.remote.run(
@@ -167,10 +167,12 @@ class SystemDState(DaemonState):
         :return: The PID if remote run command value is set, False otherwise.
         """
         pid = self.pid
-        if pid > 0:
-            return pid
-        else:
+        if pid is None:
+            return None
+        elif pid <= 0:
             return None
+        else:
+            return pid
 
     def signal(self, sig, silent=False):
         """