fix checking for container_engine unlike docker on the platform

src/sophios/run_local.py

@@ -63,47 +63,69 @@ def run_local(args: argparse.Namespace, rose_tree: RoseTree, cachedir: Optional[
     Returns:
         retval: The return value
     """
-
+    docker_like_engines = ['docker', 'podman']
     docker_cmd: str = args.container_engine
     # Check that docker is installed, so users don't get a nasty runtime error.
-    cmd = [docker_cmd, 'run', '--rm', 'hello-world']
-    output = ''
-    try:
-        docker_cmd_exists = True
-        proc = sub.run(cmd, check=False, stdout=sub.PIPE, stderr=sub.STDOUT)
-        output = proc.stdout.decode("utf-8")
-    except FileNotFoundError:
-        docker_cmd_exists = False
-    out_d = "Hello from Docker!"
-    out_p = "Hello Podman World"
-    permission_denied = 'permission denied while trying to connect to the Docker daemon socket at'
-    if (not docker_cmd_exists or not (proc.returncode == 0 and out_d in output or out_p in output)) and not args.ignore_docker_install:
-        if permission_denied in output:
-            print('Warning! docker appears to be installed, but not configured as a non-root user.')
-            print('See https://docs.docker.com/engine/install/linux-postinstall/#manage-docker-as-a-non-root-user')
-            print('TL;DR you probably just need to run the following command (and then restart your machine)')
-            print('sudo usermod -aG docker $USER')
+    if docker_cmd in docker_like_engines:
+        cmd = [docker_cmd, 'run', '--rm', 'hello-world']
+        output = ''
+        try:
+            docker_cmd_exists = True
+            proc = sub.run(cmd, check=False, stdout=sub.PIPE, stderr=sub.STDOUT)
+            output = proc.stdout.decode("utf-8")
+        except FileNotFoundError:
+            docker_cmd_exists = False
+        out_d = "Hello from Docker!"
+        out_p = "Hello Podman World"
+        permission_denied = 'permission denied while trying to connect to the Docker daemon socket at'
+        if ((not docker_cmd_exists
+            or not (proc.returncode == 0 and out_d in output or out_p in output))
+                and not args.ignore_docker_install):
+
+            if permission_denied in output:
+                print('Warning! docker appears to be installed, but not configured as a non-root user.')
+                print('See https://docs.docker.com/engine/install/linux-postinstall/#manage-docker-as-a-non-root-user')
+                print('TL;DR you probably just need to run the following command (and then restart your machine)')
+                print('sudo usermod -aG docker $USER')
+                sys.exit(1)
+
+            print(f'Warning! The {docker_cmd} command does not appear to be installed.')
+            print(f"""Most workflows require docker containers and
+                  will fail at runtime if {docker_cmd} is not installed.""")
+            print('If you want to try running the workflow anyway, use --ignore_docker_install')
+            print("""Note that --ignore_docker_install does
+                  NOT change whether or not any step in your workflow uses docker""")
             sys.exit(1)
-        print(f'Warning! The {docker_cmd} command does not appear to be installed.')
-        print(f'Most workflows require docker containers and will fail at runtime if {docker_cmd} is not installed.')
-        print('If you want to try running the workflow anyway, use --ignore_docker_install')
-        print('Note that --ignore_docker_install does NOT change whether or not any step in your workflow uses docker')
-        sys.exit(1)
-
-    # If docker is installed, check for too many running processes. (on linux, macos)
-    if docker_cmd == 'docker' and docker_cmd_exists and sys.platform != "win32":
-        cmd = 'pgrep com.docker | wc -l'  # type: ignore
-        proc = sub.run(cmd, check=False, stdout=sub.PIPE, stderr=sub.STDOUT, shell=True)
-        output = proc.stdout.decode("utf-8")
-        num_processes = int(output.strip())
-        max_processes = 1000
-        if num_processes > max_processes and not args.ignore_docker_processes:
-            print(f'Warning! There are {num_processes} running docker processes.')
-            print(f'More than {max_processes} may potentially cause intermittent hanging issues.')
-            print('It is recommended to terminate the processes using the command')
-            print('`sudo pkill com.docker && sudo pkill Docker`')
-            print('and then restart Docker.')
-            print('If you want to run the workflow anyway, use --ignore_docker_processes')
+
+        # If docker is installed, check for too many running processes. (on linux, macos)
+        if docker_cmd_exists and sys.platform != "win32":
+            cmd = 'pgrep com.docker | wc -l'  # type: ignore
+            proc = sub.run(cmd, check=False, stdout=sub.PIPE, stderr=sub.STDOUT, shell=True)
+            output = proc.stdout.decode("utf-8")
+            num_processes = int(output.strip())
+            max_processes = 1000
+            if num_processes > max_processes and not args.ignore_docker_processes:
+                print(f'Warning! There are {num_processes} running docker processes.')
+                print(f'More than {max_processes} may potentially cause intermittent hanging issues.')
+                print('It is recommended to terminate the processes using the command')
+                print('`sudo pkill com.docker && sudo pkill Docker`')
+                print('and then restart Docker.')
+                print('If you want to run the workflow anyway, use --ignore_docker_processes')
+                sys.exit(1)
+    else:
+        cmd = [docker_cmd, '--version']
+        output = ''
+        try:
+            docker_cmd_exists = True
+            proc = sub.run(cmd, check=False, stdout=sub.PIPE, stderr=sub.STDOUT)
+            output = proc.stdout.decode("utf-8")
+        except FileNotFoundError:
+            docker_cmd_exists = False
+        if not docker_cmd_exists and not args.ignore_docker_install:
+            print(f'Warning! The {docker_cmd} command does not appear to be installed.')
+            print('If you want to try running the workflow anyway, use --ignore_docker_install')
+            print('Note that --ignore_docker_install does NOT change whether or not')
+            print('any step in your workflow uses docker or any other containers')
             sys.exit(1)
 
     yaml_path = args.yaml