Add handling of stderr being None in the scheduler

This commit is contained in:
Théophile Diot 2023-05-22 12:38:28 -04:00
parent 7506768c4a
commit 0b082bdab7
No known key found for this signature in database
GPG Key ID: E752C80DB72BB014
2 changed files with 4 additions and 4 deletions

View File

@ -141,7 +141,7 @@ class JobScheduler(ApiCaller):
self.__logger.info("Successfully reloaded nginx")
else:
self.__logger.error(
f"Error while reloading nginx - returncode: {proc.returncode} - error: {proc.stderr.decode()}",
f"Error while reloading nginx - returncode: {proc.returncode} - error: {proc.stderr.decode() if proc.stderr else 'Missing stderr'}",
)
else:
self.__logger.info("Reloading nginx ...")

View File

@ -462,11 +462,11 @@ if __name__ == "__main__":
logger.info("Successfully started nginx")
else:
logger.error(
f"Error while starting nginx - returncode: {proc.returncode} - error: {proc.stderr.decode('utf-8')}",
f"Error while starting nginx - returncode: {proc.returncode} - error: {proc.stderr.decode('utf-8') if proc.stderr else 'Missing stderr'}",
)
else:
logger.error(
f"Error while sending stop signal to temp nginx - returncode: {proc.returncode} - error: {proc.stderr.decode('utf-8')}",
f"Error while sending stop signal to temp nginx - returncode: {proc.returncode} - error: {proc.stderr.decode('utf-8') if proc.stderr else 'Missing stderr'}",
)
else:
if api_caller._send_to_apis("POST", "/reload"):
@ -529,7 +529,7 @@ if __name__ == "__main__":
logger.info("Successfully reloaded nginx")
else:
logger.error(
f"Error while reloading nginx - returncode: {proc.returncode} - error: {proc.stderr.decode('utf-8')}",
f"Error while reloading nginx - returncode: {proc.returncode} - error: {proc.stderr.decode('utf-8') if proc.stderr else 'Missing stderr'}",
)
else:
need_reload = True