Skip to content

Adding missing cdi-hooks in readme - #1998

Merged
henry118 merged 1 commit into
NVIDIA:mainfrom
henry118:hook-doc
Aug 17, 2026
Merged

henry118 merged 1 commit into
NVIDIA:mainfrom
henry118:hook-doc

Conversation

@henry118

Copy link
Copy Markdown
Member

Add the newly added hooks into the readme file.

@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 31850129937

Coverage remained the same at 43.626%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 15266
Covered Lines: 6660
Line Coverage: 43.63%
Coverage Strength: 0.49 hits per line

💛 - Coveralls

Comment thread cmd/nvidia-cdi-hook/README.md
@tariq1890

tariq1890 commented Aug 14, 2026 •

Copy link
Copy Markdown
Contributor

nit: Can we order the hook names lexically?

@henry118

henry118 commented Aug 14, 2026 •

Copy link
Copy Markdown
Member Author

nit: Can we order the hook names lexically?

tbh, I don't think it's necessary. And it would require even more effort to make changes in the future.

@henry118
henry118 requested a review from cdesiniotis August 15, 2026 00:35
Comment thread cmd/nvidia-cdi-hook/README.md Outdated
* `chmod` - Change the permissions of a file or directory inside the directory path to be mounted into a container.
* `create-symlinks` - Create symlinks inside the directory path to be mounted into a container.
* `update-ldcache` - Update the dynamic linker cache inside the directory path to be mounted into a container.
* `enable-cuda-compat` - Ensure that the directory containing the CUDA compat libraries is added to the ldconfig search path.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* `enable-cuda-compat` - Ensure that the directory containing the CUDA compat libraries is added to the ldconfig search path.
* `enable-cuda-compat` - Ensure that the directory containing the CUDA compat libraries is added to the ldconfig search path if required.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated

Signed-off-by: Henry Wang <henrwang@nvidia.com>
@henry118
henry118 merged commit 5eee4c5 into NVIDIA:main Aug 17, 2026
21 of 22 checks passed
@henry118
henry118 deleted the hook-doc branch August 17, 2026 20:13
@tariq1890

Copy link
Copy Markdown
Contributor

/cherry-pick release-1.20

@github-actions

Copy link
Copy Markdown

🤖 Backport PR created for release-1.20: #2092 ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants