Skip to content

fix: add missing device-plugin ServiceAccount for OLM deployments (NETOP-174) (#328)#94

Merged
sajmera-pensando merged 1 commit into
mainfrom
CP.O2O.pensando.network-operator.328.rocm.network-operator.main
Jun 11, 2026
Merged

fix: add missing device-plugin ServiceAccount for OLM deployments (NETOP-174) (#328)#94
sajmera-pensando merged 1 commit into
mainfrom
CP.O2O.pensando.network-operator.328.rocm.network-operator.main

Conversation

@yuva29

@yuva29 yuva29 commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

Cherry-pick of https://github.com/pensando/network-operator/pull/328

The device-plugin DaemonSet failed on OpenShift OLM deployments because the amd-network-operator-device-plugin ServiceAccount was never created. SCC rules for device-plugin, cni-plugins, and kmm-module-loader were in the CSV permissions section (namespace-scoped RoleBindings) instead of clusterPermissions (ClusterRoleBindings), which is required for SCC access.

  • Move device-plugin, cni-plugins, kmm-module-loader SCC rules to clusterPermissions in the CSV
  • Remove unused kmm-device-plugin SA (GPU operator leftover)
  • Fix RoleBinding → ClusterRoleBinding in kustomize and Helm RBAC
  • Add missing cni-plugins SA and RBAC to OpenShift Helm chart

Note: The bundle/manifests/amd-network-operator.clusterserviceversion.yaml change was dropped since the bundle/ directory does not exist on ROCm main.

…TOP-174) (#328)

The device-plugin DaemonSet failed on OpenShift OLM deployments because
the amd-network-operator-device-plugin ServiceAccount was never created.
SCC rules for device-plugin, cni-plugins, and kmm-module-loader were in
the CSV permissions section (namespace-scoped RoleBindings) instead of
clusterPermissions (ClusterRoleBindings), which is required for SCC access.

- Move device-plugin, cni-plugins, kmm-module-loader SCC rules to
  clusterPermissions in the CSV
- Remove unused kmm-device-plugin SA (GPU operator leftover)
- Fix RoleBinding → ClusterRoleBinding in kustomize and Helm RBAC
- Add missing cni-plugins SA and RBAC to OpenShift Helm chart

Co-authored-by: Yuva Shankar <11082310+yuva29@users.noreply.github.com>
Co-authored-by: Claude Opus 4 (1M context) <noreply@anthropic.com>
# Conflicts:
#	bundle/manifests/amd-network-operator.clusterserviceversion.yaml
@sajmera-pensando sajmera-pensando merged commit aa96667 into main Jun 11, 2026
2 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