feat: raise exception when docker version is below 20.0.0 (#2895)

jina/excepts.py

@@ -122,3 +122,7 @@ class RoutingTableCyclicError(Exception):
 
 class RuntimeRunForeverEarlyError(Exception):
     """Raised when an error occurs when starting the run_forever of Runtime"""
+
+
+class DockerVersionError(SystemError):
+    """Raised when the docker version is incompatible"""

jina/peapods/runtimes/container/__init__.py

@@ -9,7 +9,7 @@
 from ..zmq.base import ZMQRuntime
 from ...zmq import Zmqlet
 from .... import __docker_host__
-from ....excepts import BadImageNameError
+from ....excepts import BadImageNameError, DockerVersionError
 from ...zmq import send_ctrl_message
 from ....helper import ArgNamespace, slugify
 
@@ -81,6 +81,18 @@ def _docker_run(self, replay: bool = False):
 
         client = docker.from_env()
 
+        docker_version = client.version().get('Version')
+        if not docker_version:
+            raise DockerVersionError('docker version can not be resolved')
+
+        docker_version = tuple(docker_version.split('.'))
+        # docker daemon versions below 20.0x do not support "host.docker.internal:host-gateway"
+        if docker_version < ('20',):
+            raise DockerVersionError(
+                f'docker version {".".join(docker_version)} is below 20.0.0 and does not '
+                f'support "host.docker.internal:host-gateway" : https://github.com/docker/cli/issues/2664'
+            )
+
         if self.args.uses.startswith('docker://'):
             uses_img = self.args.uses.replace('docker://', '')
             self.logger.debug(f'will use Docker image: {uses_img}')

tests/unit/peapods/runtimes/container/test_container_runtime.py

@@ -251,6 +251,9 @@ def __init__(self, *args, **kwargs):
         def close(self):
             pass
 
+        def version(self):
+            return {'Version': '20.0.1'}
+
         @property
         def networks(self):
             return {'bridge': None}