Skip to content

feat: add HostNetwork field to DevicePlugin and MetricsExporter#465

Merged
sajmera-pensando merged 2 commits intoROCm:mainfrom
uMetalooper:feature/host-network-device-plugin-metrics-exporter
Mar 13, 2026
Merged

feat: add HostNetwork field to DevicePlugin and MetricsExporter#465
sajmera-pensando merged 2 commits intoROCm:mainfrom
uMetalooper:feature/host-network-device-plugin-metrics-exporter

Conversation

@uMetalooper
Copy link
Contributor

@uMetalooper uMetalooper commented Mar 9, 2026

Summary

In Uber, we use host networking for our setup instead of relying on k8s cluster networking. So it would be great to allow daemonsets to communicate via host network namespace.

  • Add optional HostNetwork *bool field to DevicePluginSpec in the DeviceConfig CRD
  • Add optional HostNetwork *bool field to MetricsExporterSpec in the DeviceConfig CRD
  • Wire up HostNetwork in internal/plugin/plugin.go to set HostNetwork on the device plugin DaemonSet pod spec
  • Wire up HostNetwork in internal/metricsexporter/metricsexporter.go to set HostNetwork on the metrics exporter DaemonSet pod spec

Test plan

  • Verify device plugin DaemonSet respects hostNetwork: true when set in DeviceConfig spec
  • Verify metrics exporter DaemonSet respects hostNetwork: true when set in DeviceConfig spec
  • Verify default behavior (no hostNetwork set) is unchanged

🤖 Generated with Claude Code

@yansun1996 yansun1996 force-pushed the feature/host-network-device-plugin-metrics-exporter branch 2 times, most recently from aa29c7c to 8ea40d0 Compare March 11, 2026 05:33
@yansun1996 yansun1996 force-pushed the feature/host-network-device-plugin-metrics-exporter branch from f7ca301 to f4cd475 Compare March 11, 2026 23:37
uMetalooper and others added 2 commits March 11, 2026 23:37
Add optional HostNetwork field to DevicePluginSpec and MetricsExporterSpec
in the DeviceConfig CRD, allowing users to configure host network namespace
for the respective DaemonSet pods.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: yansun1996 <Yan.Sun3@amd.com>
@yansun1996 yansun1996 force-pushed the feature/host-network-device-plugin-metrics-exporter branch from f4cd475 to ec2fc0d Compare March 11, 2026 23:38
@sajmera-pensando sajmera-pensando merged commit 92e0d5d into ROCm:main Mar 13, 2026
3 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.

3 participants