Merge pull request #765 from timocov/devel

Added handling `pid` option

podman_compose.py

@@ -981,6 +981,8 @@ async def container_to_args(compose, cnt, detached=True):
  podman_args.append("--tty")
  if cnt.get("privileged", None):
  podman_args.append("--privileged")
+ if cnt.get("pid", None):
+ podman_args.extend(["--pid", cnt["pid"]])
  pull_policy = cnt.get("pull_policy", None)
  if pull_policy is not None and pull_policy != "build":
  podman_args.extend(["--pull", pull_policy])

pytests/test_container_to_args.py

@@ -130,3 +130,25 @@ async def test_sysctl_wrong_type(self):
  with self.assertRaises(TypeError):
  cnt["sysctls"] = wrong_type
  await container_to_args(c, cnt)
+
+ async def test_pid(self):
+ c = create_compose_mock()
+ cnt = get_minimal_container()
+
+ cnt["pid"] = "host"
+
+ args = await container_to_args(c, cnt)
+ self.assertEqual(
+ args,
+ [
+ "--name=project_name_service_name1",
+ "-d",
+ "--net",
+ "",
+ "--network-alias",
+ "service_name",
+ "--pid",
+ "host",
+ "busybox",
+ ],
+ )

tests/pid/docker-compose.yml

@@ -0,0 +1,6 @@
+version: "3"
+services:
+ serv:
+ image: busybox
+ pid: host
+ command: sh -c "ps all"