udocker: use --nobanner as well (#1761)

Author: mr-c

cwltool/docker.py

@@ -334,10 +334,11 @@ def create_runtime(
         any_path_okay = self.builder.get_requirement("DockerRequirement")[1] or False
         user_space_docker_cmd = runtimeContext.user_space_docker_cmd
         if user_space_docker_cmd:
-            if "udocker" in user_space_docker_cmd and not runtimeContext.debug:
-                runtime = [user_space_docker_cmd, "--quiet", "run"]
-                # udocker 1.1.1 will output diagnostic messages to stdout
-                # without this
+            if "udocker" in user_space_docker_cmd:
+                if runtimeContext.debug:
+                    runtime = [user_space_docker_cmd, "run", "--nobanner"]
+                else:
+                    runtime = [user_space_docker_cmd, "--quiet", "run", "--nobanner"]
             else:
                 runtime = [user_space_docker_cmd, "run"]
         elif runtimeContext.podman:

tests/test_udocker.py

@@ -89,3 +89,19 @@ def test_udocker_should_display_memory_usage(udocker: str, tmp_path: Path) -> No
 
     assert "completed success" in stderr, stderr
     assert "Max memory" in stderr, stderr
+
+
+@pytest.mark.skipif(not LINUX, reason="LINUX only")
+def test_udocker_nobanner(udocker: str, tmp_path: Path) -> None:
+    """Avoid the banner when running udocker."""
+    with working_directory(tmp_path):
+        error_code, stdout, stderr = get_main_output(
+            [
+                "--user-space-docker-cmd=" + udocker,
+                get_data("tests/wf/cat-tool.cwl"),
+                get_data("tests/wf/wc-job.json"),
+            ]
+        )
+
+    assert "completed success" in stderr, stderr
+    assert "sha1$327fc7aedf4f6b69a42a7c8b808dc5a7aff61376" in stdout, stdout

tests/wf/cat-tool.cwl

@@ -5,7 +5,7 @@ cwlVersion: v1.0
 
 hints:
   DockerRequirement:
-    dockerPull: docker.io/bash:4.4`
+    dockerPull: docker.io/bash:4.4
 
 inputs:
   file1: File