Update validation-tests.yml

Author: Mionsz

.github/workflows/validation-tests.yml

@@ -37,46 +37,46 @@ jobs:
     outputs:
       pipenv-activate: ${{ steps.pipenv-install.outputs.VIRTUAL_ENV }}
     steps:
-      - name: Harden Runner
+      - name: 'preparation: Harden Runner'
         uses: step-security/harden-runner@91182cccc01eb5e619899d80e4e971d6181294a7 # v2.10.1
         with:
           egress-policy: audit
 
-      - name: Checkout MTL
+      - name: 'preparation: Checkout MTL'
         uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
         with:
           ref: '${{ inputs.branch-to-checkout }}'
 
-      - name: Checkout DPDK
+      - name: 'preparation: Checkout DPDK'
         uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
         with:
           repository: 'DPDK/dpdk'
           ref: 'v${{  env.DPDK_VERSION  }}'
           path: 'dpdk'
 
-      - name: Install the build dependency
+      - name: 'configuration: Install the build dependency'
         run: |
           sudo apt-get install -y git gcc meson python3 python3-pyelftools pkg-config libnuma-dev libjson-c-dev libpcap-dev libgtest-dev libsdl2-dev libsdl2-ttf-dev libssl-dev
           sudo apt-get install -y systemtap-sdt-dev
 
-      - name: Apply dpdk patches
+      - name: 'configuration: Apply dpdk patches'
         run: |
           patch -d "dpdk" -p1 -i <(cat patches/dpdk/${{  env.DPDK_VERSION  }}/*.patch)
 
-      - name: Build dpdk
+      - name: 'installation: Build dpdk'
         run: |
           cd dpdk
           meson build
           ninja -C build
           cd build
           sudo ninja install
 
-      - name: Build
+      - name: 'installation: Build mtl'
         run: |
           ./build.sh
           sudo ldconfig
 
-      - name: Prepare pipenv environment
+      - name: 'installation: Install pipenv environment'
         working-directory: tests/validation
         id: pipenv-install
         run: |
@@ -90,30 +90,49 @@ jobs:
     needs: [validation-build-mtl]
     runs-on: [Linux, self-hosted, DPDK]
     steps:
-      - name: Harden Runner
+      - name: 'preparation: Harden Runner'
         uses: step-security/harden-runner@91182cccc01eb5e619899d80e4e971d6181294a7 # v2.10.1
         with:
           egress-policy: audit
 
-      - name: Kill previous pytest routine
+      - name: 'preparation: Kill MtlManager and pytest routines'
         run: |
           sudo killall -SIGINT pipenv || true
           sudo killall -SIGINT pytest || true
           sudo killall -SIGINT MtlManager || true
 
-      - name: Binding network adapter
+      - name: 'preparation: Binding network adapter'
         run: |
           sudo ./script/nicctl.sh create_vf "${TEST_PF_PORT_P}" || true
           sudo ./dpdk/usertools/dpdk-devbind.py -b vfio-pci "${TEST_DMA_PORT_P}" || true
           sudo ./dpdk/usertools/dpdk-devbind.py -b vfio-pci "${TEST_DMA_PORT_R}" || true
 
-      - name: Start MtlManager at background
+      - name: 'preparation: Start MtlManager at background'
         run: |
           sudo MtlManager &
 
-      - name: Run tests in pipenv environment
+      - name: 'execution: Run tests in pipenv environment'
         working-directory: tests/validation
         run: |
-          echo source "${{ needs.validation-build-mtl.outputs.pipenv-activate }}"
           source "$(python3 -m pipenv --venv)/bin/activate"
-          pytest '${{ inputs.validation-tests }}' --nic="${TEST_PORT_P},${TEST_PORT_R}" --media=/mnt/media --build='../..'
+          sudo pytest '${{ inputs.validation-tests }}' --nic="${TEST_PORT_P},${TEST_PORT_R}" --media=/mnt/media --build='../..'
+
+  validation-cleanup-tests:
+    needs: [validation-build-mtl, validation-run-tests]
+    runs-on: [Linux, self-hosted, DPDK]
+    if: always()
+    steps:
+      - name: 'preparation: Harden Runner'
+        uses: step-security/harden-runner@91182cccc01eb5e619899d80e4e971d6181294a7 # v2.10.1
+        with:
+          egress-policy: audit
+
+      - name: 'cleanup: Kill MtlManager and pytest routines'
+        run: |
+          sudo killall -SIGINT pipenv || true
+          sudo killall -SIGINT pytest || true
+          sudo killall -SIGINT MtlManager || true
+
+      - name: 'cleanup: Restore valid owner to repository and directories'
+        run: |
+          sudo chown -R "${USER}" "$(pwd)"