scott/es24n-conf
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix fabric firmware version path — add VikingEnterpriseSolutions key

Browse Source 
The Redfish response nests the version under
Oem.VikingEnterpriseSolutions.Version.ActiveFirmwareVersion, not
Oem.Version.ActiveFirmwareVersion. Fixed in both redfish.py
(_get_fabric_fw_version, used by network check and firmware workflow)
and workflow_check.py (serial inline parsing).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
scott
2026-04-16 11:28:07 -04:00
parent f08550dbea
commit 6912680ed4
2 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
modules/redfish.py
View File

@@ -208,6 +208,7 @@ def _get_fabric_fw_version(password: str, host: str, iom: str) -> str:
) )
if ok_flag and isinstance(data, dict): if ok_flag and isinstance(data, dict):
version = (data.get("Oem", {}) version = (data.get("Oem", {})
.get("VikingEnterpriseSolutions", {})
.get("Version", {}) .get("Version", {})
.get("ActiveFirmwareVersion")) .get("ActiveFirmwareVersion"))
return version or "Unknown" return version or "Unknown"

1
modules/workflow_check.py
View File

@@ -151,6 +151,7 @@ def _check_via_serial():
) )
fab_ver = ( fab_ver = (
(fab_data.get("Oem", {}) (fab_data.get("Oem", {})
.get("VikingEnterpriseSolutions", {})
.get("Version", {}) .get("Version", {})
.get("ActiveFirmwareVersion", "Unknown")) .get("ActiveFirmwareVersion", "Unknown"))
if (fab_ok and isinstance(fab_data, dict)) if (fab_ok and isinstance(fab_data, dict))