From f478399525fd6bdffad81f7f0d12fd873cc494d3 Mon Sep 17 00:00:00 2001
From: Georg Pfuetzenreuter <mail@georg-pfuetzenreuter.net>
Date: Dec 27 2023 16:55:50 +0000
Subject: Only log nginx roles in test_nginx


More easily identify relevant log files in the artifact browser
by avoiding files with no content (i.e. ones for roles without
nginx) being created.

Signed-off-by: Georg Pfuetzenreuter <mail@georg-pfuetzenreuter.net>

---

diff --git a/bin/test_nginx.sh b/bin/test_nginx.sh
index eed1abf..f66425c 100755
--- a/bin/test_nginx.sh
+++ b/bin/test_nginx.sh
@@ -83,8 +83,8 @@ for role in $(bin/get_roles.py); do
     rolestatus=0
     sls_role="salt/role/${role/./\/}.sls"
     out="$role.txt"
-    echo "START OF $role" > "$out"
     if grep nginx "$sls_role" > /dev/null; then
+        echo "START OF $role" > "$out"
         echo_INFO "Testing role: $role"
         reset_nginx
         reset_ip
@@ -143,8 +143,8 @@ for role in $(bin/get_roles.py); do
             STATUS=1
         fi
         echo
+        echo "END OF $role" >> "$out"
     fi
-    echo "END OF $role" >> "$out"
 done
 
 rpm -qa --qf '%{name}\n' | sort > /tmp/packages-after