NVIDIA / NVIDIA/cuda-python

[FEA]: Improve CI coverage of cuFile

Open
#1,143 4 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

blocked CI/CD cuda.bindings feature
Dominant language
Cython
Stars
3.4k
Forks
329
Avg merge
1d 23h
Merged PRs (30d)
116

Description

Is this a duplicate?
Area

cuda.bindings

Is your feature request related to a problem? Please describe.

The majority of the tests in test_cufile.py are skipped because:

@pytest.mark.skipif(not isSupportedFilesystem(), reason="cuFile handle_register requires ext4 or xfs filesystem")

Maybe we should try to add a worker to the mix that /is/ on one of these filesystems to get some better coverage there. I don't know what filesystem our existing workers have (maybe btrfs?)

Describe the solution you'd like

Find a worker that meets the requirements of cuFile that we can run on.

Describe alternatives you've considered

No response

Additional context

No response

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with test_cufile.py and the isSupportedFilesystem skip condition, then inspect the repository's CI worker definitions to determine which workers use ext4 or xfs. Done means a suitable worker is identified and the cuFile tests run there instead of being skipped.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
ci-cd, testing-qa
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.