[REF] refactor docker 'ports' args

odoo_openupgrade_wizard/tools_docker.py

@@ -26,7 +26,7 @@ def run_container(
     image_name,
     container_name,
     command=None,
-    ports=False,
+    ports={},
     volumes={},
     environments={},
     links={},
@@ -37,13 +37,9 @@ def run_container(
 
     logger.debug("Launching Docker container named %s ..." % (image_name))
     debug_docker_command = "docker run --name %s\\\n" % (container_name)
-    if ports:
-        for internal_port, host_port in ports.items():
-            debug_docker_command += (
-                " --publish {host_port}:{internal_port}\\\n".format(
-                    internal_port=internal_port, host_port=host_port
-                )
-            )
+
+    for k, v in ports.items():
+        debug_docker_command += " --publish {k}={v}\\\n".format(k=k, v=v)
     for k, v in volumes.items():
         debug_docker_command += " --volume {k}:{v}\\\n".format(
             k=str(k), v=str(v)
@@ -65,7 +61,7 @@ def run_container(
         image_name,
         name=container_name,
         command=command,
-        ports=ports,
+        ports={x: y for y, x in ports.items()},
         volumes=[str(k) + ":" + str(v) for k, v in volumes.items()],
         environment=environments,
         links=links,

odoo_openupgrade_wizard/tools_odoo.py

@@ -177,14 +177,17 @@ def run_odoo(
         demo=demo,
     )
 
+    host_xmlrpc_port = (
+        alternative_xml_rpc_port
+        and alternative_xml_rpc_port
+        or ctx.obj["config"]["odoo_host_xmlrpc_port"]
+    )
     return run_container(
         get_docker_image_tag(ctx, odoo_version),
         get_docker_container_name(ctx, migration_step),
         command=command,
         ports={
-            "8069": alternative_xml_rpc_port
-            and alternative_xml_rpc_port
-            or ctx.obj["config"]["odoo_host_xmlrpc_port"],
+            host_xmlrpc_port: 8069,
         },
         volumes={
             env_path: "/env/",

odoo_openupgrade_wizard/tools_postgres.py

@@ -24,7 +24,7 @@ def get_postgres_container(ctx):
         image_name,
         container_name,
         ports={
-            "5432": ctx.obj["config"]["postgres_host_port"],
+            ctx.obj["config"]["postgres_host_port"]: 5432,
         },
         environments={
             "POSTGRES_USER": "odoo",