Limit parallelism, Better verbosity on last step and no error

Author: klopfenstein-puzzle

.github/workflows/test-kubernetes.yaml

@@ -13,6 +13,7 @@ jobs:
   test:
     runs-on: ubuntu-latest
     strategy:
+      max-parallel: 2
       matrix:
         # Always quote versions to prevent int truncation (1.30 -> 1.3)
         # https://kubernetes.io/releases
@@ -28,7 +29,6 @@ jobs:
       with:
         kubernetes-version: ${{ matrix.k8s-version }}
 
-
     - name: Install cert-manager, patch upstream dns servers, wait for readiness
       run: |
         echo "Target cert-manager version: ${{ matrix.cm-version }}"
@@ -37,7 +37,6 @@ jobs:
         kubectl patch deployment cert-manager -n cert-manager --type='json' -p='[{"op": "add", "path": "/spec/template/spec/containers/0/args/-", "value": "--dns01-recursive-nameservers=ns1.dnsimple.com:53"}]'
         kubectl wait --for=condition=available --timeout=600s deployment/cert-manager-webhook -n cert-manager
 
-
     - name: Install cert-manager-webhook-dnsimple, wait for readiness
       env:
         DNSIMPLE_API_TOKEN: ${{ secrets.DNSIMPLE_API_TOKEN }}
@@ -54,7 +53,7 @@ jobs:
 
         helm -n cert-manager list
 
-        max_wait_time_seconds=600
+        max_wait_time_seconds=800
         sleep_between_iterations=10
 
         start=$(date +%s)
@@ -63,7 +62,7 @@ jobs:
         echo ""
         echo "Awaiting succesful deployment for max ${max_wait_time_seconds} seconds or until $(date --date="@$end")"
         while [ $(date +%s) -le $end ]; do
-            echo "[i] New iteration at $(date +%s)"
+            echo "[i] New iteration at $(date)"
             kubectl -n cert-manager get po
 
             if [ $(kubectl -n cert-manager get po | grep Crash | wc -l) -gt 0 ]; then
@@ -107,7 +106,6 @@ jobs:
         """  > certificate.yaml
         kubectl apply -f certificate.yaml
 
-
     - name: Assert that the DNS record was created
       env:
         DNSIMPLE_ZONE_NAME: ${{ secrets.DNSIMPLE_ZONE_NAME }}
@@ -120,9 +118,21 @@ jobs:
           sleep 30
         done
 
-
     - name: Check the certificate status
       run: |
-        kubectl wait --for=condition=ready --timeout=600s certificate/dnsimple-test
-        # this should not be necessary since the certificate is usually ready once the DNS record is propagated
-        kubectl get certificate dnsimple-test -o jsonpath='{.status.conditions[?(@.type=="Ready")].status}' | grep True
+        max_wait_time_seconds=300
+        end=$(( $(date +%s) + $max_wait_time_seconds ))
+        start=$(date +%s) 
+
+        while [ $(date +%s) -le $end ]; do
+          kubectl get certificate/dnsimple-test -o jsonpath='{.status}'
+
+          if [ $(kubectl get certificate dnsimple-test -o jsonpath='{.status.conditions[?(@.type=="Ready")].status}') == "True" ]; then
+            echo "Certificate is ready after $(( $(date +%s) - $start )) seconds"
+            exit 0
+          fi
+          sleep 10
+          echo "[i] New iteration at $(date)"
+        done
+
+        echo "::warning title=Certificate timed out::Have timed out waiting for certificate"