Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix NVIDIA_IMEX_CHANNELS handling on legacy images #799

Merged
merged 1 commit into from
Nov 15, 2024

Conversation

elezar
Copy link
Member

@elezar elezar commented Nov 14, 2024

For legacy images (images with a CUDA_VERSION set but no CUDA_REQUIRES set), the default behaviour for device envvars is to treat non-existence as all.

This change ensures that the NVIDIA_IMEX_CHANNELS envvar is not treated in the same way, instead returning no devices if the envvar is not set.

See #797

For legacy images (images with a CUDA_VERSION set but no CUDA_REQUIRES set), the
default behaviour for device envvars is to treat non-existence as all.

This change ensures that the NVIDIA_IMEX_CHANNELS envvar is not treated in the same
way, instead returning no devices if the envvar is not set.

Signed-off-by: Evan Lezar <[email protected]>
@elezar elezar self-assigned this Nov 14, 2024
@elezar elezar requested a review from tariq1890 November 14, 2024 20:33
@elezar elezar added the must-backport The changes in PR need to be backported to at least one stable release branch. label Nov 14, 2024
Copy link
Contributor

@klueska klueska left a comment

Choose a reason for hiding this comment

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

It's a little weird that unset == all, but I see how this change works, and its fine for now.

@elezar
Copy link
Member Author

elezar commented Nov 14, 2024

It's a little weird that unset == all, but I see how this change works, and its fine for now.

Note that this is for legacy images only. We could look into deprecating support for those to simplify our reasoning about these cases.

@elezar elezar merged commit 5c3ffc2 into NVIDIA:main Nov 15, 2024
10 checks passed
@astefanutti
Copy link

@elezar @klueska thank you for the quick fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
must-backport The changes in PR need to be backported to at least one stable release branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants