diff --git a/.github/workflows/test-platforms.yml b/.github/workflows/test-platforms.yml index fe433f4b..e2574295 100644 --- a/.github/workflows/test-platforms.yml +++ b/.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":"rvykydal@redhat.com" }, - ${{ 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