When docker is unavailable and marked as required, exit accordingly

Author: mr-c

cwltool/job.py

@@ -18,7 +18,8 @@
 from .docker_uid import docker_vm_uid
 from .errors import WorkflowException
 from .pathmapper import PathMapper
-from .process import get_feature, empty_subtree, stageFiles
+from .process import (get_feature, empty_subtree, stageFiles,
+                      UnsupportedRequirement)
 
 _logger = logging.getLogger("cwltool")
 
@@ -335,7 +336,8 @@ def run(self, pull_image=True, rm_container=True,
         except Exception as e:
             _logger.debug("Docker error", exc_info=True)
             if docker_is_req:
-                raise WorkflowException("Docker is required to run this tool: %s" % e)
+                raise UnsupportedRequirement(
+                        "Docker is required to run this tool: %s" % e)
             else:
                 raise WorkflowException("Docker is not available for this tool, try --no-container to disable Docker: %s" % e)
 

cwltool/process.py

@@ -530,11 +530,6 @@ def _init_job(self, joborder, **kwargs):
         builder.mutation_manager = kwargs.get("mutation_manager")
 
         dockerReq, is_req = self.get_requirement("DockerRequirement")
-
-        if dockerReq and is_req and not kwargs.get("use_container"):
-            raise WorkflowException(
-                "Document has DockerRequirement under 'requirements' but use_container is false.  DockerRequirement must be under 'hints' or use_container must be true.")
-
         builder.make_fs_access = kwargs.get("make_fs_access") or StdFsAccess
         builder.fs_access = builder.make_fs_access(kwargs["basedir"])