Skip to content

remove dependency on /etc/os-release host mount#2138

Merged
tariq1890 merged 1 commit intomainfrom
rm-os-release
Feb 19, 2026
Merged

remove dependency on /etc/os-release host mount#2138
tariq1890 merged 1 commit intomainfrom
rm-os-release

Conversation

@tariq1890
Copy link
Contributor

@tariq1890 tariq1890 commented Feb 19, 2026

This PR builds on top of #2101 - using the NFD labels to get the /etc/os-release information instead of directly querying the node's /etc/os-release file via a host mount.

With the existing approach, the gpu-operator assumes that the node on which the gpu-operator pod is scheduled has the same OS as the GPU node. There are cases where users may choose to deploy the gpu-operator pod on a CPU node having an OS different from that of the GPU node. These changes enable the gpu-operator to function in these scenarios.

@coveralls
Copy link

coveralls commented Feb 19, 2026

Coverage Status

coverage: 26.161% (+0.02%) from 26.142%
when pulling 14a6a02 on rm-os-release
into 2665c10 on main.

@tariq1890 tariq1890 force-pushed the rm-os-release branch 2 times, most recently from 580d036 to 36717e5 Compare February 19, 2026 05:08
Signed-off-by: Tariq Ibrahim <tibrahim@nvidia.com>
Copy link
Contributor

@rahulait rahulait left a comment

Choose a reason for hiding this comment

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

LGTM

@tariq1890 tariq1890 merged commit dcb3f47 into main Feb 19, 2026
17 checks passed
@tariq1890 tariq1890 deleted the rm-os-release branch February 19, 2026 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

Comments