chore(ci): test prepare-cluster on pr

.github/workflows/test-create-customer.yml

@@ -29,10 +29,6 @@ jobs:
       - name: build
         run: make package-create-customer
 
-      - name: pull helm chart
-        run: |
-          helm pull oci://registry-1.docker.io/bitnamicharts/wordpress --version ${{ env.WORDPRESS_CHART_VERSION }}
-
       - name: test create-customer
         id: create-customer
         uses: ./create-customer

.github/workflows/test-create-release.yml

@@ -33,10 +33,18 @@ jobs:
         run: |
           helm pull oci://registry-1.docker.io/bitnamicharts/wordpress --version ${{ env.WORDPRESS_CHART_VERSION }}
 
-      - name: test create-release
-        id: create-release
+      - name: test create-release / helm
+        id: create-release-helm
         uses: ./create-release
         with:
           api-token: ${{ secrets.REPLICATED_API_TOKEN }}
           app-slug: replicated-actions-ci # from the Replicated QA team
           chart: wordpress-${{ env.WORDPRESS_CHART_VERSION }}.tgz
+
+      - name: test create-release / kots-install
+        id: create-release-kots-install
+        uses: ./create-release
+        with:
+          api-token: ${{ secrets.REPLICATED_API_TOKEN }}
+          app-slug: replicated-actions-ci # from the Replicated QA team
+          yaml-dir: ./test/kots-install

.github/workflows/test-kots-install.yml

@@ -30,17 +30,13 @@ jobs:
       - name: build
         run: make package-kots-install
 
-      - name: pull helm chart
-        run: |
-          helm pull oci://registry-1.docker.io/bitnamicharts/wordpress --version ${{ env.WORDPRESS_CHART_VERSION }}
-
       - name: create-release
         id: create-release
         uses: replicatedhq/compatibility-actions/create-release@v1
         with:
           api-token: ${{ secrets.REPLICATED_API_TOKEN }}
           app-slug: ${{ env.APP_SLUG }}
-          chart: wordpress-${{ env.WORDPRESS_CHART_VERSION }}.tgz
+          yaml-dir: ./test/kots-install
 
       - name: create-customer
         id: create-customer
@@ -60,19 +56,6 @@ jobs:
           kubernetes-distribution: kind
           timeout-minutes: 5
 
-      - name: helm-install
-        id: helm-install
-        uses: replicatedhq/compatibility-actions/helm-install@v1
-        with:
-          kubeconfig: ${{ steps.create-cluster.outputs.cluster-kubeconfig }}
-          helm-path: helm
-          registry-username: ${{ github.ref_name }}@example.com
-          registry-password: ${{ steps.create-customer.outputs.license-id }}
-          chart: oci://registry.replicated.com/${{ env.APP_SLUG }}/release__${{ steps.create-release.outputs.release-sequence }}/wordpress
-          name: wordpress
-          namespace: default
-          run-preflights: false
-
       - name: test kots-install
         id: kots-install
         uses: ./kots-install

.github/workflows/test-prepare-cluster.yml

@@ -17,7 +17,7 @@ concurrency:
   cancel-in-progress: true
 
 jobs:
-  test:
+  test-helm:
     runs-on: ubuntu-latest
     env:
       WORDPRESS_CHART_VERSION: 22.2.8
@@ -57,3 +57,38 @@ jobs:
         with:
           api-token: ${{ secrets.REPLICATED_API_TOKEN }}
           cluster-id: ${{ steps.prepare-cluster.outputs.cluster-id }}
+
+  test-kots-install:
+    runs-on: ubuntu-latest
+    env:
+      WORDPRESS_CHART_VERSION: 22.2.8
+
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 20
+          cache: "npm"
+          cache-dependency-path: |
+            **/package-lock.json
+
+      - name: build
+        run: make package-prepare-cluster
+
+      - name: test prepare-cluster
+        id: prepare-cluster
+        uses: ./prepare-cluster
+        with:
+          api-token: ${{ secrets.REPLICATED_API_TOKEN }}
+          app-slug: replicated-actions-ci # from the Replicated QA team
+          yaml-dir: ./test/kots-install
+          kubernetes-distribution: kind
+
+      - name: remove cluster
+        id: remove-cluster
+        if: always()
+        uses: replicatedhq/compatibility-actions/remove-cluster@v1
+        continue-on-error: true # It could be that the cluster is already removed
+        with:
+          api-token: ${{ secrets.REPLICATED_API_TOKEN }}
+          cluster-id: ${{ steps.prepare-cluster.outputs.cluster-id }}

Makefile

@@ -26,12 +26,12 @@ package-prepare-cluster: package-main
 
 .PHONY: package-archive-channel
 package-archive-channel:
-	rm -rf ./archive-channel/build ./archive-channel/dist ./archive-channel/node_modules
+	rm -rf ./archive-channel/build ./archive-channel/dist
 	cd ./archive-channel && npm install && npm run build && npm run package
 
 .PHONY: package-archive-customer
 package-archive-customer:
-	rm -rf ./archive-customer/build ./archive-customer/dist ./archive-customer/node_modules
+	rm -rf ./archive-customer/build ./archive-customer/dist
 	cd ./archive-customer && npm install && npm run build && npm run package
 
 .PHONY: package-create-cluster
@@ -41,17 +41,17 @@ package-create-cluster: package-main
 
 .PHONY: package-create-object-store
 package-create-object-store:
-	rm -rf ./create-object-store/build ./create-object-store/dist ./create-object-store/node_modules
+	rm -rf ./create-object-store/build ./create-object-store/dist
 	cd ./create-object-store && npm install && npm run build && npm run package
 
 .PHONY: package-create-postgres
 package-create-postgres:
-	rm -rf ./create-postgres/build ./create-postgres/dist ./create-postgres/node_modules
+	rm -rf ./create-postgres/build ./create-postgres/dist
 	cd ./create-postgres && npm install && npm run build && npm run package
 
 .PHONY: package-expose-port
 package-expose-port:
-	rm -rf ./expose-port/build ./expose-port/dist ./expose-port/node_modules
+	rm -rf ./expose-port/build ./expose-port/dist
 	cd ./expose-port && npm install && npm run build && npm run package
 
 .PHONY: package-create-customer
@@ -76,7 +76,7 @@ package-kots-install: package-main
 
 .PHONY: package-promote-release
 package-promote-release:
-	rm -rf ./promote-release/build ./promote-release/dist ./promote-release/node_modules
+	rm -rf ./promote-release/build ./promote-release/dist
 	cd ./promote-release && npm install && npm run build && npm run package
 
 .PHONY: package-remove-cluster
@@ -86,17 +86,17 @@ package-remove-cluster: package-main
 
 .PHONY: package-get-customer-instances
 package-get-customer-instances:
-	rm -rf ./get-customer-instances/build ./get-customer-instances/dist ./get-customer-instances/node_modules
+	rm -rf ./get-customer-instances/build ./get-customer-instances/dist
 	cd ./get-customer-instances && npm install && npm run build && npm run package
 
 .PHONY: package-report-compatibility-result
 package-report-compatibility-result:
-	rm -rf ./report-compatibility-result/build ./report-compatibility-result/dist ./report-compatibility-result/node_modules
+	rm -rf ./report-compatibility-result/build ./report-compatibility-result/dist
 	cd ./report-compatibility-result && npm install && npm run build && npm run package
 
 .PHONY: package-upgrade-cluster
 package-upgrade-cluster:
-	rm -rf ./upgrade-cluster/build ./upgrade-cluster/dist ./upgrade-cluster/node_modules
+	rm -rf ./upgrade-cluster/build ./upgrade-cluster/dist
 	cd ./upgrade-cluster && npm install && npm run build && npm run package
 
 .PHONY: readme-all

create-cluster/README.md

@@ -50,7 +50,7 @@ create_cluster ---> cluster_kubeconfig
 | kubernetes-distribution |  | True | Kubernetes distribution of the cluster to provision. |
 | kubernetes-version |  | False | Kubernetes version to provision (format is distribution dependent). |
 | license-id |  | False | License ID to use for the installation (required for Embedded Cluster distribution). |
-| cluster-name |  | True | Name of the cluster to provision |
+| cluster-name |  | False | Name of the cluster to provision |
 | ttl |  | False | Cluster TTL (duration, max 48h) |
 | disk |  | False | Disk size in GiB |
 | nodes |  | False | Number of nodes to provision |

create-cluster/action.yml

@@ -4,7 +4,6 @@ inputs:
   api-token:
     description: "API Token."
     required: true
-    default: ""
   kubernetes-distribution:
     description: "Kubernetes distribution of the cluster to provision."
     required: true
@@ -16,8 +15,7 @@ inputs:
     required: false
   cluster-name:
     description: "Name of the cluster to provision"
-    required: true
-    default: ""
+    required: false
   ttl:
     description: "Cluster TTL (duration, max 48h)"
     required: false

create-customer/action.yml

@@ -4,11 +4,9 @@ inputs:
   app-slug:
     description: "App Slug."
     required: true
-    default: ""
   api-token:
     description: "API Token."
     required: true
-    default: ""
   customer-name:
     description: "Name of the customer."
     required: true

create-release/action.yml

@@ -4,11 +4,9 @@ inputs:
   app-slug:
     description: "App Slug."
     required: true
-    default: ""
   api-token:
     description: "API Token."
     required: true
-    default: ""
   chart:
     description: "Path to the helm chart (One of `chart` or `yaml-dir` is required)."
     required: false

expose-port/src/index.ts

@@ -6,7 +6,7 @@ async function run() {
     const apiToken = core.getInput("api-token");
     const clusterId = core.getInput("cluster-id");
     const port = core.getInput("port");
-    const protocols = core.getInput("protocols").split(",");
+    const protocols = core.getInput("protocols") ? core.getInput("protocols").split(",") : ["https"];
     const apiEndpoint = core.getInput("replicated-api-endpoint");
     const isWildcard = core.getBooleanInput("wildcard");