From 40f1cb2e7e597f6b51feee5fb68d0aef5d1af7e3 Mon Sep 17 00:00:00 2001 From: dafeliton Date: Tue, 5 Mar 2024 16:41:04 -0800 Subject: [PATCH] add docker error handling, try full build --- scripts/docker_adapter.py | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/scripts/docker_adapter.py b/scripts/docker_adapter.py index c8629ac1..994feffe 100644 --- a/scripts/docker_adapter.py +++ b/scripts/docker_adapter.py @@ -81,12 +81,18 @@ def build(node: Node) -> Tuple[bool, str]: report += content_str + '\n' # Error detection. The docker client is not throwing errors if the build fails. - # These errors are caught by our tests unless we scan these lines manually (not a fan of this). + # These errors are caught by our tests unless we scan these lines manually (not a big fan of this). error_patterns = { 'apt': re.compile(r'\x1b\[91mE:'), 'pip': re.compile(r'\x1b\[91mERROR:'), 'conda': re.compile(r'Solving environment: failed'), 'mamba': re.compile(r'Could not solve for environment specs'), + 'cp': re.compile(r'cp: '), + 'mv': re.compile(r'mv: '), + 'ln': re.compile(r'ln: '), + 'mkdir': re.compile(r'mkdir: '), + 'chmod': re.compile(r'chmod: '), + 'chown': re.compile(r'chown: '), } for key, val in error_patterns.items(): if val.search(content_str):