feat(docker): split expose_port into expose setting

Instead of defining expose_port inside a service,
you now have to define port and service inside the new "expose" setting.

Author: saitho

.github/workflows/go.yml

@@ -2,13 +2,11 @@ name: Go
 
 on:
   push:
-    branches: [ master ]
-    paths:
-      - src
+    branches:
+    - master
   pull_request:
-    branches: [ master ]
-    paths:
-      - src
+    branches:
+    - master
 
 jobs:
 

examples/invalid/docker_missing-expose-port.yml

@@ -0,0 +1,17 @@
+---
+domain: mydomain.com
+deployment:
+  type: docker
+  settings:
+    expose:
+      service: app
+    services:
+      app:
+        image: getstackhead/project-demo-php
+        links:
+          - db
+      db:
+        image: mariadb:10.5
+        expose_port: 3306
+        environment:
+          MYSQL_ROOT_PASSWORD: example

examples/invalid/docker_double-expose.yml renamed to examples/invalid/docker_missing-expose-service.yml

@@ -3,10 +3,11 @@ domain: mydomain.com
 deployment:
   type: docker
   settings:
+    expose:
+      port: 8080
     services:
       app:
         image: getstackhead/project-demo-php
-        expose_port: 8080
         links:
           - db
       db:

examples/valid/docker_project.yml

@@ -3,10 +3,12 @@ domain: mydomain.com
 deployment:
   type: docker
   settings:
+    expose:
+      service: app
+      port: 8080
     services:
       app:
         image: getstackhead/project-demo-php
-        expose_port: 8080
         links:
           - db
       db:

schema/project-definition.schema.json

@@ -16,10 +16,6 @@
           "description": "Name of Docker image",
           "type": "string"
         },
-        "expose_port": {
-          "description": "Port that receives web traffic (e.g. 80, 3000, 8080 for most applications). Only one in this file!",
-          "type": "integer"
-        },
         "environment": {
           "description": "Docker environment variables",
           "type": "object"
@@ -60,6 +56,20 @@
         "settings": {
           "additionalProperties": false,
           "properties": {
+            "expose": {
+              "type": "object",
+              "properties": {
+                "service": {
+                  "type": "string",
+                  "description": "Name of the service below that exposes the port"
+                },
+                "port": {
+                  "type": "integer",
+                  "description": "Port that receives web traffic (e.g. 80, 3000, 8080 for most applications)"
+                }
+              },
+              "required": ["service", "port"]
+            },
             "services": {
               "description": "List of Docker services to be started",
               "type": "object",