From 3d29277285d05d2918c957d73249bb445d586f61 Mon Sep 17 00:00:00 2001 From: Michael Jerger Date: Fri, 21 Apr 2023 14:20:03 +0200 Subject: [PATCH] tried to get debugg outputs --- .../infrastructure/release_mixin/infrastructure_api.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/src/main/python/ddadevops/infrastructure/release_mixin/infrastructure_api.py b/src/main/python/ddadevops/infrastructure/release_mixin/infrastructure_api.py index d4c8ca1..57cc506 100644 --- a/src/main/python/ddadevops/infrastructure/release_mixin/infrastructure_api.py +++ b/src/main/python/ddadevops/infrastructure/release_mixin/infrastructure_api.py @@ -5,6 +5,7 @@ from abc import ABC, abstractmethod from typing import Optional, Union from pathlib import Path from os import environ +import logging # TODO: jem, zam - 2023_04_18: Discuss if we can move more functionality to domain? class FileHandler(ABC): @@ -185,6 +186,7 @@ class SystemApi(): def __init__(self): self.stdout = [""] self.stderr = [""] + self.exitcode = 0 def run(self, args: list[str]): sanitized_args = [] @@ -206,9 +208,14 @@ class SystemApi(): else: self.stderr = None + self.exitcode = stream.returncode + def run_checked(self, *args): self.run(args) + logging.warning(f"err: {self.stderr}") + logging.warning(f"exit: {self.exitcode}") + if len(self.stderr) > 0: raise Exception(f"Command failed with: {self.stderr}")