Skip to content

Conversation

cdesiniotis
Copy link
Contributor

These permissions are required when GFD is configured to use the Node Feature API instead of feature files.

tariq1890
tariq1890 previously approved these changes Oct 10, 2025
@cdesiniotis
Copy link
Contributor Author

I just realized this is not sufficient. We will also have to add these permissions to the gpu-operator cluster role as well -- otherwise it won't be able to create the GFD cluster role.

@tariq1890 tariq1890 dismissed their stale review October 10, 2025 19:56

Needs more changes

These permissions are required when GFD is configured to use the Node Feature API instead of feature files.

Signed-off-by: Christopher Desiniotis <[email protected]>
@cdesiniotis cdesiniotis force-pushed the gfd-clusterperms-for-node-features branch from 1695c5b to 2a419fc Compare October 10, 2025 20:05
@cdesiniotis cdesiniotis changed the title Add permissions for nodefeatures in GFD's cluster role Add permissions for nodefeatures in GFD's role object Oct 10, 2025
@cdesiniotis
Copy link
Contributor Author

I realized that NodeFeatures are a namespaced resource. I have moved the permissions to the role objects for gpu-operator and GFD respectively.

@cdesiniotis cdesiniotis requested a review from tariq1890 October 10, 2025 20:21
@cdesiniotis cdesiniotis enabled auto-merge October 10, 2025 20:27
@cdesiniotis cdesiniotis merged commit 1f98e1d into NVIDIA:main Oct 10, 2025
16 checks passed
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.

2 participants