Mellanox / network-operator / 32228652704

63%

Build Type

push

github

Committed by web-flow

Commit Message

fix: hash NicNodePolicy DaemonSet names (#3027)

## Summary

- use one deterministic policy-name hash suffix for NicNodePolicy-owned
OFED, RDMA shared device plugin, and SR-IOV device plugin DaemonSets
- preserve all existing NicClusterPolicy DaemonSet names
- render NNP DaemonSets as:
  - `mofed-<os>-<kernel-hash>-<policy-name-hash>-ds`
  - `rdma-shared-dp-ds-<policy-name-hash>`
  - `network-operator-sriov-device-plugin-<policy-name-hash>`
- use the hashed OFED identity consistently in DaemonSet metadata,
selectors, and pod labels
- preserve readable `ds-owner` values, policy-specific ConfigMap names,
and the RDMA `app` selector used by SOS report discovery
- centralize the shared hashed suffix helper and update focused render
tests and heterogeneous-cluster documentation

This is a follow-up to #3019 and replaces direct policy-name suffixes in
NicNodePolicy-owned DaemonSet names.

## Verification

- `CGO_ENABLED=0 go build ./...`
- `CGO_ENABLED=0 go test ./pkg/state/... -count=1`
- `CGO_ENABLED=0 make lint`
- `bash -n scripts/sosreport/generate-maps.sh
scripts/sosreport/kubectl-netop_sosreport
scripts/sosreport/test-sosreport.sh`
- verified the OFED SOS component selector remains
`nvidia.com/ofed-driver=`
- verified the RDMA SOS component key remains `rdma-shared-dp-ds` with
selector `app=rdma-shared-dp__NAME_SUFFIX__`

Coverage Stats

18 of 19 new or added lines in 4 files covered. (94.74%)

43 existing lines in 5 files now uncovered.

4999 of 8003 relevant lines covered (62.46%)

44.07 hits per line