Add more logging for service level failures

pyninja/service.py

@@ -1,9 +1,12 @@
+import logging
 import platform
 import subprocess
 from http import HTTPStatus
 
 from pyninja import exceptions, models
 
+LOGGER = logging.getLogger("uvicorn.default")
+
 current_os = platform.system()
 
 if current_os not in ("Darwin", "Linux", "Windows"):
@@ -54,7 +57,8 @@ def get_service_status(service_name: str) -> models.ServiceStatus:
                 return stopped
             else:
                 return unknown
-        except subprocess.CalledProcessError:
+        except subprocess.CalledProcessError as error:
+            LOGGER.error("%d - %s", 404, error)
             return unavailable
 
     if current_os == "Linux":
@@ -71,7 +75,8 @@ def get_service_status(service_name: str) -> models.ServiceStatus:
                     status_code=HTTPStatus.NOT_IMPLEMENTED.real,
                     description=f"{service_name} - {output}",
                 )
-        except subprocess.CalledProcessError:
+        except subprocess.CalledProcessError as error:
+            LOGGER.error("%d - %s", 404, error)
             return unavailable
 
     if current_os == "Darwin":
@@ -83,5 +88,6 @@ def get_service_status(service_name: str) -> models.ServiceStatus:
                 return running
             else:
                 return stopped
-        except subprocess.CalledProcessError:
+        except subprocess.CalledProcessError as error:
+            LOGGER.error("%d - %s", 404, error)
             return unavailable