Fixed: Added more verbose output to terminal

Fixed: Set an timeout to 12h as tests can be lengthy

Signed-off-by: Milosz Linkiewicz <[email protected]>

Author: Mionsz

.github/workflows/validation-tests.yml

@@ -189,9 +189,11 @@ jobs:
           python3 -m pipenv install -r requirements.txt
           echo "VIRTUAL_ENV=$(python3 -m pipenv --venv)/bin/activate" >> "$GITHUB_ENV"
 
+  # Timeout of this job is set to 12h [60m/h*12h=720m]
   validation-run-tests:
     needs: [validation-build-mtl]
     runs-on: [Linux, self-hosted, DPDK]
+    timeout-minutes: 720
     env:
       PYTEST_ALIAS: 'sudo --preserve-env python3 -m pipenv run pytest'
       PYTEST_PARAMS: '--media=/mnt/media --build="../.."'
@@ -239,6 +241,7 @@ jobs:
         working-directory: tests/validation
         run: |
           set +e
+          echo "::group::pre-execution-summary"
           export TEST_PORT_P="${{ env.TEST_PORT_P }}"
           export TEST_PORT_R="${{ env.TEST_PORT_R }}"
           TESTS_INCLUDED_IN_EXECUTION=( $(grep -v "collected in" <(${{ env.PYTEST_ALIAS }} "tests/${{ inputs.validation-tests-1 }}" ${{ env.PYTEST_PARAMS }} --nic="${TEST_PORT_P},${TEST_PORT_R}" --collect-only -q --no-summary 2>&1)) )
@@ -251,9 +254,14 @@ jobs:
           NUMBER_OF_TESTS="${#TESTS_INCLUDED_IN_EXECUTION[@]}"
           TESTS_FAIL=()
           TESTS_SUCCESS=()
+
+          echo "${SUMMARY_MAIN_HEADER} tests (total ${NUMBER_OF_TESTS}) :rocket:\n----------------------------------"
+          echo "Tests to be executed:\n${TESTS_INCLUDED_IN_EXECUTION[@]}"
+
           echo "## ${SUMMARY_MAIN_HEADER} tests (total ${NUMBER_OF_TESTS}) :rocket:" >> "$GITHUB_STEP_SUMMARY"
           echo "| ❌/✅ | Collected Test | Started | Ended | Took (s) | Result |"  >> "$GITHUB_STEP_SUMMARY"
           echo "| --- | ---| --- | --- | --- | --- |" >> "$GITHUB_STEP_SUMMARY"
+          echo "::endgroup::"
 
           for test in ${TESTS_INCLUDED_IN_EXECUTION[@]}; do
             echo "::group::${test}"
@@ -313,11 +321,16 @@ jobs:
           sudo killall -SIGINT pytest || true
           sudo killall -SIGINT MtlManager || true
 
+      - name: 'cleanup: Create archive with logs from execution'
+        if: always()
+        working-directory: tests/validation
+        run: |
+          sudo tar -czf "validation-execution-logs.tar.gz" "./logs"
+          sudo rm -rf "./logs"
+
       - name: 'cleanup: Restore valid owner to repository and directories'
         if: always()
         run: |
-          sudo tar -czf "${{ github.workspace }}/tests/validation/validation-execution-logs.tar.gz" "${{ github.workspace }}/tests/validation/logs"
-          sudo rm -rf "${{ github.workspace }}/tests/validation/logs"
           sudo chown -R "${USER}" "$(pwd)"
 
       - name: 'cleanup: Bind pf to kernel driver'
@@ -332,7 +345,7 @@ jobs:
         if: always()
         uses: actions/upload-artifact@65c4c4a1ddee5b72f698fdd19549f0f0fb45cf08 # v4.6.0
         with:
-          name: 'validation-execution-logs'
+          name: 'validation-execution-logs.tar.gz'
           path: '${{ github.workspace }}/tests/validation/validation-execution-logs.tar.gz'
 
       - name: 'cleanup: Generate runner summary'