Skip to content

Add ownerReference to NodeFeature CRs for garbage collection#1597

Merged
rajathagasthya merged 1 commit intomainfrom
feature/nodefeature-ownerref
Jan 26, 2026
Merged

Add ownerReference to NodeFeature CRs for garbage collection#1597
rajathagasthya merged 1 commit intomainfrom
feature/nodefeature-ownerref

Conversation

@rajathagasthya
Copy link
Copy Markdown
Contributor

Adds the Pod and Daemonset (controller reference) as owners, so that the NodeFeature CR is deleted when the GFD daemonset is deleted. This mirrors the behavior of how NFD adds ownerReferences to NodeFeature CRs it creates.

Comment thread internal/lm/output.go
Comment thread internal/lm/output.go Outdated
Comment thread internal/lm/output.go Outdated
@rajathagasthya rajathagasthya force-pushed the feature/nodefeature-ownerref branch from 53edb01 to 997a9e9 Compare January 23, 2026 16:44
Copy link
Copy Markdown
Member

@karthikvetrivel karthikvetrivel left a comment

Choose a reason for hiding this comment

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

LGTM!

Adds the Pod and Daemonset (controller reference) as owners, so that the
NodeFeature CR is deleted when the GFD daemonset is deleted. This
mirrors the behavior of how NFD adds ownerReferences to NodeFeature CRs
it creates.

Signed-off-by: Rajath Agasthya <[email protected]>
@rajathagasthya rajathagasthya force-pushed the feature/nodefeature-ownerref branch from 997a9e9 to 3968b3d Compare January 23, 2026 16:46
Comment thread deployments/helm/nvidia-device-plugin/templates/daemonset-gfd.yml
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.

3 participants