Change from --volume to --mount

Author: mccalluc

cwltool/docker.py

@@ -1,14 +1,15 @@
 """Enables Docker software containers via the {dx-,u,}docker runtimes."""
 
 import datetime
+import csv
 import os
 import re
 import shutil
 import sys
 import tempfile
 import threading
 from distutils import spawn
-from io import open  # pylint: disable=redefined-builtin
+from io import open, StringIO  # pylint: disable=redefined-builtin
 from typing import Dict, List, MutableMapping, Optional, Set, Tuple
 
 import requests
@@ -222,10 +223,18 @@ def get_from_requirements(
     def append_volume(runtime, source, target, writable=False):
         # type: (List[str], str, str, bool) -> None
         """Add binding arguments to the runtime list."""
+        options = [
+            'type=bind',
+            'source=' + source,
+            'target=' + target,
+        ]
+        if not writable:
+            options.append('readonly')
+        output = StringIO()
+        csv.writer(output).writerow(options)
+        mount_arg = output.getvalue().strip()
         runtime.append(
-            "--volume={}:{}:{}".format(
-                docker_windows_path_adjust(source), target, "rw" if writable else "ro"
-            )
+            "--mount={}".format(mount_arg)
         )
 
     def add_file_or_directory_volume(

tests/test_docker_paths_with_colons.py

@@ -0,0 +1,39 @@
+import pytest
+
+from cwltool.docker import DockerCommandLineJob
+from cwltool.main import main
+
+from .util import needs_docker
+
+
+def test_docker_append_volume_read_only():
+    runtime = ['runtime']
+    characters = ':,"\''
+    DockerCommandLineJob.append_volume(
+        runtime,
+        '/source' + characters,
+        '/target' + characters
+    )
+    assert runtime == [
+        'runtime',
+        '--mount=type=bind,'
+        '"source=/source:,""\'",'
+        '"target=/target:,""\'",'
+        'readonly'
+    ]
+
+def test_docker_append_volume_read_write():
+    runtime = ['runtime']
+    characters = ':,"\''
+    DockerCommandLineJob.append_volume(
+        runtime,
+        '/source' + characters,
+        '/target' + characters,
+        True
+    )
+    assert runtime == [
+        'runtime',
+        '--mount=type=bind,'
+        '"source=/source:,""\'",'
+        '"target=/target:,""\'"'
+    ]