Skip to content

Commit d41d977

Browse files
authored
Merge pull request #85 from burtenshaw/expose-container-logs
[ENHANCEMENT] expose raised errors from container subprocess
2 parents 4d0e5bb + a8fa54d commit d41d977

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

src/core/containers/runtime/providers.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -169,8 +169,12 @@ def start_container(
169169
cmd.append(image)
170170

171171
# Run container
172-
result = subprocess.run(cmd, capture_output=True, text=True, check=True)
173-
self._container_id = result.stdout.strip()
172+
try:
173+
result = subprocess.run(cmd, capture_output=True, text=True, check=True)
174+
self._container_id = result.stdout.strip()
175+
except subprocess.CalledProcessError as e:
176+
error_msg = f"Failed to start Docker container.\nCommand: {' '.join(cmd)}\nExit code: {e.returncode}\nStderr: {e.stderr}\nStdout: {e.stdout}"
177+
raise RuntimeError(error_msg) from e
174178

175179
# Wait a moment for container to start
176180
time.sleep(1)

0 commit comments

Comments
 (0)