CloudXR service tests can only run against a mocked runtime; no GPU path for src/core/cloudxr_tests
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 56/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- docker-compose, github-actions, python
- Área
- ci-cd, infrastructure, testing
Línea de trabajo
Comienza con src/core/cloudxr_tests y el precedente del workflow test-viz-gpu. Añade un nivel de pruebas de runtime real que se omita cuando no haya ningún runtime disponible y que se ejecute en el GPU runner, manteniendo los mocks solo para estados imposibles. Se considera terminado cuando el comportamiento del ciclo de vida, como start, stop y status, se prueba contra el runtime real, mientras que CPU ctest permanece limpio.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
src/core/cloudxr_tests and src/core/rig_tests run only under ctest on GitHub-hosted, GPU-less runners, so every test that touches the runtime has to fake one: a hand-bound unix socket for is_runtime_live(), a mocked Popen for the runtime process, and now a patched runtime_version() (it dlopens libcloudxr, which needs libcuda). At that point we are testing the mocks as much as the service. Nothing verifies that service start really detaches, that stop really tears the runtime down, or that status reads a real /proc cmdline.
What already exists
- GPU runners:
[self-hosted, linux, gpu, x64|arm64], used bytest-viz-gpu,test-cloudxr,test-teleop-ros2. Gated only onneeds: build-ubuntu, so they do run on PRs. - ctest never runs on them. It runs only in
build-ubuntu, unfiltered. test-viz-gpuis the precedent worth copying: viz python tests carrypytestmark = skipif(not _gpu_available()), skip cleanly on the CPU runner, and the same files are re-run withpytestagainst the built wheel on the GPU runner.test-cloudxrcannot host these as-is — the runtime is a separate compose service and the run dir is mounted:ro, so no lifecycle test can write to it.
Direction
Add a real-runtime tier to cloudxr_tests that skips when no runtime is available, and run it on the GPU runner the way test-viz-gpu runs the viz suite. Keep mocks only for branches a real runtime cannot produce (wedged service, stale pid).
Related: #880 (hand-registered GPU example tests — same root cause on the examples/ side). Stubs introduced by #927.
- Lenguaje dominante
- Python
- Estrellas
- 403
- Forks
- 94
- Merge medio
- 2 d 17 min
- PR fusionados (30 d)
- 121
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de NVIDIA/IsaacCapture
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
NVIDIA/IsaacCapture#910 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
NVIDIA/IsaacCapture#891 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
NVIDIA/IsaacCapture#1217 ·
Los mantenedores suelen responder en 1 día
-
Release Notes Draft: 1.5Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 20/100
NVIDIA/IsaacCapture#1214 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 50/100
NVIDIA/IsaacCapture#1181 ·
Los mantenedores suelen responder en 1 día
Todos los issues de NVIDIA/IsaacCapture
Issues similares
-
area:core-platform effort:S priority:P3 status:in-review type:test
Dificultad 2/5 Menos de una hora Aptitud para principiantes 80/100
randomparity/kdive#3246 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Requantification library converts m/z to mass with the hydrogen-atom mass instead of the proton massAbiertobug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
nf-core/metaboigniter#130 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
common-workflow-language/cwl-utils#477 ·
Los mantenedores suelen responder en 1 día
-
Add dependabotAbiertofeat
Dificultad 2/5 Menos de una hora Aptitud para principiantes 72/100
open-nudge/opentemplate#204 ·
Los mantenedores suelen responder en 1 día
-
github-stale-ci-pr-closer: KV state is double-wrapped, never persists, and hits the depth-32 limitPosiblemente ocupada @VascoSch92 la tomó hoy. Abiertopriority:medium ready-for-dev
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
OpenHands/extensions#781 ·
Los mantenedores suelen responder en 1 día