Merge pull request #339 from umohnani8/docfixes-2

Fix readthedocs yaml

.readthedocs.yaml

@@ -1,15 +1,26 @@
+# Read the Docs configuration file
+# See https://docs.readthedocs.io/en/stable/config-file/v2.html for details
+
+# Required
 version: 2
 
 formats:
   - pdf
   - epub
 
+# Set the version of Python and other tools you might need
+build:
+  os: ubuntu-22.04
+  tools:
+    python: "3.11"
+
+# We recommend specifying your dependencies to enable reproducible builds:
+# https://docs.readthedocs.io/en/stable/guides/reproducible-builds.html
 python:
-  version: "3.6"
   install:
-    - requirements: requirements.txt
+  - requirements: requirements.txt
 
+# Build documentation in the docs/ directory with Sphinx
 sphinx:
   configuration: docs/source/conf.py
   fail_on_warning: false
-

podman/domain/containers_create.py

@@ -657,13 +657,11 @@ def parse_host_port(_container_port, _protocol, _host):
         }
 
         for item in args.pop("ulimits", []):
-            params["r_limits"].append(
-                {
-                    "type": item["Name"],
-                    "hard": item["Hard"],
-                    "soft": item["Soft"],
-                }
-            )
+            params["r_limits"].append({
+                "type": item["Name"],
+                "hard": item["Hard"],
+                "soft": item["Soft"],
+            })
 
         for item in args.pop("volumes", {}).items():
             key, value = item

podman/domain/images_manager.py

@@ -153,12 +153,10 @@ def prune(
                 error.append(element["Err"])
             else:
                 reclaimed += element["Size"]
-                deleted.append(
-                    {
-                        "Deleted": element["Id"],
-                        "Untagged": "",
-                    }
-                )
+                deleted.append({
+                    "Deleted": element["Id"],
+                    "Untagged": "",
+                })
         if len(error) > 0:
             raise APIError(response.url, response=response, explanation="; ".join(error))
 

podman/domain/ipam.py

@@ -25,14 +25,12 @@ def __init__(
             aux_addresses: Ignored.
         """
         super().__init__()
-        self.update(
-            {
-                "AuxiliaryAddresses": aux_addresses,
-                "Gateway": gateway,
-                "IPRange": iprange,
-                "Subnet": subnet,
-            }
-        )
+        self.update({
+            "AuxiliaryAddresses": aux_addresses,
+            "Gateway": gateway,
+            "IPRange": iprange,
+            "Subnet": subnet,
+        })
 
 
 class IPAMConfig(dict):
@@ -52,10 +50,8 @@ def __init__(
             options: Options to provide to the Network driver.
         """
         super().__init__()
-        self.update(
-            {
-                "Config": pool_configs or [],
-                "Driver": driver,
-                "Options": options or {},
-            }
-        )
+        self.update({
+            "Config": pool_configs or [],
+            "Driver": driver,
+            "Options": options or {},
+        })

podman/tests/integration/test_container_create.py

@@ -149,12 +149,10 @@ def test_container_ports(self):
             self.containers.append(container)
 
             self.assertTrue(
-                all(
-                    [
-                        x in port_test['expected_output']
-                        for x in container.attrs.get('HostConfig', {}).get('PortBindings')
-                    ]
-                )
+                all([
+                    x in port_test['expected_output']
+                    for x in container.attrs.get('HostConfig', {}).get('PortBindings')
+                ])
             )
 
     def test_container_healthchecks(self):
@@ -243,13 +241,11 @@ def test_container_devices(self):
             for device in devices:
                 path_on_host, path_in_container = device.split(':', 1)
                 self.assertTrue(
-                    any(
-                        [
-                            c.get('PathOnHost') == path_on_host
-                            and c.get('PathInContainer') == path_in_container
-                            for c in container_devices
-                        ]
-                    )
+                    any([
+                        c.get('PathOnHost') == path_on_host
+                        and c.get('PathInContainer') == path_in_container
+                        for c in container_devices
+                    ])
                 )
 
         with self.subTest("Check devices in running container object"):

podman/tests/integration/utils.py

@@ -66,14 +66,12 @@ def __init__(
         if os.environ.get("container") == "oci":
             self.cmd.append("--storage-driver=vfs")
 
-        self.cmd.extend(
-            [
-                "system",
-                "service",
-                f"--time={timeout}",
-                socket_uri,
-            ]
-        )
+        self.cmd.extend([
+            "system",
+            "service",
+            f"--time={timeout}",
+            socket_uri,
+        ])
 
         process = subprocess.run(
             [podman_exe, "--version"], check=True, stdout=subprocess.PIPE, stderr=subprocess.STDOUT

podman/tlsconfig.py

@@ -1,5 +1,6 @@
 """Holds TLS configuration."""
 
+
 class TLSConfig:
     """TLS configuration.