Merge pull request #168 from Project-MONAI/build_with_uid

Build package with local UID/GID and fast-retrieve app/package information

Author: gigony

monai/deploy/packager/util.py

@@ -229,10 +229,12 @@ def build_image(args: dict, temp_dir: str):
         dockerignore_file.write(docker_ignore_template)
 
     # Build dockerfile into an MAP image
-    docker_build_cmd = ["docker", "build", "-f", docker_file_path, "-t", tag, temp_dir]
+    docker_build_cmd = f'''docker build -f "{docker_file_path}" -t {tag} "{temp_dir}"'''
+    if sys.platform != "win32":
+        docker_build_cmd += """ --build-arg MONAI_UID=$(id -u) --build-arg MONAI_GID=$(id -g)"""
     if no_cache:
-        docker_build_cmd.append("--no-cache")
-    proc = subprocess.Popen(docker_build_cmd, stdout=subprocess.PIPE)
+        docker_build_cmd += " --no-cache"
+    proc = subprocess.Popen(docker_build_cmd, stdout=subprocess.PIPE, shell=True)
 
     spinner = ProgressSpinner("Building MONAI Application Package... ")
     spinner.start()

monai/deploy/runner/runner.py

@@ -39,8 +39,14 @@ def fetch_map_manifest(map_name: str) -> Tuple[dict, dict, int]:
     logger.info("\nReading MONAI App Package manifest...")
 
     with tempfile.TemporaryDirectory() as info_dir:
-        cmd = f"docker run --rm -a STDOUT -a STDERR -v {info_dir}:/var/run/monai/export/config {map_name}"
-
+        if sys.platform == "win32":
+            cmd = f'docker run --rm -a STDOUT -a STDERR -v "{info_dir}":/var/run/monai/export/config {map_name}'
+        else:
+            cmd = f"""docker_id=$(docker create {map_name})
+docker cp $docker_id:/etc/monai/app.json "{info_dir}/app.json"
+docker cp $docker_id:/etc/monai/pkg.json "{info_dir}/pkg.json"
+docker rm -v $docker_id > /dev/null
+"""
         returncode = run_cmd(cmd)
         if returncode != 0:
             return {}, {}, returncode