Merge pull request #116 from Cray-HPE/CASMTRIAGE-5914

Fix HSN NIC numbering

.version

@@ -1 +1 @@
-1.58.3
+1.58.4

CHANGELOG.md

@@ -5,6 +5,12 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.58.4] - 2023-08-21
+
+### Fixed
+
+- Fixed bug in HSN NIC numbering.
+
 ## [1.58.3] - 2023-04-12
 
 ### Fixed

pkg/redfish/rfcomponents-hpe.go

@@ -256,9 +256,9 @@ func (ep *RedfishEP) getHpeDeviceOrdinal(d *EpHpeDevice) int {
 					// Cassini shows as HPE instead of BEI in Proliant iLO redfish
 					// implementations so we check for both just incase this changes
 					// in the future. 
-					if strings.Contains(strings.ToLower(d.DeviceRF.Manufacturer), "mellanox") ||
-					   strings.Contains(strings.ToLower(d.DeviceRF.Manufacturer), "hpe") ||
-					   strings.Contains(strings.ToLower(d.DeviceRF.Manufacturer), "bei") {
+					if strings.Contains(strings.ToLower(device.DeviceRF.Manufacturer), "mellanox") ||
+					   strings.Contains(strings.ToLower(device.DeviceRF.Manufacturer), "hpe") ||
+					   strings.Contains(strings.ToLower(device.DeviceRF.Manufacturer), "bei") {
 						dsOIDs = append(dsOIDs, oid)
 					}
 				} else {