Merge pull request #59 from SQLJames/bug/test-connection

Bug/test connection

Author: SQLJames

CHANGELOG.md

@@ -1,5 +1,11 @@
 ## Changelog
 
+### V2.0.3
+
+#### Non-Breaking Changes
+
+- Updated Chart test to wait for the service to become available before testing rcon port
+
 ### V2.0.2
 
 #### Non-Breaking Changes

charts/factorio-server-charts/Chart.yaml

@@ -20,7 +20,7 @@ sources:
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 2.0.2
+version: 2.0.3
 
 # This is the version number of the application being deployed. This version number should be
 # incremented each time you make changes to the application. Versions are not expected to

charts/factorio-server-charts/templates/tests/test-connection.yaml

@@ -12,10 +12,19 @@ spec:
       image: outdead/rcon
       command:
         - /bin/sh
-        - -ec
+        - -xec
         - |
+          until nc -z {{ include "factorio-server-charts.fullname" . }}-rcon {{ .Values.rcon.port }}; do
+            echo "Waiting for main service to be ready..." >&2
+            sleep 5
+          done
+          echo "Main service is ready, proceeding with connection test."
           RCONPW="$(cat /rconpw/rconpw)"
-          ./rcon -a {{ include "factorio-server-charts.fullname" . }}-rcon:{{ .Values.rcon.port }} -p $RCONPW -t rcon -T 180s /s Test
+          if ./rcon -a {{ include "factorio-server-charts.fullname" . }}-rcon:{{ .Values.rcon.port }} -p $RCONPW -t rcon -T 180s /s Test; then
+            echo "RCON test command completed successfully."
+          else
+            echo "RCON test failed." >&2
+          fi
       volumeMounts:
         - mountPath: /rconpw
           name: rconpw