fixes for real run

.github/workflows/test-platforms.yml

@@ -53,12 +53,18 @@ jobs:
           LAUNCH_ARGS=$(echo "$BODY" | sed -n '1 s/^[^ ]* *//p' | sed 's/[[:space:]]*$//')
           echo "launch arguments are: $LAUNCH_ARGS"
           echo "launch_args=${LAUNCH_ARGS}" >> $GITHUB_OUTPUT
+          # check for dry run mode
+          DRY_RUN=False
+          echo "$BODY" | grep -q "/test-platforms-dry" && DRY_RUN=True
+          echo "Dry run: ${DRY_RUN}"
+          echo "dry_run=${DRY_RUN}" >> $GITHUB_OUTPUT
 
     outputs:
       allowed_user: ${{ steps.check_user_perm.outputs.allowed_user }}
       base_ref: ${{ fromJson(steps.pr_api.outputs.data).base.ref }}
       sha: ${{ fromJson(steps.pr_api.outputs.data).head.sha }}
       launch_args: ${{ steps.parse_launch_args.outputs.launch_args }}
+      dry_run: ${{ steps.parse_launch_args.outputs.dry_run }}
 
 
   platform:
@@ -133,7 +139,7 @@ jobs:
           echo "changed_tests=${CHANGED_TESTS}" >> $GITHUB_OUTPUT
 
       - name: Get skipped tests for platform ${{ matrix.platform }}
-        id: get_platform_skiptest
+        id: get_platform_specs
         working-directory: ./kickstart-tests
         run: |
           set -eux
@@ -142,15 +148,19 @@ jobs:
           if [ ${{ matrix.platform }} == "daily-iso" ]; then
             echo "skip_tests=skip-on-fedora" >> $GITHUB_OUTPUT
             echo "disabled_testtypes=$SKIP_TESTTYPES_DAILY_ISO" >> $GITHUB_OUTPUT
+            echo "platform=rawhide" >> $GITHUB_OUTPUT
           elif [ ${{ matrix.platform }} == "rawhide" ]; then
             echo "skip_tests=skip-on-fedora" >> $GITHUB_OUTPUT
             echo "disabled_testtypes=$SKIP_TESTTYPES_RAWHIDE" >> $GITHUB_OUTPUT
+            echo "platform=rawhide" >> $GITHUB_OUTPUT
           elif [ ${{ matrix.platform }} == "rhel8" ]; then
             echo "skip_tests=skip-on-rhel,skip-on-rhel-8" >> $GITHUB_OUTPUT
             echo "disabled_testtypes=$SKIP_TESTTYPES_RHEL8" >> $GITHUB_OUTPUT
+            echo "platform=rhel8" >> $GITHUB_OUTPUT
           elif [ ${{ matrix.platform }} == "rhel9" ]; then
             echo "skip_tests=skip-on-rhel,skip-on-rhel-9" >> $GITHUB_OUTPUT
             echo "disabled_testtypes=$SKIP_TESTTYPES_RHEL9" >> $GITHUB_OUTPUT
+            echo "platform=rhel9" >> $GITHUB_OUTPUT
           else
             echo "Platform is not supported by kickstart tests yet!"
             exit 1
@@ -163,13 +173,13 @@ jobs:
           set -eux
 
           CHANGED_TESTS="${{steps.get_changed_tests.outputs.changed_tests }}"
-          DISABLED_TESTTYPES="${{ steps.get_platform_skiptest.outputs.disabled_testtypes }}"
+          DISABLED_TESTTYPES="${{ steps.get_platform_specs.outputs.disabled_testtypes }}"
           LAUNCH_ARGS="${{ needs.pr-info.outputs.launch_args }}"
 
           if [ -n "${LAUNCH_ARGS}" ]; then
             echo "description=Running tests required by the PR comment explicitly." >> $GITHUB_OUTPUT
           elif [ -n "${CHANGED_TESTS}" ]; then
-            echo "description=Running tests affected by changes." >> $GITHUB_OUTPUT
+            echo "description=Running tests affected by changes (excluding disabled)." >> $GITHUB_OUTPUT
             LAUNCH_ARGS="${CHANGED_TESTS}"
             if [ -n "${DISABLED_TESTTYPES}" ]; then
               LAUNCH_ARGS="--skip-testtypes ${DISABLED_TESTTYPES} ${LAUNCH_ARGS}"
@@ -182,7 +192,7 @@ jobs:
           source ./containers/runner/skip-testtypes
           PERMIAN_QUERY=$(scripts/generate-permian-query.py \
              --skip-testtypes $SKIP_TESTTYPES_ANACONDA_PR \
-             --skip-testtypes ${{ steps.get_platform_skiptest.outputs.skip_tests }} \
+             --skip-testtypes ${{ steps.get_platform_specs.outputs.skip_tests }} \
              ${LAUNCH_ARGS} )
           if [ $? == 0 ]; then
             echo "launch_args=$LAUNCH_ARGS" >> $GITHUB_OUTPUT
@@ -205,7 +215,6 @@ jobs:
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
-
       # Fetch boot.iso and configiure its local location
       - name: Fetch boot.iso if available for platform ${{ matrix.platform }}
         id: boot_iso_for_platform
@@ -272,9 +281,9 @@ jobs:
                 "configurations":[{"architecture":"x86_64"}],
                 "point_person":"[email protected]"
               },
-              ${{ steps.boot_iso_from_platform.outputs.boot_iso }}
+              ${{ steps.boot_iso_for_platform.outputs.boot_iso }}
               "kstestParams":{
-                "platform":"${{ matrix.platform }}",
+                "platform":"${{ steps.get_platform_specs.outputs.platform }}",
                 "urls":{
                   "x86_64":{
                     "installation_tree":"${{ steps.set_installation_urls.outputs.installation_tree }}",
@@ -289,7 +298,7 @@ jobs:
           if [ -n "$rc" ]; then
             exit $rc
           else
-            exit 111
+            grep -q "All execution and reporting is done" permian.log || exit 111
           fi
 
       - name: Collect anaconda logs