Set HOSTFILTER only if TARGET_MACHINES is set.

Fixes bug introduced in #10146 which broke all-nodes deployments.
alphagov · Feb 14, 2020 · 9365972 · 9365972
1 parent f00aea7
commit 9365972
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/modules/govuk_jenkins/templates/jobs/deploy_app.yaml.erb b/modules/govuk_jenkins/templates/jobs/deploy_app.yaml.erb
@@ -20,7 +20,9 @@
             export DEPLOY_TO="<%= @environment -%>"
             export DEPLOY_TASK="$DEPLOY_TASK"
             export TAG="$TAG"
-            export HOSTFILTER="$TARGET_MACHINES"
+            if [ "${TARGET_MACHINES:-all}" != "all" ]; then
+              export HOSTFILTER="$TARGET_MACHINES"
+            fi
             export ORGANISATION="<%= @environment -%>"
             export CI_DEPLOY_JENKINS_API_KEY="<%= @ci_deploy_jenkins_api_key -%>"
             export GRAPHITE_HOST="<%= @graphite_host -%>"