search

apache / arrow-nanoarrow

Helpers for Arrow C Data & Arrow C Stream interfaces
https://arrow.apache.org/nanoarrow
Apache License 2.0
169 stars 35 forks source link

chore(ci): Add CUDA runner for testing device extension #490

Closed paleolimbot closed 3 months ago

paleolimbot commented 4 months ago

This PR adds the INFRA-provided runner ["self-hosted", "cuda"] so that PRs that modify CUDA code get better CI feedback (before this, the extension pretty much was only ever built on my local setup).

The self-hosted runner is running an older linux than the usual ubuntu-latest, so a few modifications were needed to reflect compiler warning policies. The cache key we were using for Arrow C++ is also no longer sufficient for the device workflow because a cache restore results in an unusable Arrow C++ static library on the self-hosted runner. This shouldn't be necessary for much longer (just until Arrow C++-dependent tests are factored out and run separately).

lidavidm commented 4 months ago

Might need INFRA to enable the CUDA runners on this repo? Not sure who controls that