fix(ci): fix controller packaging bug and wire up gpu-offload pytest CI

Address PR review feedback that the 44 tests added under
gpu-offload/{controller,runtime}/tests never run in CI.

- gpu-offload/controller/pyproject.toml: drop the unused [build-system]
  (Hatchling) section and mark the project virtual (tool.uv.package = false).
  mutate.py is a flat script, never installed as a wheel or built as an
  editable install anywhere; Hatchling's wheel builder failed with "Unable
  to determine which files to ship" because there is no gpu_offload_controller/
  package directory. Re-locked uv.lock accordingly (source: editable -> virtual).
- .github/workflows/pytest-gpu-offload.yml: new reusable workflow, matrixed
  over controller and runtime, each synced from its own uv.lock in an
  isolated environment and run with pytest.
- pr-validation.yml: path-gate the new job on gpu-offload/{controller,runtime}/
  changes and add it to the validation summary.
- main.yml: run the new job unconditionally and add it to the release gate.
- CONTRIBUTING.md: document the new component suite and its local-run command.

Coverage/Codecov integration is intentionally left out of scope; both
subprojects currently test without coverage tooling.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

Author: dwhiteddsoft

.github/workflows/main.yml

@@ -144,6 +144,13 @@ jobs:
       contents: read
       id-token: write
 
+  # Pytest: gpu-offload controller and runtime components
+  pytest-gpu-offload:
+    name: Pytest GPU Offload
+    uses: ./.github/workflows/pytest-gpu-offload.yml
+    permissions:
+      contents: read
+
   # Pytest: shared CI scripts component
   pytest-shared-ci:
     name: Pytest Shared CI
@@ -307,6 +314,7 @@ jobs:
       - pytest-dm-tools
       - pytest-data-pipeline
       - pytest-inference
+      - pytest-gpu-offload
       - pytest-shared-ci
       - dataviewer-frontend-tests
       - dataviewer-backend-pytests

.github/workflows/pr-validation.yml

@@ -29,6 +29,7 @@ jobs:
       dm_tools: ${{ steps.filter.outputs.dm_tools }}
       data_pipeline: ${{ steps.filter.outputs.data_pipeline }}
       inference: ${{ steps.filter.outputs.inference }}
+      gpu_offload: ${{ steps.filter.outputs.gpu_offload }}
       shared_ci: ${{ steps.filter.outputs.shared_ci }}
       dv_backend: ${{ steps.filter.outputs.dv_backend }}
       dv_frontend: ${{ steps.filter.outputs.dv_frontend }}
@@ -69,6 +70,7 @@ jobs:
             echo "dm_tools=$(match '^data-management/tools/')"
             echo "data_pipeline=$(match '^data-pipeline/capture/')"
             echo "inference=$(match '^fleet-deployment/inference/')"
+            echo "gpu_offload=$(match '^gpu-offload/(controller|runtime)/')"
             echo "shared_ci=$(match '^shared/ci/')"
             echo "dv_backend=$(match '^(data-management/viewer/backend/|data-management/viewer/docker-compose\.yml|shared/ci/smoke-docker\.sh|\.github/workflows/smoke-cpu\.yml)')"
             echo "dv_frontend=$(match '^data-management/viewer/frontend/')"
@@ -332,6 +334,15 @@ jobs:
       contents: read
       id-token: write
 
+  # Pytest: gpu-offload controller and runtime components
+  pytest-gpu-offload:
+    name: Pytest GPU Offload
+    needs: changes
+    if: needs.changes.outputs.gpu_offload == 'true'
+    uses: ./.github/workflows/pytest-gpu-offload.yml
+    permissions:
+      contents: read
+
   # Pytest: shared CI scripts component
   pytest-shared-ci:
     name: Pytest Shared CI
@@ -515,6 +526,7 @@ jobs:
       - pytest-dm-tools
       - pytest-data-pipeline
       - pytest-inference
+      - pytest-gpu-offload
       - pytest-shared-ci
       - dataviewer-backend-pytests
       - evaluation-pytests

.github/workflows/pytest-gpu-offload.yml

@@ -0,0 +1,44 @@
+---
+name: Pytest GPU Offload
+
+'on':
+  workflow_call:
+
+permissions:
+  contents: read
+
+jobs:
+  pytest-gpu-offload:
+    name: Pytest GPU Offload (${{ matrix.name }})
+    strategy:
+      fail-fast: false
+      matrix:
+        include:
+          - name: controller
+            path: gpu-offload/controller
+          - name: runtime
+            path: gpu-offload/runtime
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1  # v7.0.1
+        with:
+          persist-credentials: false
+
+      - name: Setup Python
+        uses: actions/setup-python@5fda3b95a4ea91299a34e894583c3862153e4b97  # v7.0.0
+        with:
+          python-version: '3.12'
+
+      - name: Setup uv
+        uses: astral-sh/setup-uv@20cfd1bf945f4377ade1205e4dbc17946fc9a30d  # v10.0.1
+
+      - name: Install dependencies
+        working-directory: ${{ matrix.path }}
+        run: uv sync --locked --extra test
+
+      - name: Run pytest
+        working-directory: ${{ matrix.path }}
+        run: uv run pytest -o addopts="" tests -v

CONTRIBUTING.md

@@ -371,10 +371,10 @@ Reviewers verify regression tests are included. Compliance is tracked over time
 
 ### Running Tests
 
-Tests are split into eight pytest component suites that mirror the CI
-`pytest-*` flags (`pytest-training`, `pytest-dm-tools`,
-`pytest-data-pipeline`, `pytest-inference`, `pytest-shared-ci`,
-`pytest-dataviewer`, `pytest-evaluation`, `pytest-fuzz`). Run a single
+Tests are split into pytest component suites that mirror the CI `pytest-*`
+flags (`pytest-training`, `pytest-dm-tools`, `pytest-data-pipeline`,
+`pytest-inference`, `pytest-shared-ci`, `pytest-dataviewer`,
+`pytest-evaluation`, `pytest-fuzz`, `pytest-gpu-offload`). Run a single
 component locally:
 
 ```bash
@@ -399,6 +399,12 @@ cd data-management/viewer/backend && uv run pytest -v
 # Evaluation (evaluation/, run from that dir)
 cd evaluation && uv run pytest -v
 
+# GPU offload controller (gpu-offload/controller, run from that dir; own uv.lock)
+cd gpu-offload/controller && uv sync --extra test && uv run pytest -o addopts="" tests -v
+
+# GPU offload runtime (gpu-offload/runtime, run from that dir; own uv.lock)
+cd gpu-offload/runtime && uv sync --extra test && uv run pytest -o addopts="" tests -v
+
 # Fuzz / regression (tests/, run from repo root)
 uv run pytest tests/ -v
 ```
@@ -427,6 +433,9 @@ Substitute the component path and `--cov` target for the pytest component
 suite you are validating. Codecov tracks pytest uploads by flag, but only the
 named project statuses in `codecov.yml` are top-level project gates;
 `pytest-fuzz` is advisory and `terraform` uploads test results only.
+`pytest-gpu-offload` runs its two subprojects (`gpu-offload/controller`,
+`gpu-offload/runtime`) in their own locked environments and is not yet
+wired into Codecov.
 
 ### Test Organization
 

gpu-offload/controller/pyproject.toml

@@ -1,7 +1,3 @@
-[build-system]
-requires = ["hatchling==1.27.0"]
-build-backend = "hatchling.build"
-
 [project]
 name = "gpu-offload-controller"
 version = "0.0.0"
@@ -17,3 +13,6 @@ dependencies = [
 test = [
     "pytest==9.1.1",
 ]
+
+[tool.uv]
+package = false