assume host mode for postgres short ports

Author: jordanstephens

src/pkg/cli/compose/fixup.go

@@ -241,6 +241,14 @@ func fixupPostgresService(svccfg *composeTypes.ServiceConfig, provider client.Pr
 		}
 		term.Debugf("service %q: adding postgres host port %d", svccfg.Name, port)
 		svccfg.Ports = []composeTypes.ServicePortConfig{{Target: port, Mode: Mode_HOST, Protocol: Protocol_TCP}}
+	} else {
+		for i, port := range svccfg.Ports {
+			if port.Mode == Mode_INGRESS || port.Mode == "" {
+				svccfg.Ports[i].Mode = Mode_HOST
+				svccfg.Ports[i].Published = ""   // ignore published port in host mode
+				svccfg.Ports[i].AppProtocol = "" // ignore app_protocol in host mode
+			}
+		}
 	}
 	return nil
 }

src/testdata/postgres/compose.yaml

@@ -29,3 +29,9 @@ services:
     x-defang-postgres:
     environment:
       - PGPORT=5433
+
+  short-ports:
+    image: postgres
+    x-defang-postgres:
+    ports:
+      - 5432:5432

src/testdata/postgres/compose.yaml.fixup

@@ -47,6 +47,21 @@
       }
     ]
   },
+  "short-ports": {
+    "command": null,
+    "entrypoint": null,
+    "image": "postgres",
+    "networks": {
+      "default": null
+    },
+    "ports": [
+      {
+        "mode": "host",
+        "target": 5432,
+        "protocol": "tcp"
+      }
+    ]
+  },
   "with-ext": {
     "command": null,
     "entrypoint": null,

src/testdata/postgres/compose.yaml.golden

@@ -20,6 +20,16 @@ services:
     networks:
       default: null
     x-defang-postgres: null
+  short-ports:
+    image: postgres
+    networks:
+      default: null
+    ports:
+      - mode: ingress
+        target: 5432
+        published: "5432"
+        protocol: tcp
+    x-defang-postgres: null
   with-ext:
     image: postgres
     networks:

src/testdata/postgres/compose.yaml.warnings

@@ -2,6 +2,8 @@
  ! service "no-ext": stateful service will lose data on restart; use a managed service instead
  ! service "no-ports": missing memory reservation; using provider-specific defaults. Specify deploy.resources.reservations.memory to avoid out-of-memory errors
  ! service "no-ports-override": missing memory reservation; using provider-specific defaults. Specify deploy.resources.reservations.memory to avoid out-of-memory errors
+ ! service "short-ports": ingress port without healthcheck defaults to GET / HTTP/1.1
+ ! service "short-ports": missing memory reservation; using provider-specific defaults. Specify deploy.resources.reservations.memory to avoid out-of-memory errors
  ! service "with-ext": missing memory reservation; using provider-specific defaults. Specify deploy.resources.reservations.memory to avoid out-of-memory errors
  ! service "wrong-image": managed Postgres service should use a postgres image
  ! service "wrong-image": missing memory reservation; using provider-specific defaults. Specify deploy.resources.reservations.memory to avoid out-of-memory errors