k8s-device-plugin icon indicating copy to clipboard operation
k8s-device-plugin copied to clipboard

Published 20 hours ago • verified publisher icon ROCm

Issues in helm chart

Open boniek83 opened this issue 3 years ago • 1 comments

Helm Chart version: 0.0.2 By DaemonSet I mean k8s-device-plugin DS. DaemonSet should only create pods on nodes that have AMD hardware (should have and respect node_selector value from default values file -it doesn't at this time) DaemonSet should have resources section (to limit resource usage) DaemonSet should have priorityClassName: system-node-critical set (annotation is deprecated: https://github.com/kubernetes/website/commit/315c716774af079afa0c0479ff4513549c48a5e9/) It would be better (although not critical as long as it can be overridden) if default node_selector was compatible with how NVidias bundled NFD behaves (for clusters with mixed GPUs) and have default value of: feature.node.kubernetes.io/pci-1002.present=true Using latest tag for labeller is bad idea - this is not reproducible.

boniek83 avatar May 05 '21 22:05 boniek83

I've created a pull request to fix this issue https://github.com/RadeonOpenCompute/k8s-device-plugin/pull/29 that I'm suffering too.

julian3xl avatar Oct 11 '22 13:10 julian3xl