Skip to content

Comments

Add Radeon AI PRO R9700 to gpu-nfd-default-rule files#416

Open
tzmtl wants to merge 1 commit intoROCm:mainfrom
tzmtl:AI-PRO-9700
Open

Add Radeon AI PRO R9700 to gpu-nfd-default-rule files#416
tzmtl wants to merge 1 commit intoROCm:mainfrom
tzmtl:AI-PRO-9700

Conversation

@tzmtl
Copy link

@tzmtl tzmtl commented Jan 22, 2026

Motivation

The pci id of new released Radeon AI PRO R9700 is 1002:7551 7551 is missing in gpu-nfd-default-rule files, so nfd cannot find AMD GPU on a node with Radeon AI PRO R9700.

Technical Details

Modified 2 rule files
hack/k8s-patch/template-patch/gpu-nfd-default-rule.yaml
helm-charts-k8s/templates/gpu-nfd-default-rule.yaml

Add a section of id 7551

Test Plan

Tested with a real k8s cluster with Radeon AI PRO R9700

Test Result

It's able to recognize the AMD GPU with nfd and gpu device plugin is created on these nodes

@yansun1996 yansun1996 self-requested a review February 5, 2026 03:14
Copy link
Member

@yansun1996 yansun1996 left a comment

Choose a reason for hiding this comment

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

LGTM

@tzmtl
Copy link
Author

tzmtl commented Feb 20, 2026

LGTM
Thanks for your review!

Now merge is blocked by spelling check on documentation? I didn't touch documentation. How can I fix and move on?

@yansun1996
Copy link
Member

LGTM

Thanks for your review!

Now merge is blocked by spelling check on documentation? I didn't touch documentation. How can I fix and move on?

We just pushed some fixes to the CI test, could you pls rebase your PR ?

The pci id of new released Radeon AI PRO R9700 is 1002:7551
7551 is missing in gpu-nfd-default-rule files, so nfd cannot
find AMD GPU on a node with Radeon AI PRO R9700.
@tzmtl
Copy link
Author

tzmtl commented Feb 20, 2026

LGTM

Thanks for your review!
Now merge is blocked by spelling check on documentation? I didn't touch documentation. How can I fix and move on?

We just pushed some fixes to the CI test, could you pls rebase your PR ?

It's rebased

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