First pass at uploading test results to codecov

* Create an explicit directory for test reports rather than the workspace-tmp dir which might contain other files

Signed-off-by: David Gardner <dagardner@nvidia.com>

Author: dagardner-nv

.github/workflows/ci_pipe.yml

@@ -39,6 +39,9 @@ on:
         description: 'The base SHA of the PR'
         required: true
         type: string
+    secrets:
+      CODECOV_TOKEN:
+        required: true
 
 # We only support ubuntu, so bash is the default
 defaults:
@@ -109,6 +112,10 @@ jobs:
           - python-version: "3.12"
             container: ${{ inputs.py_12_container }}
 
+            env:
+              CI_PYTHON_VERSION: ${{ matrix.python-version }}
+              CI_ARCH: ${{ matrix.arch }}
+
     steps:
       - name: Checkout
         uses: actions/checkout@v4
@@ -127,9 +134,20 @@ jobs:
         if: ${{ always() }}
         with:
           name: "test-results-${{ matrix.arch }}-py${{ matrix.python-version }}"
-          path: "${{ github.workspace }}/tmp/*.xml"
+          path: "${{ github.workspace }}/tmp/reports/*.xml"
           if-no-files-found: error
-
+      - name: Upload coverage to Codecov
+        uses: codecov/codecov-action@v5
+        # Upload test results even if the tests fail
+        if: ${{ always() }}
+        with:
+          directory: "${{ github.workspace }}/tmp/reports"
+          env_vars: CI_ARCH,CI_PYTHON_VERSION
+          fail_ci_if_error: true
+          flags: unittests
+          name: nat-code-cov-${{ matrix.arch }}-py${{ matrix.python-version }}
+          token: ${{ secrets.CODECOV_TOKEN }}
+          verbose: true
 
   documentation:
     name: Documentation

ci/scripts/github/tests.sh

@@ -19,6 +19,7 @@ set -e
 GITHUB_SCRIPT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )"
 
 source ${GITHUB_SCRIPT_DIR}/common.sh
+export REPORTS_DIR=${WORKSPACE_TMP}/reports
 get_lfs_files
 
 create_env group:dev extra:all
@@ -27,9 +28,10 @@ rapids-logger "Git Version: $(git describe)"
 rapids-logger "Running tests with Python version $(python --version) and pytest version $(pytest --version) on $(arch)"
 set +e
 
-pytest --junit-xml=${WORKSPACE_TMP}/report_pytest.xml \
+REPORT_IDENT_SLUG="$(arch)-py${PYTHON_VERSION}"
+pytest --junit-xml=${REPORTS_DIR}/report-${REPORT_IDENT_SLUG}_pytest.xml \
        --cov=nat --cov-report term-missing \
-       --cov-report=xml:${WORKSPACE_TMP}/report_pytest_coverage.xml
+       --cov-report=xml:${REPORTS_DIR}/report-${REPORT_IDENT_SLUG}_pytest_coverage.xml
 PYTEST_RESULTS=$?
 
 exit ${PYTEST_RESULTS}