Compare commits
13 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| b04877549a | |||
| 8ca173c99b | |||
| f19a3454fa | |||
|
|
becdca1d7e | ||
|
|
e10440ae32 | ||
| 5c2a21aff1 | |||
|
|
9df13327aa | ||
|
|
7e9af89c46 | ||
|
|
db74df9994 | ||
|
|
bb82387d48 | ||
|
|
475f6ac472 | ||
|
|
93ce676f04 | ||
|
|
c47c34fd11 |
2
bible
2
bible
Submodule bible updated: 52444350c1...456c1f022c
@@ -34,6 +34,7 @@ All modes converge on the same normalized hardware model and exporter pipeline.
|
|||||||
- NVIDIA HGX Field Diagnostics
|
- NVIDIA HGX Field Diagnostics
|
||||||
- NVIDIA Bug Report
|
- NVIDIA Bug Report
|
||||||
- Unraid
|
- Unraid
|
||||||
|
- xFusion iBMC dump / file export
|
||||||
- XigmaNAS
|
- XigmaNAS
|
||||||
- Generic fallback parser
|
- Generic fallback parser
|
||||||
|
|
||||||
|
|||||||
@@ -58,6 +58,7 @@ Responses:
|
|||||||
|
|
||||||
Optional request field:
|
Optional request field:
|
||||||
- `power_on_if_host_off`: when `true`, Redfish collection may power on the host before collection if preflight found it powered off
|
- `power_on_if_host_off`: when `true`, Redfish collection may power on the host before collection if preflight found it powered off
|
||||||
|
- `debug_payloads`: when `true`, collector keeps extra diagnostic payloads and enables extended plan-B retries for slow HGX component inventory branches (`Assembly`, `Accelerators`, `Drives`, `NetworkAdapters`, `PCIeDevices`)
|
||||||
|
|
||||||
### `POST /api/collect/probe`
|
### `POST /api/collect/probe`
|
||||||
|
|
||||||
|
|||||||
@@ -27,6 +27,7 @@ Request fields passed from the server:
|
|||||||
- credential field (`password` or token)
|
- credential field (`password` or token)
|
||||||
- `tls_mode`
|
- `tls_mode`
|
||||||
- optional `power_on_if_host_off`
|
- optional `power_on_if_host_off`
|
||||||
|
- optional `debug_payloads` for extended diagnostics
|
||||||
|
|
||||||
### Core rule
|
### Core rule
|
||||||
|
|
||||||
@@ -35,18 +36,38 @@ If the collector adds a fallback, probe, or normalization rule, replay must mirr
|
|||||||
|
|
||||||
### Preflight and host power
|
### Preflight and host power
|
||||||
|
|
||||||
- `Probe()` may be used before collection to verify API connectivity and current host `PowerState`
|
- `Probe()` is used before collection to verify API connectivity and report current host `PowerState`
|
||||||
- if the host is off and the user chose power-on, the collector may issue `ComputerSystem.Reset`
|
- if the host is off, the collector logs a warning and proceeds with collection; inventory data may
|
||||||
with `ResetType=On`
|
be incomplete when the host is powered off
|
||||||
- power-on attempts are bounded and logged
|
- power-on and power-off are not performed by the collector
|
||||||
- after a successful power-on, the collector waits an extra stabilization window, then checks
|
|
||||||
`PowerState` again and only starts collection if the host is still on
|
### Skip hung requests
|
||||||
- if the collector powered on the host itself for collection, it must attempt to power it back off
|
|
||||||
after collection completes
|
Redfish collection uses a two-level context model:
|
||||||
- if the host was already on before collection, the collector must not power it off afterward
|
|
||||||
- if power-on fails, collection still continues against the powered-off host
|
- `ctx` — job lifetime context, cancelled only on explicit job cancel
|
||||||
- all power-control decisions and attempts must be visible in the collection log so they are
|
- `collectCtx` — collection phase context, derived from `ctx`; covers snapshot, prefetch, and plan-B
|
||||||
preserved in raw-export bundles
|
|
||||||
|
`collectCtx` is cancelled when the user presses "Пропустить зависшие" (skip hung).
|
||||||
|
On skip, all in-flight HTTP requests in the current phase are aborted immediately via context
|
||||||
|
cancellation, the crawler and plan-B loops exit, and execution proceeds to the replay phase using
|
||||||
|
whatever was collected in `rawTree`. The result is partial but valid.
|
||||||
|
|
||||||
|
The skip signal travels: UI button → `POST /api/collect/{id}/skip` → `JobManager.SkipJob()` →
|
||||||
|
closes `skipCh` → goroutine in `Collect()` → `cancelCollect()`.
|
||||||
|
|
||||||
|
The skip button is visible during `running` state and hidden once the job reaches a terminal state.
|
||||||
|
|
||||||
|
### Extended diagnostics toggle
|
||||||
|
|
||||||
|
The live collect form exposes a user-facing checkbox for extended diagnostics.
|
||||||
|
|
||||||
|
- default collection prioritizes inventory completeness and bounded runtime
|
||||||
|
- when extended diagnostics is off, heavy HGX component-chassis critical plan-B retries
|
||||||
|
(`Assembly`, `Accelerators`, `Drives`, `NetworkAdapters`, `PCIeDevices`) are skipped
|
||||||
|
- when extended diagnostics is on, those retries are allowed and extra debug payloads are collected
|
||||||
|
|
||||||
|
This toggle is intended for operator-driven deep diagnostics on problematic hosts, not for the default path.
|
||||||
|
|
||||||
### Discovery model
|
### Discovery model
|
||||||
|
|
||||||
|
|||||||
@@ -58,6 +58,7 @@ When `vendor_id` and `device_id` are known but the model name is missing or gene
|
|||||||
| `nvidia` | HGX Field Diagnostics | GPU- and fabric-heavy diagnostic input |
|
| `nvidia` | HGX Field Diagnostics | GPU- and fabric-heavy diagnostic input |
|
||||||
| `nvidia_bug_report` | `nvidia-bug-report-*.log.gz` | dmidecode, lspci, NVIDIA driver sections |
|
| `nvidia_bug_report` | `nvidia-bug-report-*.log.gz` | dmidecode, lspci, NVIDIA driver sections |
|
||||||
| `unraid` | Unraid diagnostics/log bundles | Server and storage-focused parsing |
|
| `unraid` | Unraid diagnostics/log bundles | Server and storage-focused parsing |
|
||||||
|
| `xfusion` | xFusion iBMC `tar.gz` dump / file export | AppDump + RTOSDump + LogDump merge for hardware and firmware |
|
||||||
| `xigmanas` | XigmaNAS plain logs | FreeBSD/NAS-oriented inventory |
|
| `xigmanas` | XigmaNAS plain logs | FreeBSD/NAS-oriented inventory |
|
||||||
| `generic` | fallback | Low-confidence text fallback when nothing else matches |
|
| `generic` | fallback | Low-confidence text fallback when nothing else matches |
|
||||||
|
|
||||||
@@ -148,6 +149,29 @@ entire internal `zbb` schema.
|
|||||||
|
|
||||||
---
|
---
|
||||||
|
|
||||||
|
### xFusion iBMC Dump / File Export (`xfusion`)
|
||||||
|
|
||||||
|
**Status:** Ready (v1.1.0). Tested on xFusion G5500 V7 `tar.gz` exports.
|
||||||
|
|
||||||
|
**Archive format:** `tar.gz` dump exported from the iBMC UI, including `AppDump/`, `RTOSDump/`,
|
||||||
|
and `LogDump/` trees.
|
||||||
|
|
||||||
|
**Detection:** `AppDump/FruData/fruinfo.txt`, `AppDump/card_manage/card_info`,
|
||||||
|
`RTOSDump/versioninfo/app_revision.txt`, and `LogDump/netcard/netcard_info.txt`.
|
||||||
|
|
||||||
|
**Extracted data (current):**
|
||||||
|
- Board / FRU inventory from `fruinfo.txt`
|
||||||
|
- CPU inventory from `CpuMem/cpu_info`
|
||||||
|
- Memory DIMM inventory from `CpuMem/mem_info`
|
||||||
|
- GPU inventory from `card_info`
|
||||||
|
- OCP NIC inventory by merging `card_info` with `LogDump/netcard/netcard_info.txt`
|
||||||
|
- PSU inventory from `BMC/psu_info.txt`
|
||||||
|
- Physical storage from `StorageMgnt/PhysicalDrivesInfo/*/disk_info`
|
||||||
|
- System firmware entries from `RTOSDump/versioninfo/app_revision.txt`
|
||||||
|
- Maintenance events from `LogDump/maintenance_log`
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
### Generic text fallback (`generic`)
|
### Generic text fallback (`generic`)
|
||||||
|
|
||||||
**Status:** Ready (v1.0.0).
|
**Status:** Ready (v1.0.0).
|
||||||
@@ -173,6 +197,7 @@ entire internal `zbb` schema.
|
|||||||
| NVIDIA HGX Field Diag | `nvidia` | Ready | Various HGX servers |
|
| NVIDIA HGX Field Diag | `nvidia` | Ready | Various HGX servers |
|
||||||
| NVIDIA Bug Report | `nvidia_bug_report` | Ready | H100 systems |
|
| NVIDIA Bug Report | `nvidia_bug_report` | Ready | H100 systems |
|
||||||
| Unraid | `unraid` | Ready | Unraid diagnostics archives |
|
| Unraid | `unraid` | Ready | Unraid diagnostics archives |
|
||||||
|
| xFusion iBMC dump | `xfusion` | Ready | G5500 V7 file-export `tar.gz` bundles |
|
||||||
| XigmaNAS | `xigmanas` | Ready | FreeBSD NAS logs |
|
| XigmaNAS | `xigmanas` | Ready | FreeBSD NAS logs |
|
||||||
| H3C SDS G5 | `h3c_g5` | Ready | H3C UniServer R4900 G5 SDS archives |
|
| H3C SDS G5 | `h3c_g5` | Ready | H3C UniServer R4900 G5 SDS archives |
|
||||||
| H3C SDS G6 | `h3c_g6` | Ready | H3C UniServer R4700 G6 SDS archives |
|
| H3C SDS G6 | `h3c_g6` | Ready | H3C UniServer R4700 G6 SDS archives |
|
||||||
|
|||||||
@@ -994,9 +994,146 @@ significant complexity before proving user value.
|
|||||||
- decode the outer `ABJR` container
|
- decode the outer `ABJR` container
|
||||||
- gunzip embedded members when applicable
|
- gunzip embedded members when applicable
|
||||||
- extract inventory from printable SMBIOS/FRU payloads
|
- extract inventory from printable SMBIOS/FRU payloads
|
||||||
- extract storage/controller details from embedded Redfish JSON objects
|
- extract storage/controller/backplane details from embedded Redfish JSON objects
|
||||||
|
- enrich firmware and PSU inventory from auxiliary package payloads such as `bcert.pkg`
|
||||||
- do not attempt complete semantic decoding of the internal `zbb` record format
|
- do not attempt complete semantic decoding of the internal `zbb` record format
|
||||||
**Consequences:**
|
**Consequences:**
|
||||||
- Parser reaches inventory-grade usefulness quickly for HPE `.ahs` uploads.
|
- Parser reaches inventory-grade usefulness quickly for HPE `.ahs` uploads.
|
||||||
- Storage inventory is stronger than text-only parsing because it reuses structured Redfish data when present.
|
- Storage inventory is stronger than text-only parsing because it reuses structured Redfish data when present.
|
||||||
|
- Auxiliary package payloads can supply missing firmware/PSU fields even when the main SMBIOS-like blob is incomplete.
|
||||||
- Future deeper `zbb` decoding can be added incrementally without replacing the current parser contract.
|
- Future deeper `zbb` decoding can be added incrementally without replacing the current parser contract.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## ADL-039 — Canonical inventory keeps DIMMs with unknown capacity when identity is known
|
||||||
|
|
||||||
|
**Date:** 2026-03-30
|
||||||
|
**Context:** Some sources, notably HPE iLO AHS SMBIOS-like blobs, expose installed DIMM identity
|
||||||
|
(slot, serial, part number, manufacturer) but do not include capacity. The parser already extracts
|
||||||
|
those modules into `Hardware.Memory`, but canonical device building and export previously dropped
|
||||||
|
them because `size_mb == 0`.
|
||||||
|
**Decision:** Treat a DIMM as installed inventory when `present=true` and it has identifying
|
||||||
|
memory fields such as serial number or part number, even if `size_mb` is unknown.
|
||||||
|
**Consequences:**
|
||||||
|
- HPE AHS uploads now show real installed memory modules instead of hiding them.
|
||||||
|
- Empty slots still stay filtered because they lack inventory identity or are marked absent.
|
||||||
|
- Specification/export can include "size unknown" memory entries without inventing capacity data.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## ADL-040 — HPE Redfish normalization prefers chassis `Devices/*` over generic PCIe topology labels
|
||||||
|
|
||||||
|
**Date:** 2026-03-30
|
||||||
|
**Context:** HPE ProLiant Gen11 Redfish snapshots expose parallel inventory trees. `Chassis/*/PCIeDevices/*`
|
||||||
|
is good for topology presence, but often reports only generic `DeviceType` values such as
|
||||||
|
`SingleFunction`. `Chassis/*/Devices/*` carries the concrete slot label, richer device type, and
|
||||||
|
product-vs-spare part identifiers for the same physical NIC/controller. Replay fallback over empty
|
||||||
|
storage volume collections can also discover `Volumes/Capabilities` children, which are not real
|
||||||
|
logical volumes.
|
||||||
|
|
||||||
|
**Decision:**
|
||||||
|
- Treat Redfish `SKU` as a valid fallback for `hardware.board.part_number` when `PartNumber` is empty.
|
||||||
|
- Ignore `Volumes/Capabilities` documents during logical-volume parsing.
|
||||||
|
- Enrich `Chassis/*/PCIeDevices/*` entries with matching `Chassis/*/Devices/*` documents by
|
||||||
|
serial/name/part identity.
|
||||||
|
- Keep `pcie.device_class` semantic; do not replace it with model or part-number strings when
|
||||||
|
Redfish exposes only generic topology labels.
|
||||||
|
|
||||||
|
**Consequences:**
|
||||||
|
- HPE Redfish imports now keep the server SKU in `hardware.board.part_number`.
|
||||||
|
- Empty volume collections no longer produce fake `Capabilities` volume records.
|
||||||
|
- HPE PCIe inventory gets better slot labels like `OCP 3.0 Slot 15` plus concrete classes such as
|
||||||
|
`LOM/NIC` or `SAS/SATA Storage Controller`.
|
||||||
|
- `part_number` remains available separately for model identity, without polluting the class field.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## ADL-041 — Redfish replay drops topology-only PCIe noise classes from canonical inventory
|
||||||
|
|
||||||
|
**Date:** 2026-04-01
|
||||||
|
**Context:** Some Redfish BMCs, especially MSI/AMI GPU systems, expose a very wide PCIe topology
|
||||||
|
tree under `Chassis/*/PCIeDevices/*`. Besides real endpoint devices, the replay sees bridge stages,
|
||||||
|
CPU-side helper functions, IMC/mesh signal-processing nodes, USB/SPI side controllers, and GPU
|
||||||
|
display-function duplicates reported as generic `Display Device`. Keeping all of them in
|
||||||
|
`hardware.pcie_devices` pollutes downstream exports such as Reanimator and hides the actual
|
||||||
|
endpoint inventory signal.
|
||||||
|
|
||||||
|
**Decision:**
|
||||||
|
- Filter topology-only PCIe records during Redfish replay, not in the UI layer.
|
||||||
|
- Drop PCIe entries with replay-resolved classes:
|
||||||
|
- `Bridge`
|
||||||
|
- `Processor`
|
||||||
|
- `SignalProcessingController`
|
||||||
|
- `SerialBusController`
|
||||||
|
- Drop `DisplayController` entries when the source Redfish PCIe document is the generic MSI-style
|
||||||
|
`Description: "Display Device"` duplicate.
|
||||||
|
- Drop PCIe network endpoints when their PCIe functions already link to `NetworkDeviceFunctions`,
|
||||||
|
because those devices are represented canonically in `hardware.network_adapters`.
|
||||||
|
- When `Systems/*/NetworkInterfaces/*` links back to a chassis `NetworkAdapter`, match against the
|
||||||
|
fully enriched chassis NIC identity to avoid creating a second ghost NIC row with the raw
|
||||||
|
`NetworkAdapter_*` slot/name.
|
||||||
|
- Treat generic Redfish object names such as `NetworkAdapter_*` and `PCIeDevice_*` as placeholder
|
||||||
|
models and replace them from PCI IDs when a concrete vendor/device match exists.
|
||||||
|
- Drop MSI-style storage service PCIe endpoints whose resolved device names are only
|
||||||
|
`Volume Management Device NVMe RAID Controller` or `PCIe Switch management endpoint`; storage
|
||||||
|
inventory already comes from the Redfish storage tree.
|
||||||
|
- Normalize Ethernet-class NICs into the single exported class `NetworkController`; do not split
|
||||||
|
`EthernetController` into a separate top-level inventory section.
|
||||||
|
- Keep endpoint classes such as `NetworkController`, `MassStorageController`, and dedicated GPU
|
||||||
|
inventory coming from `hardware.gpus`.
|
||||||
|
|
||||||
|
**Consequences:**
|
||||||
|
- `hardware.pcie_devices` becomes closer to real endpoint inventory instead of raw PCIe topology.
|
||||||
|
- Reanimator exports stop showing MSI bridge/processor/display duplicate noise.
|
||||||
|
- Reanimator exports no longer duplicate the same MSI NIC as both `PCIeDevice_*` and
|
||||||
|
`NetworkAdapter_*`.
|
||||||
|
- Replay no longer creates extra NIC rows from `Systems/NetworkInterfaces` when the same adapter
|
||||||
|
was already normalized from `Chassis/NetworkAdapters`.
|
||||||
|
- MSI VMD / PCIe switch storage service endpoints no longer pollute PCIe inventory.
|
||||||
|
- UI/Reanimator group all Ethernet NICs under the same `NETWORKCONTROLLER` section.
|
||||||
|
- Canonical NIC inventory prefers resolved PCI product names over generic Redfish placeholder names.
|
||||||
|
- The raw Redfish snapshot still remains available in `raw_payloads.redfish_tree` for low-level
|
||||||
|
troubleshooting if topology details are ever needed.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## ADL-042 — xFusion file-export archives merge AppDump inventory with RTOS/Log snapshots
|
||||||
|
|
||||||
|
**Date:** 2026-04-04
|
||||||
|
**Context:** xFusion iBMC `tar.gz` exports expose the base inventory in `AppDump/`, but the most
|
||||||
|
useful NIC and firmware details live elsewhere: NIC firmware/MAC snapshots in
|
||||||
|
`LogDump/netcard/netcard_info.txt` and system firmware versions in
|
||||||
|
`RTOSDump/versioninfo/app_revision.txt`. Parsing only `AppDump/` left xFusion uploads detectable but
|
||||||
|
incomplete for UI and Reanimator consumers.
|
||||||
|
|
||||||
|
**Decision:**
|
||||||
|
- Treat xFusion file-export `tar.gz` bundles as a first-class archive parser input.
|
||||||
|
- Merge OCP NIC identity from `AppDump/card_manage/card_info` with the latest per-slot snapshot
|
||||||
|
from `LogDump/netcard/netcard_info.txt` to produce `hardware.network_adapters`.
|
||||||
|
- Import system-level firmware from `RTOSDump/versioninfo/app_revision.txt` into
|
||||||
|
`hardware.firmware`.
|
||||||
|
- Allow FRU fallback from `RTOSDump/versioninfo/fruinfo.txt` when `AppDump/FruData/fruinfo.txt`
|
||||||
|
is absent.
|
||||||
|
|
||||||
|
**Consequences:**
|
||||||
|
- xFusion uploads now preserve NIC BDF, MAC, firmware, and serial identity in normalized output.
|
||||||
|
- System firmware such as BIOS and iBMC versions survives xFusion file exports.
|
||||||
|
- xFusion archives participate more reliably in canonical device/export flows without special UI
|
||||||
|
cases.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## ADL-043 — Extended HGX diagnostic plan-B is opt-in from the live collect form
|
||||||
|
|
||||||
|
**Date:** 2026-04-13
|
||||||
|
**Context:** Some Supermicro HGX Redfish targets expose slow or hanging component-chassis inventory
|
||||||
|
collections during critical plan-B, especially under `Chassis/HGX_*` for `Assembly`,
|
||||||
|
`Accelerators`, `Drives`, `NetworkAdapters`, and `PCIeDevices`. Default collection should not
|
||||||
|
block operators on deep diagnostic retries that are useful mainly for troubleshooting.
|
||||||
|
**Decision:** Keep the normal snapshot/replay path unchanged, but gate those heavy HGX
|
||||||
|
component-chassis critical plan-B retries behind the existing live-collect `debug_payloads` flag,
|
||||||
|
presented in the UI as "Сбор расширенных данных для диагностики".
|
||||||
|
**Consequences:**
|
||||||
|
- Default live collection skips those heavy diagnostic plan-B retries and reaches replay faster.
|
||||||
|
- Operators can explicitly opt into the slower diagnostic path when they need deeper collection.
|
||||||
|
- The same user-facing toggle continues to enable extra debug payload capture for troubleshooting.
|
||||||
|
|||||||
File diff suppressed because it is too large
Load Diff
57
internal/collector/redfish_planb_test.go
Normal file
57
internal/collector/redfish_planb_test.go
Normal file
@@ -0,0 +1,57 @@
|
|||||||
|
package collector
|
||||||
|
|
||||||
|
import "testing"
|
||||||
|
|
||||||
|
func TestShouldIncludeCriticalPlanBPath(t *testing.T) {
|
||||||
|
tests := []struct {
|
||||||
|
name string
|
||||||
|
req Request
|
||||||
|
path string
|
||||||
|
want bool
|
||||||
|
}{
|
||||||
|
{
|
||||||
|
name: "skip hgx erot pcie without extended diagnostics",
|
||||||
|
req: Request{},
|
||||||
|
path: "/redfish/v1/Chassis/HGX_ERoT_NVSwitch_0/PCIeDevices",
|
||||||
|
want: false,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "skip hgx chassis assembly without extended diagnostics",
|
||||||
|
req: Request{},
|
||||||
|
path: "/redfish/v1/Chassis/HGX_Chassis_0/Assembly",
|
||||||
|
want: false,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "keep standard chassis inventory without extended diagnostics",
|
||||||
|
req: Request{},
|
||||||
|
path: "/redfish/v1/Chassis/1/PCIeDevices",
|
||||||
|
want: true,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "keep nvme storage backplane drives without extended diagnostics",
|
||||||
|
req: Request{},
|
||||||
|
path: "/redfish/v1/Chassis/NVMeSSD.0.Group.0.StorageBackplane/Drives",
|
||||||
|
want: true,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "keep system processors without extended diagnostics",
|
||||||
|
req: Request{},
|
||||||
|
path: "/redfish/v1/Systems/HGX_Baseboard_0/Processors",
|
||||||
|
want: true,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "include hgx erot pcie when extended diagnostics enabled",
|
||||||
|
req: Request{DebugPayloads: true},
|
||||||
|
path: "/redfish/v1/Chassis/HGX_ERoT_NVSwitch_0/PCIeDevices",
|
||||||
|
want: true,
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
for _, tt := range tests {
|
||||||
|
t.Run(tt.name, func(t *testing.T) {
|
||||||
|
if got := shouldIncludeCriticalPlanBPath(tt.req, tt.path); got != tt.want {
|
||||||
|
t.Fatalf("shouldIncludeCriticalPlanBPath(%q) = %v, want %v", tt.path, got, tt.want)
|
||||||
|
}
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -1244,6 +1244,15 @@ func (r redfishSnapshotReader) getLinkedPCIeFunctions(doc map[string]interface{}
|
|||||||
}
|
}
|
||||||
return out
|
return out
|
||||||
}
|
}
|
||||||
|
if ref, ok := links["PCIeFunction"].(map[string]interface{}); ok {
|
||||||
|
memberPath := asString(ref["@odata.id"])
|
||||||
|
if memberPath != "" {
|
||||||
|
memberDoc, err := r.getJSON(memberPath)
|
||||||
|
if err == nil {
|
||||||
|
return []map[string]interface{}{memberDoc}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
if pcieFunctions, ok := doc["PCIeFunctions"].(map[string]interface{}); ok {
|
if pcieFunctions, ok := doc["PCIeFunctions"].(map[string]interface{}); ok {
|
||||||
if collectionPath := asString(pcieFunctions["@odata.id"]); collectionPath != "" {
|
if collectionPath := asString(pcieFunctions["@odata.id"]); collectionPath != "" {
|
||||||
@@ -1256,6 +1265,33 @@ func (r redfishSnapshotReader) getLinkedPCIeFunctions(doc map[string]interface{}
|
|||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func dedupeJSONDocsByPath(docs []map[string]interface{}) []map[string]interface{} {
|
||||||
|
if len(docs) == 0 {
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
seen := make(map[string]struct{}, len(docs))
|
||||||
|
out := make([]map[string]interface{}, 0, len(docs))
|
||||||
|
for _, doc := range docs {
|
||||||
|
if len(doc) == 0 {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
key := normalizeRedfishPath(asString(doc["@odata.id"]))
|
||||||
|
if key == "" {
|
||||||
|
payload, err := json.Marshal(doc)
|
||||||
|
if err != nil {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
key = string(payload)
|
||||||
|
}
|
||||||
|
if _, ok := seen[key]; ok {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
seen[key] = struct{}{}
|
||||||
|
out = append(out, doc)
|
||||||
|
}
|
||||||
|
return out
|
||||||
|
}
|
||||||
|
|
||||||
func (r redfishSnapshotReader) getLinkedSupplementalDocs(doc map[string]interface{}, keys ...string) []map[string]interface{} {
|
func (r redfishSnapshotReader) getLinkedSupplementalDocs(doc map[string]interface{}, keys ...string) []map[string]interface{} {
|
||||||
if len(doc) == 0 || len(keys) == 0 {
|
if len(doc) == 0 || len(keys) == 0 {
|
||||||
return nil
|
return nil
|
||||||
|
|||||||
@@ -31,7 +31,7 @@ func (r redfishSnapshotReader) enrichNICsFromNetworkInterfaces(nics *[]models.Ne
|
|||||||
// the real NIC that came from Chassis/NetworkAdapters (e.g. "RISER 5
|
// the real NIC that came from Chassis/NetworkAdapters (e.g. "RISER 5
|
||||||
// slot 1 (7)"). Try to find the real NIC via the Links.NetworkAdapter
|
// slot 1 (7)"). Try to find the real NIC via the Links.NetworkAdapter
|
||||||
// cross-reference before creating a ghost entry.
|
// cross-reference before creating a ghost entry.
|
||||||
if linkedIdx := r.findNICIndexByLinkedNetworkAdapter(iface, bySlot); linkedIdx >= 0 {
|
if linkedIdx := r.findNICIndexByLinkedNetworkAdapter(iface, *nics, bySlot); linkedIdx >= 0 {
|
||||||
idx = linkedIdx
|
idx = linkedIdx
|
||||||
ok = true
|
ok = true
|
||||||
}
|
}
|
||||||
@@ -75,28 +75,53 @@ func (r redfishSnapshotReader) collectNICs(chassisPaths []string) []models.Netwo
|
|||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
for _, doc := range adapterDocs {
|
for _, doc := range adapterDocs {
|
||||||
nic := parseNIC(doc)
|
nics = append(nics, r.buildNICFromAdapterDoc(doc))
|
||||||
for _, pciePath := range networkAdapterPCIeDevicePaths(doc) {
|
|
||||||
pcieDoc, err := r.getJSON(pciePath)
|
|
||||||
if err != nil {
|
|
||||||
continue
|
|
||||||
}
|
|
||||||
functionDocs := r.getLinkedPCIeFunctions(pcieDoc)
|
|
||||||
supplementalDocs := r.getLinkedSupplementalDocs(pcieDoc, "EnvironmentMetrics", "Metrics")
|
|
||||||
for _, fn := range functionDocs {
|
|
||||||
supplementalDocs = append(supplementalDocs, r.getLinkedSupplementalDocs(fn, "EnvironmentMetrics", "Metrics")...)
|
|
||||||
}
|
|
||||||
enrichNICFromPCIe(&nic, pcieDoc, functionDocs, supplementalDocs)
|
|
||||||
}
|
|
||||||
if len(nic.MACAddresses) == 0 {
|
|
||||||
r.enrichNICMACsFromNetworkDeviceFunctions(&nic, doc)
|
|
||||||
}
|
|
||||||
nics = append(nics, nic)
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return dedupeNetworkAdapters(nics)
|
return dedupeNetworkAdapters(nics)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func (r redfishSnapshotReader) buildNICFromAdapterDoc(adapterDoc map[string]interface{}) models.NetworkAdapter {
|
||||||
|
nic := parseNIC(adapterDoc)
|
||||||
|
adapterFunctionDocs := r.getNetworkAdapterFunctionDocs(adapterDoc)
|
||||||
|
for _, pciePath := range networkAdapterPCIeDevicePaths(adapterDoc) {
|
||||||
|
pcieDoc, err := r.getJSON(pciePath)
|
||||||
|
if err != nil {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
functionDocs := r.getLinkedPCIeFunctions(pcieDoc)
|
||||||
|
for _, adapterFnDoc := range adapterFunctionDocs {
|
||||||
|
functionDocs = append(functionDocs, r.getLinkedPCIeFunctions(adapterFnDoc)...)
|
||||||
|
}
|
||||||
|
functionDocs = dedupeJSONDocsByPath(functionDocs)
|
||||||
|
supplementalDocs := r.getLinkedSupplementalDocs(pcieDoc, "EnvironmentMetrics", "Metrics")
|
||||||
|
for _, fn := range functionDocs {
|
||||||
|
supplementalDocs = append(supplementalDocs, r.getLinkedSupplementalDocs(fn, "EnvironmentMetrics", "Metrics")...)
|
||||||
|
}
|
||||||
|
enrichNICFromPCIe(&nic, pcieDoc, functionDocs, supplementalDocs)
|
||||||
|
}
|
||||||
|
if len(nic.MACAddresses) == 0 {
|
||||||
|
r.enrichNICMACsFromNetworkDeviceFunctions(&nic, adapterDoc)
|
||||||
|
}
|
||||||
|
return nic
|
||||||
|
}
|
||||||
|
|
||||||
|
func (r redfishSnapshotReader) getNetworkAdapterFunctionDocs(adapterDoc map[string]interface{}) []map[string]interface{} {
|
||||||
|
ndfCol, ok := adapterDoc["NetworkDeviceFunctions"].(map[string]interface{})
|
||||||
|
if !ok {
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
colPath := asString(ndfCol["@odata.id"])
|
||||||
|
if colPath == "" {
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
funcDocs, err := r.getCollectionMembers(colPath)
|
||||||
|
if err != nil {
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
return funcDocs
|
||||||
|
}
|
||||||
|
|
||||||
func (r redfishSnapshotReader) collectPCIeDevices(systemPaths, chassisPaths []string) []models.PCIeDevice {
|
func (r redfishSnapshotReader) collectPCIeDevices(systemPaths, chassisPaths []string) []models.PCIeDevice {
|
||||||
collections := make([]string, 0, len(systemPaths)+len(chassisPaths))
|
collections := make([]string, 0, len(systemPaths)+len(chassisPaths))
|
||||||
for _, systemPath := range systemPaths {
|
for _, systemPath := range systemPaths {
|
||||||
@@ -116,13 +141,16 @@ func (r redfishSnapshotReader) collectPCIeDevices(systemPaths, chassisPaths []st
|
|||||||
if looksLikeGPU(doc, functionDocs) {
|
if looksLikeGPU(doc, functionDocs) {
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
|
if replayPCIeDeviceBackedByCanonicalNIC(doc, functionDocs) {
|
||||||
|
continue
|
||||||
|
}
|
||||||
supplementalDocs := r.getLinkedSupplementalDocs(doc, "EnvironmentMetrics", "Metrics")
|
supplementalDocs := r.getLinkedSupplementalDocs(doc, "EnvironmentMetrics", "Metrics")
|
||||||
supplementalDocs = append(supplementalDocs, r.getChassisScopedPCIeSupplementalDocs(doc)...)
|
supplementalDocs = append(supplementalDocs, r.getChassisScopedPCIeSupplementalDocs(doc)...)
|
||||||
for _, fn := range functionDocs {
|
for _, fn := range functionDocs {
|
||||||
supplementalDocs = append(supplementalDocs, r.getLinkedSupplementalDocs(fn, "EnvironmentMetrics", "Metrics")...)
|
supplementalDocs = append(supplementalDocs, r.getLinkedSupplementalDocs(fn, "EnvironmentMetrics", "Metrics")...)
|
||||||
}
|
}
|
||||||
dev := parsePCIeDeviceWithSupplementalDocs(doc, functionDocs, supplementalDocs)
|
dev := parsePCIeDeviceWithSupplementalDocs(doc, functionDocs, supplementalDocs)
|
||||||
if isUnidentifiablePCIeDevice(dev) {
|
if shouldSkipReplayPCIeDevice(doc, dev) {
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
out = append(out, dev)
|
out = append(out, dev)
|
||||||
@@ -136,31 +164,162 @@ func (r redfishSnapshotReader) collectPCIeDevices(systemPaths, chassisPaths []st
|
|||||||
for idx, fn := range functionDocs {
|
for idx, fn := range functionDocs {
|
||||||
supplementalDocs := r.getLinkedSupplementalDocs(fn, "EnvironmentMetrics", "Metrics")
|
supplementalDocs := r.getLinkedSupplementalDocs(fn, "EnvironmentMetrics", "Metrics")
|
||||||
dev := parsePCIeFunctionWithSupplementalDocs(fn, supplementalDocs, idx+1)
|
dev := parsePCIeFunctionWithSupplementalDocs(fn, supplementalDocs, idx+1)
|
||||||
|
if shouldSkipReplayPCIeDevice(fn, dev) {
|
||||||
|
continue
|
||||||
|
}
|
||||||
out = append(out, dev)
|
out = append(out, dev)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return dedupePCIeDevices(out)
|
return dedupePCIeDevices(out)
|
||||||
}
|
}
|
||||||
|
|
||||||
func (r redfishSnapshotReader) getChassisScopedPCIeSupplementalDocs(doc map[string]interface{}) []map[string]interface{} {
|
func shouldSkipReplayPCIeDevice(doc map[string]interface{}, dev models.PCIeDevice) bool {
|
||||||
if !looksLikeNVSwitchPCIeDoc(doc) {
|
if isUnidentifiablePCIeDevice(dev) {
|
||||||
return nil
|
return true
|
||||||
}
|
}
|
||||||
|
if replayNetworkFunctionBackedByCanonicalNIC(doc, dev) {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
if isReplayStorageServiceEndpoint(doc, dev) {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
if isReplayNoisePCIeClass(dev.DeviceClass) {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
if isReplayDisplayDeviceDuplicate(doc, dev) {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
|
||||||
|
func replayPCIeDeviceBackedByCanonicalNIC(doc map[string]interface{}, functionDocs []map[string]interface{}) bool {
|
||||||
|
if !looksLikeReplayNetworkPCIeDevice(doc, functionDocs) {
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
for _, fn := range functionDocs {
|
||||||
|
if hasRedfishLinkedMember(fn, "NetworkDeviceFunctions") {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
|
||||||
|
func replayNetworkFunctionBackedByCanonicalNIC(doc map[string]interface{}, dev models.PCIeDevice) bool {
|
||||||
|
if !looksLikeReplayNetworkClass(dev.DeviceClass) {
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
return hasRedfishLinkedMember(doc, "NetworkDeviceFunctions")
|
||||||
|
}
|
||||||
|
|
||||||
|
func looksLikeReplayNetworkPCIeDevice(doc map[string]interface{}, functionDocs []map[string]interface{}) bool {
|
||||||
|
for _, fn := range functionDocs {
|
||||||
|
if looksLikeReplayNetworkClass(asString(fn["DeviceClass"])) {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
joined := strings.ToLower(strings.TrimSpace(strings.Join([]string{
|
||||||
|
asString(doc["DeviceType"]),
|
||||||
|
asString(doc["Description"]),
|
||||||
|
asString(doc["Name"]),
|
||||||
|
asString(doc["Model"]),
|
||||||
|
}, " ")))
|
||||||
|
return strings.Contains(joined, "network")
|
||||||
|
}
|
||||||
|
|
||||||
|
func looksLikeReplayNetworkClass(class string) bool {
|
||||||
|
class = strings.ToLower(strings.TrimSpace(class))
|
||||||
|
return strings.Contains(class, "network") || strings.Contains(class, "ethernet")
|
||||||
|
}
|
||||||
|
|
||||||
|
func isReplayStorageServiceEndpoint(doc map[string]interface{}, dev models.PCIeDevice) bool {
|
||||||
|
class := strings.ToLower(strings.TrimSpace(dev.DeviceClass))
|
||||||
|
if class != "massstoragecontroller" && class != "mass storage controller" {
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
name := strings.ToLower(strings.TrimSpace(firstNonEmpty(
|
||||||
|
dev.PartNumber,
|
||||||
|
asString(doc["PartNumber"]),
|
||||||
|
asString(doc["Description"]),
|
||||||
|
)))
|
||||||
|
if strings.Contains(name, "pcie switch management endpoint") {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
if strings.Contains(name, "volume management device nvme raid controller") {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
|
||||||
|
func hasRedfishLinkedMember(doc map[string]interface{}, key string) bool {
|
||||||
|
links, ok := doc["Links"].(map[string]interface{})
|
||||||
|
if !ok {
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
if asInt(links[key+"@odata.count"]) > 0 {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
linked, ok := links[key]
|
||||||
|
if !ok {
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
switch v := linked.(type) {
|
||||||
|
case []interface{}:
|
||||||
|
return len(v) > 0
|
||||||
|
case map[string]interface{}:
|
||||||
|
if asString(v["@odata.id"]) != "" {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
return len(v) > 0
|
||||||
|
default:
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func isReplayNoisePCIeClass(class string) bool {
|
||||||
|
switch strings.ToLower(strings.TrimSpace(class)) {
|
||||||
|
case "bridge", "processor", "signalprocessingcontroller", "signal processing controller", "serialbuscontroller", "serial bus controller":
|
||||||
|
return true
|
||||||
|
default:
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func isReplayDisplayDeviceDuplicate(doc map[string]interface{}, dev models.PCIeDevice) bool {
|
||||||
|
class := strings.ToLower(strings.TrimSpace(dev.DeviceClass))
|
||||||
|
if class != "displaycontroller" && class != "display controller" {
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
return strings.EqualFold(strings.TrimSpace(asString(doc["Description"])), "Display Device")
|
||||||
|
}
|
||||||
|
|
||||||
|
func (r redfishSnapshotReader) getChassisScopedPCIeSupplementalDocs(doc map[string]interface{}) []map[string]interface{} {
|
||||||
docPath := normalizeRedfishPath(asString(doc["@odata.id"]))
|
docPath := normalizeRedfishPath(asString(doc["@odata.id"]))
|
||||||
chassisPath := chassisPathForPCIeDoc(docPath)
|
chassisPath := chassisPathForPCIeDoc(docPath)
|
||||||
if chassisPath == "" {
|
if chassisPath == "" {
|
||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
out := make([]map[string]interface{}, 0, 4)
|
|
||||||
for _, path := range []string{
|
out := make([]map[string]interface{}, 0, 6)
|
||||||
joinPath(chassisPath, "/EnvironmentMetrics"),
|
if looksLikeNVSwitchPCIeDoc(doc) {
|
||||||
joinPath(chassisPath, "/ThermalSubsystem/ThermalMetrics"),
|
for _, path := range []string{
|
||||||
} {
|
joinPath(chassisPath, "/EnvironmentMetrics"),
|
||||||
supplementalDoc, err := r.getJSON(path)
|
joinPath(chassisPath, "/ThermalSubsystem/ThermalMetrics"),
|
||||||
if err != nil || len(supplementalDoc) == 0 {
|
} {
|
||||||
continue
|
supplementalDoc, err := r.getJSON(path)
|
||||||
|
if err != nil || len(supplementalDoc) == 0 {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
out = append(out, supplementalDoc)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
deviceDocs, err := r.getCollectionMembers(joinPath(chassisPath, "/Devices"))
|
||||||
|
if err == nil {
|
||||||
|
for _, deviceDoc := range deviceDocs {
|
||||||
|
if !redfishPCIeMatchesChassisDeviceDoc(doc, deviceDoc) {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
out = append(out, deviceDoc)
|
||||||
}
|
}
|
||||||
out = append(out, supplementalDoc)
|
|
||||||
}
|
}
|
||||||
return out
|
return out
|
||||||
}
|
}
|
||||||
@@ -332,8 +491,9 @@ func redfishManagerInterfaceScore(summary map[string]any) int {
|
|||||||
|
|
||||||
// findNICIndexByLinkedNetworkAdapter resolves a NetworkInterface document to an
|
// findNICIndexByLinkedNetworkAdapter resolves a NetworkInterface document to an
|
||||||
// existing NIC in bySlot by following Links.NetworkAdapter → the Chassis
|
// existing NIC in bySlot by following Links.NetworkAdapter → the Chassis
|
||||||
// NetworkAdapter doc → its slot label. Returns -1 if no match is found.
|
// NetworkAdapter doc and reconstructing the canonical NIC identity. Returns -1
|
||||||
func (r redfishSnapshotReader) findNICIndexByLinkedNetworkAdapter(iface map[string]interface{}, bySlot map[string]int) int {
|
// if no match is found.
|
||||||
|
func (r redfishSnapshotReader) findNICIndexByLinkedNetworkAdapter(iface map[string]interface{}, existing []models.NetworkAdapter, bySlot map[string]int) int {
|
||||||
links, ok := iface["Links"].(map[string]interface{})
|
links, ok := iface["Links"].(map[string]interface{})
|
||||||
if !ok {
|
if !ok {
|
||||||
return -1
|
return -1
|
||||||
@@ -350,15 +510,58 @@ func (r redfishSnapshotReader) findNICIndexByLinkedNetworkAdapter(iface map[stri
|
|||||||
if err != nil || len(adapterDoc) == 0 {
|
if err != nil || len(adapterDoc) == 0 {
|
||||||
return -1
|
return -1
|
||||||
}
|
}
|
||||||
adapterNIC := parseNIC(adapterDoc)
|
adapterNIC := r.buildNICFromAdapterDoc(adapterDoc)
|
||||||
|
if serial := normalizeRedfishIdentityField(adapterNIC.SerialNumber); serial != "" {
|
||||||
|
for idx, nic := range existing {
|
||||||
|
if strings.EqualFold(normalizeRedfishIdentityField(nic.SerialNumber), serial) {
|
||||||
|
return idx
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if bdf := strings.TrimSpace(adapterNIC.BDF); bdf != "" {
|
||||||
|
for idx, nic := range existing {
|
||||||
|
if strings.EqualFold(strings.TrimSpace(nic.BDF), bdf) {
|
||||||
|
return idx
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
if slot := strings.ToLower(strings.TrimSpace(adapterNIC.Slot)); slot != "" {
|
if slot := strings.ToLower(strings.TrimSpace(adapterNIC.Slot)); slot != "" {
|
||||||
if idx, ok := bySlot[slot]; ok {
|
if idx, ok := bySlot[slot]; ok {
|
||||||
return idx
|
return idx
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
for idx, nic := range existing {
|
||||||
|
if networkAdaptersShareMACs(nic, adapterNIC) {
|
||||||
|
return idx
|
||||||
|
}
|
||||||
|
}
|
||||||
return -1
|
return -1
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func networkAdaptersShareMACs(a, b models.NetworkAdapter) bool {
|
||||||
|
if len(a.MACAddresses) == 0 || len(b.MACAddresses) == 0 {
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
seen := make(map[string]struct{}, len(a.MACAddresses))
|
||||||
|
for _, mac := range a.MACAddresses {
|
||||||
|
normalized := strings.ToUpper(strings.TrimSpace(mac))
|
||||||
|
if normalized == "" {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
seen[normalized] = struct{}{}
|
||||||
|
}
|
||||||
|
for _, mac := range b.MACAddresses {
|
||||||
|
normalized := strings.ToUpper(strings.TrimSpace(mac))
|
||||||
|
if normalized == "" {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
if _, ok := seen[normalized]; ok {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
|
||||||
// enrichNICMACsFromNetworkDeviceFunctions reads the NetworkDeviceFunctions
|
// enrichNICMACsFromNetworkDeviceFunctions reads the NetworkDeviceFunctions
|
||||||
// collection linked from a NetworkAdapter document and populates the NIC's
|
// collection linked from a NetworkAdapter document and populates the NIC's
|
||||||
// MACAddresses from each function's Ethernet.PermanentMACAddress / MACAddress.
|
// MACAddresses from each function's Ethernet.PermanentMACAddress / MACAddress.
|
||||||
|
|||||||
File diff suppressed because it is too large
Load Diff
@@ -326,6 +326,47 @@ func TestBuildAnalysisDirectives_SupermicroEnablesStorageRecovery(t *testing.T)
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestMatchProfiles_LenovoXCCSelectsMatchedModeAndExcludesSensors(t *testing.T) {
|
||||||
|
match := MatchProfiles(MatchSignals{
|
||||||
|
SystemManufacturer: "Lenovo",
|
||||||
|
ChassisManufacturer: "Lenovo",
|
||||||
|
OEMNamespaces: []string{"Lenovo"},
|
||||||
|
})
|
||||||
|
if match.Mode != ModeMatched {
|
||||||
|
t.Fatalf("expected matched mode, got %q", match.Mode)
|
||||||
|
}
|
||||||
|
found := false
|
||||||
|
for _, profile := range match.Profiles {
|
||||||
|
if profile.Name() == "lenovo" {
|
||||||
|
found = true
|
||||||
|
break
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if !found {
|
||||||
|
t.Fatal("expected lenovo profile to be selected")
|
||||||
|
}
|
||||||
|
|
||||||
|
// Verify the acquisition plan excludes noisy Lenovo-specific snapshot paths.
|
||||||
|
plan := BuildAcquisitionPlan(MatchSignals{
|
||||||
|
SystemManufacturer: "Lenovo",
|
||||||
|
ChassisManufacturer: "Lenovo",
|
||||||
|
OEMNamespaces: []string{"Lenovo"},
|
||||||
|
})
|
||||||
|
wantExcluded := []string{"/Sensors/", "/Oem/Lenovo/LEDs/", "/Oem/Lenovo/Slots/"}
|
||||||
|
for _, want := range wantExcluded {
|
||||||
|
found := false
|
||||||
|
for _, ex := range plan.Tuning.SnapshotExcludeContains {
|
||||||
|
if ex == want {
|
||||||
|
found = true
|
||||||
|
break
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if !found {
|
||||||
|
t.Errorf("expected SnapshotExcludeContains to include %q, got %v", want, plan.Tuning.SnapshotExcludeContains)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func TestMatchProfiles_OrderingIsDeterministic(t *testing.T) {
|
func TestMatchProfiles_OrderingIsDeterministic(t *testing.T) {
|
||||||
signals := MatchSignals{
|
signals := MatchSignals{
|
||||||
SystemManufacturer: "Micro-Star International Co., Ltd.",
|
SystemManufacturer: "Micro-Star International Co., Ltd.",
|
||||||
|
|||||||
65
internal/collector/redfishprofile/profile_lenovo.go
Normal file
65
internal/collector/redfishprofile/profile_lenovo.go
Normal file
@@ -0,0 +1,65 @@
|
|||||||
|
package redfishprofile
|
||||||
|
|
||||||
|
func lenovoProfile() Profile {
|
||||||
|
return staticProfile{
|
||||||
|
name: "lenovo",
|
||||||
|
priority: 20,
|
||||||
|
safeForFallback: true,
|
||||||
|
matchFn: func(s MatchSignals) int {
|
||||||
|
score := 0
|
||||||
|
if containsFold(s.SystemManufacturer, "lenovo") ||
|
||||||
|
containsFold(s.ChassisManufacturer, "lenovo") {
|
||||||
|
score += 80
|
||||||
|
}
|
||||||
|
for _, ns := range s.OEMNamespaces {
|
||||||
|
if containsFold(ns, "lenovo") {
|
||||||
|
score += 30
|
||||||
|
break
|
||||||
|
}
|
||||||
|
}
|
||||||
|
// Lenovo XClarity Controller (XCC) is the BMC product line.
|
||||||
|
if containsFold(s.ServiceRootProduct, "xclarity") ||
|
||||||
|
containsFold(s.ServiceRootProduct, "xcc") {
|
||||||
|
score += 30
|
||||||
|
}
|
||||||
|
return min(score, 100)
|
||||||
|
},
|
||||||
|
extendAcquisition: func(plan *AcquisitionPlan, _ MatchSignals) {
|
||||||
|
// Lenovo XCC BMC exposes Chassis/1/Sensors with hundreds of individual
|
||||||
|
// sensor member documents (e.g. Chassis/1/Sensors/101L1). These are
|
||||||
|
// not used by any LOGPile parser — thermal/power data is read from
|
||||||
|
// the aggregate Chassis/*/Thermal and Chassis/*/Power endpoints. On
|
||||||
|
// a real server they largely return errors, wasting many minutes.
|
||||||
|
// Lenovo OEM subtrees under Oem/Lenovo/LEDs and Oem/Lenovo/Slots also
|
||||||
|
// enumerate dozens of individual documents not relevant to inventory.
|
||||||
|
ensureSnapshotExcludeContains(plan,
|
||||||
|
"/Sensors/", // individual sensor docs (Chassis/1/Sensors/NNN)
|
||||||
|
"/Oem/Lenovo/LEDs/", // individual LED status entries (~47 per server)
|
||||||
|
"/Oem/Lenovo/Slots/", // individual slot detail entries (~26 per server)
|
||||||
|
"/Oem/Lenovo/Metrics/", // operational metrics, not inventory
|
||||||
|
"/Oem/Lenovo/History", // historical telemetry
|
||||||
|
"/Oem/Lenovo/ScheduledPower", // power scheduling config
|
||||||
|
"/Oem/Lenovo/BootSettings/BootOrder", // individual boot order lists
|
||||||
|
"/PortForwardingMap/", // network port forwarding config
|
||||||
|
)
|
||||||
|
// Lenovo XCC BMC is typically slow (p95 latency often 3-5s even under
|
||||||
|
// normal load). Set rate thresholds that don't over-throttle on the
|
||||||
|
// first few requests, and give the ETA estimator a realistic baseline.
|
||||||
|
ensureRatePolicy(plan, AcquisitionRatePolicy{
|
||||||
|
TargetP95LatencyMS: 2000,
|
||||||
|
ThrottleP95LatencyMS: 4000,
|
||||||
|
MinSnapshotWorkers: 2,
|
||||||
|
MinPrefetchWorkers: 1,
|
||||||
|
DisablePrefetchOnErrors: true,
|
||||||
|
})
|
||||||
|
ensureETABaseline(plan, AcquisitionETABaseline{
|
||||||
|
DiscoverySeconds: 15,
|
||||||
|
SnapshotSeconds: 120,
|
||||||
|
PrefetchSeconds: 30,
|
||||||
|
CriticalPlanBSeconds: 40,
|
||||||
|
ProfilePlanBSeconds: 20,
|
||||||
|
})
|
||||||
|
addPlanNote(plan, "lenovo xcc acquisition extensions enabled: noisy sensor/oem paths excluded from snapshot")
|
||||||
|
},
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -56,6 +56,7 @@ func BuiltinProfiles() []Profile {
|
|||||||
supermicroProfile(),
|
supermicroProfile(),
|
||||||
dellProfile(),
|
dellProfile(),
|
||||||
hpeProfile(),
|
hpeProfile(),
|
||||||
|
lenovoProfile(),
|
||||||
inspurGroupOEMPlatformsProfile(),
|
inspurGroupOEMPlatformsProfile(),
|
||||||
hgxProfile(),
|
hgxProfile(),
|
||||||
xfusionProfile(),
|
xfusionProfile(),
|
||||||
@@ -226,6 +227,10 @@ func ensurePrefetchPolicy(plan *AcquisitionPlan, policy AcquisitionPrefetchPolic
|
|||||||
addPlanPaths(&plan.Tuning.PrefetchPolicy.ExcludeContains, policy.ExcludeContains...)
|
addPlanPaths(&plan.Tuning.PrefetchPolicy.ExcludeContains, policy.ExcludeContains...)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func ensureSnapshotExcludeContains(plan *AcquisitionPlan, patterns ...string) {
|
||||||
|
addPlanPaths(&plan.Tuning.SnapshotExcludeContains, patterns...)
|
||||||
|
}
|
||||||
|
|
||||||
func min(a, b int) int {
|
func min(a, b int) int {
|
||||||
if a < b {
|
if a < b {
|
||||||
return a
|
return a
|
||||||
|
|||||||
@@ -53,16 +53,17 @@ type AcquisitionScopedPathPolicy struct {
|
|||||||
}
|
}
|
||||||
|
|
||||||
type AcquisitionTuning struct {
|
type AcquisitionTuning struct {
|
||||||
SnapshotMaxDocuments int
|
SnapshotMaxDocuments int
|
||||||
SnapshotWorkers int
|
SnapshotWorkers int
|
||||||
PrefetchEnabled *bool
|
SnapshotExcludeContains []string
|
||||||
PrefetchWorkers int
|
PrefetchEnabled *bool
|
||||||
NVMePostProbeEnabled *bool
|
PrefetchWorkers int
|
||||||
RatePolicy AcquisitionRatePolicy
|
NVMePostProbeEnabled *bool
|
||||||
ETABaseline AcquisitionETABaseline
|
RatePolicy AcquisitionRatePolicy
|
||||||
PostProbePolicy AcquisitionPostProbePolicy
|
ETABaseline AcquisitionETABaseline
|
||||||
RecoveryPolicy AcquisitionRecoveryPolicy
|
PostProbePolicy AcquisitionPostProbePolicy
|
||||||
PrefetchPolicy AcquisitionPrefetchPolicy
|
RecoveryPolicy AcquisitionRecoveryPolicy
|
||||||
|
PrefetchPolicy AcquisitionPrefetchPolicy
|
||||||
}
|
}
|
||||||
|
|
||||||
type AcquisitionRatePolicy struct {
|
type AcquisitionRatePolicy struct {
|
||||||
|
|||||||
@@ -15,9 +15,8 @@ type Request struct {
|
|||||||
Password string
|
Password string
|
||||||
Token string
|
Token string
|
||||||
TLSMode string
|
TLSMode string
|
||||||
PowerOnIfHostOff bool
|
DebugPayloads bool
|
||||||
StopHostAfterCollect bool
|
SkipHungCh <-chan struct{}
|
||||||
DebugPayloads bool
|
|
||||||
}
|
}
|
||||||
|
|
||||||
type Progress struct {
|
type Progress struct {
|
||||||
@@ -65,10 +64,9 @@ type PhaseTelemetry struct {
|
|||||||
type ProbeResult struct {
|
type ProbeResult struct {
|
||||||
Reachable bool
|
Reachable bool
|
||||||
Protocol string
|
Protocol string
|
||||||
HostPowerState string
|
HostPowerState string
|
||||||
HostPoweredOn bool
|
HostPoweredOn bool
|
||||||
PowerControlAvailable bool
|
SystemPath string
|
||||||
SystemPath string
|
|
||||||
}
|
}
|
||||||
|
|
||||||
type Connector interface {
|
type Connector interface {
|
||||||
|
|||||||
@@ -43,13 +43,13 @@ func ConvertToReanimator(result *models.AnalysisResult) (*ReanimatorExport, erro
|
|||||||
TargetHost: targetHost,
|
TargetHost: targetHost,
|
||||||
CollectedAt: collectedAt,
|
CollectedAt: collectedAt,
|
||||||
Hardware: ReanimatorHardware{
|
Hardware: ReanimatorHardware{
|
||||||
Board: convertBoard(result.Hardware.BoardInfo),
|
Board: convertBoard(result.Hardware.BoardInfo),
|
||||||
Firmware: dedupeFirmware(convertFirmware(result.Hardware.Firmware)),
|
Firmware: dedupeFirmware(convertFirmware(result.Hardware.Firmware)),
|
||||||
CPUs: dedupeCPUs(convertCPUsFromDevices(devices, collectedAt, result.Hardware.BoardInfo.SerialNumber, buildCPUMicrocodeBySocket(result.Hardware.Firmware))),
|
CPUs: dedupeCPUs(convertCPUsFromDevices(devices, collectedAt, result.Hardware.BoardInfo.SerialNumber, buildCPUMicrocodeBySocket(result.Hardware.Firmware))),
|
||||||
Memory: dedupeMemory(convertMemoryFromDevices(devices, collectedAt)),
|
Memory: dedupeMemory(convertMemoryFromDevices(devices, collectedAt)),
|
||||||
Storage: dedupeStorage(convertStorageFromDevices(devices, collectedAt)),
|
Storage: dedupeStorage(convertStorageFromDevices(devices, collectedAt)),
|
||||||
PCIeDevices: dedupePCIe(convertPCIeFromDevices(devices, collectedAt)),
|
PCIeDevices: dedupePCIe(convertPCIeFromDevices(devices, collectedAt)),
|
||||||
PowerSupplies: dedupePSUs(convertPSUsFromDevices(devices, collectedAt)),
|
PowerSupplies: dedupePSUs(convertPSUsFromDevices(devices, collectedAt)),
|
||||||
Sensors: convertSensors(result.Sensors),
|
Sensors: convertSensors(result.Sensors),
|
||||||
EventLogs: convertEventLogs(result.Events, collectedAt),
|
EventLogs: convertEventLogs(result.Events, collectedAt),
|
||||||
},
|
},
|
||||||
@@ -358,10 +358,12 @@ func dedupeCanonicalDevices(items []models.HardwareDevice) []models.HardwareDevi
|
|||||||
prev.score = canonicalScore(prev.item)
|
prev.score = canonicalScore(prev.item)
|
||||||
byKey[key] = prev
|
byKey[key] = prev
|
||||||
}
|
}
|
||||||
// Secondary pass: for items without serial/BDF (noKey), try to merge into an
|
// Secondary pass: for PCIe-class items without serial/BDF (noKey), try to merge
|
||||||
// existing keyed entry with the same model+manufacturer. This handles the case
|
// into an existing keyed entry with the same model+manufacturer. This handles
|
||||||
// where a device appears both in PCIeDevices (with BDF) and NetworkAdapters
|
// the case where a device appears both in PCIeDevices (with BDF) and
|
||||||
// (without BDF) — e.g. Inspur outboardPCIeCard vs PCIeCard with the same model.
|
// NetworkAdapters (without BDF) — e.g. Inspur outboardPCIeCard vs PCIeCard
|
||||||
|
// with the same model. Do not apply this to storage: repeated NVMe slots often
|
||||||
|
// share the same model string and would collapse incorrectly.
|
||||||
// deviceIdentity returns the best available model name for secondary matching,
|
// deviceIdentity returns the best available model name for secondary matching,
|
||||||
// preferring Model over DeviceClass (which may hold a resolved device name).
|
// preferring Model over DeviceClass (which may hold a resolved device name).
|
||||||
deviceIdentity := func(d models.HardwareDevice) string {
|
deviceIdentity := func(d models.HardwareDevice) string {
|
||||||
@@ -377,6 +379,10 @@ func dedupeCanonicalDevices(items []models.HardwareDevice) []models.HardwareDevi
|
|||||||
var unmatched []models.HardwareDevice
|
var unmatched []models.HardwareDevice
|
||||||
for _, item := range noKey {
|
for _, item := range noKey {
|
||||||
mergeKind := canonicalMergeKind(item.Kind)
|
mergeKind := canonicalMergeKind(item.Kind)
|
||||||
|
if mergeKind != "pcie-class" {
|
||||||
|
unmatched = append(unmatched, item)
|
||||||
|
continue
|
||||||
|
}
|
||||||
identity := deviceIdentity(item)
|
identity := deviceIdentity(item)
|
||||||
mfr := strings.ToLower(strings.TrimSpace(item.Manufacturer))
|
mfr := strings.ToLower(strings.TrimSpace(item.Manufacturer))
|
||||||
if identity == "" {
|
if identity == "" {
|
||||||
@@ -669,7 +675,17 @@ func convertMemoryFromDevices(devices []models.HardwareDevice, collectedAt strin
|
|||||||
}
|
}
|
||||||
present := boolFromPresentPtr(d.Present, true)
|
present := boolFromPresentPtr(d.Present, true)
|
||||||
status := normalizeStatus(d.Status, true)
|
status := normalizeStatus(d.Status, true)
|
||||||
if !present || d.SizeMB == 0 || status == "Empty" || strings.TrimSpace(d.SerialNumber) == "" {
|
mem := models.MemoryDIMM{
|
||||||
|
Present: present,
|
||||||
|
SizeMB: d.SizeMB,
|
||||||
|
Type: d.Type,
|
||||||
|
Description: stringFromDetailMap(d.Details, "description"),
|
||||||
|
Manufacturer: d.Manufacturer,
|
||||||
|
SerialNumber: d.SerialNumber,
|
||||||
|
PartNumber: d.PartNumber,
|
||||||
|
Status: d.Status,
|
||||||
|
}
|
||||||
|
if !mem.IsInstalledInventory() || status == "Empty" || strings.TrimSpace(d.SerialNumber) == "" {
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
meta := buildStatusMeta(status, d.StatusCheckedAt, d.StatusChangedAt, d.StatusHistory, d.ErrorDescription, collectedAt)
|
meta := buildStatusMeta(status, d.StatusCheckedAt, d.StatusChangedAt, d.StatusHistory, d.ErrorDescription, collectedAt)
|
||||||
@@ -711,18 +727,16 @@ func convertStorageFromDevices(devices []models.HardwareDevice, collectedAt stri
|
|||||||
if isVirtualExportStorageDevice(d) {
|
if isVirtualExportStorageDevice(d) {
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
if strings.TrimSpace(d.SerialNumber) == "" {
|
if !shouldExportStorageDevice(d) {
|
||||||
continue
|
|
||||||
}
|
|
||||||
present := d.Present == nil || *d.Present
|
|
||||||
if !present {
|
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
|
present := boolFromPresentPtr(d.Present, true)
|
||||||
status := inferStorageStatus(models.Storage{Present: present})
|
status := inferStorageStatus(models.Storage{Present: present})
|
||||||
if strings.TrimSpace(d.Status) != "" {
|
if strings.TrimSpace(d.Status) != "" {
|
||||||
status = normalizeStatus(d.Status, false)
|
status = normalizeStatus(d.Status, !present)
|
||||||
}
|
}
|
||||||
meta := buildStatusMeta(status, d.StatusCheckedAt, d.StatusChangedAt, d.StatusHistory, d.ErrorDescription, collectedAt)
|
meta := buildStatusMeta(status, d.StatusCheckedAt, d.StatusChangedAt, d.StatusHistory, d.ErrorDescription, collectedAt)
|
||||||
|
presentValue := present
|
||||||
result = append(result, ReanimatorStorage{
|
result = append(result, ReanimatorStorage{
|
||||||
Slot: d.Slot,
|
Slot: d.Slot,
|
||||||
Type: d.Type,
|
Type: d.Type,
|
||||||
@@ -732,6 +746,7 @@ func convertStorageFromDevices(devices []models.HardwareDevice, collectedAt stri
|
|||||||
Manufacturer: d.Manufacturer,
|
Manufacturer: d.Manufacturer,
|
||||||
Firmware: d.Firmware,
|
Firmware: d.Firmware,
|
||||||
Interface: d.Interface,
|
Interface: d.Interface,
|
||||||
|
Present: &presentValue,
|
||||||
TemperatureC: floatFromDetailMap(d.Details, "temperature_c"),
|
TemperatureC: floatFromDetailMap(d.Details, "temperature_c"),
|
||||||
PowerOnHours: int64FromDetailMap(d.Details, "power_on_hours"),
|
PowerOnHours: int64FromDetailMap(d.Details, "power_on_hours"),
|
||||||
PowerCycles: int64FromDetailMap(d.Details, "power_cycles"),
|
PowerCycles: int64FromDetailMap(d.Details, "power_cycles"),
|
||||||
@@ -1334,7 +1349,7 @@ func convertMemory(memory []models.MemoryDIMM, collectedAt string) []ReanimatorM
|
|||||||
|
|
||||||
result := make([]ReanimatorMemory, 0, len(memory))
|
result := make([]ReanimatorMemory, 0, len(memory))
|
||||||
for _, mem := range memory {
|
for _, mem := range memory {
|
||||||
if !mem.Present || mem.SizeMB == 0 || normalizeStatus(mem.Status, true) == "Empty" || strings.TrimSpace(mem.SerialNumber) == "" {
|
if !mem.IsInstalledInventory() || normalizeStatus(mem.Status, true) == "Empty" || strings.TrimSpace(mem.SerialNumber) == "" {
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
status := normalizeStatus(mem.Status, true)
|
status := normalizeStatus(mem.Status, true)
|
||||||
@@ -1376,14 +1391,16 @@ func convertStorage(storage []models.Storage, collectedAt string) []ReanimatorSt
|
|||||||
|
|
||||||
result := make([]ReanimatorStorage, 0, len(storage))
|
result := make([]ReanimatorStorage, 0, len(storage))
|
||||||
for _, stor := range storage {
|
for _, stor := range storage {
|
||||||
// Skip storage without serial number
|
if isVirtualLegacyStorageDevice(stor) {
|
||||||
if stor.SerialNumber == "" {
|
continue
|
||||||
|
}
|
||||||
|
if !shouldExportLegacyStorage(stor) {
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
|
|
||||||
status := inferStorageStatus(stor)
|
status := inferStorageStatus(stor)
|
||||||
if strings.TrimSpace(stor.Status) != "" {
|
if strings.TrimSpace(stor.Status) != "" {
|
||||||
status = normalizeStatus(stor.Status, false)
|
status = normalizeStatus(stor.Status, !stor.Present)
|
||||||
}
|
}
|
||||||
meta := buildStatusMeta(
|
meta := buildStatusMeta(
|
||||||
status,
|
status,
|
||||||
@@ -1393,6 +1410,7 @@ func convertStorage(storage []models.Storage, collectedAt string) []ReanimatorSt
|
|||||||
stor.ErrorDescription,
|
stor.ErrorDescription,
|
||||||
collectedAt,
|
collectedAt,
|
||||||
)
|
)
|
||||||
|
present := stor.Present
|
||||||
|
|
||||||
result = append(result, ReanimatorStorage{
|
result = append(result, ReanimatorStorage{
|
||||||
Slot: stor.Slot,
|
Slot: stor.Slot,
|
||||||
@@ -1403,6 +1421,7 @@ func convertStorage(storage []models.Storage, collectedAt string) []ReanimatorSt
|
|||||||
Manufacturer: stor.Manufacturer,
|
Manufacturer: stor.Manufacturer,
|
||||||
Firmware: stor.Firmware,
|
Firmware: stor.Firmware,
|
||||||
Interface: stor.Interface,
|
Interface: stor.Interface,
|
||||||
|
Present: &present,
|
||||||
RemainingEndurancePct: stor.RemainingEndurancePct,
|
RemainingEndurancePct: stor.RemainingEndurancePct,
|
||||||
Status: status,
|
Status: status,
|
||||||
StatusCheckedAt: meta.StatusCheckedAt,
|
StatusCheckedAt: meta.StatusCheckedAt,
|
||||||
@@ -1414,6 +1433,53 @@ func convertStorage(storage []models.Storage, collectedAt string) []ReanimatorSt
|
|||||||
return result
|
return result
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func shouldExportStorageDevice(d models.HardwareDevice) bool {
|
||||||
|
if normalizedSerial(d.SerialNumber) != "" {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
if strings.TrimSpace(d.Slot) != "" {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
if hasMeaningfulExporterText(d.Model) {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
if hasMeaningfulExporterText(d.Type) || hasMeaningfulExporterText(d.Interface) {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
if d.SizeGB > 0 {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
return d.Present != nil
|
||||||
|
}
|
||||||
|
|
||||||
|
func shouldExportLegacyStorage(stor models.Storage) bool {
|
||||||
|
if normalizedSerial(stor.SerialNumber) != "" {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
if strings.TrimSpace(stor.Slot) != "" {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
if hasMeaningfulExporterText(stor.Model) {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
if hasMeaningfulExporterText(stor.Type) || hasMeaningfulExporterText(stor.Interface) {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
if stor.SizeGB > 0 {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
return stor.Present
|
||||||
|
}
|
||||||
|
|
||||||
|
func isVirtualLegacyStorageDevice(stor models.Storage) bool {
|
||||||
|
return isVirtualExportStorageDevice(models.HardwareDevice{
|
||||||
|
Kind: models.DeviceKindStorage,
|
||||||
|
Slot: stor.Slot,
|
||||||
|
Model: stor.Model,
|
||||||
|
Manufacturer: stor.Manufacturer,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
// convertPCIeDevices converts PCIe devices, GPUs, and network adapters to Reanimator format
|
// convertPCIeDevices converts PCIe devices, GPUs, and network adapters to Reanimator format
|
||||||
func convertPCIeDevices(hw *models.HardwareConfig, collectedAt string) []ReanimatorPCIe {
|
func convertPCIeDevices(hw *models.HardwareConfig, collectedAt string) []ReanimatorPCIe {
|
||||||
result := make([]ReanimatorPCIe, 0)
|
result := make([]ReanimatorPCIe, 0)
|
||||||
@@ -1895,7 +1961,10 @@ func pcieDedupKey(item ReanimatorPCIe) string {
|
|||||||
slot := strings.ToLower(strings.TrimSpace(item.Slot))
|
slot := strings.ToLower(strings.TrimSpace(item.Slot))
|
||||||
serial := strings.ToLower(strings.TrimSpace(item.SerialNumber))
|
serial := strings.ToLower(strings.TrimSpace(item.SerialNumber))
|
||||||
bdf := strings.ToLower(strings.TrimSpace(item.BDF))
|
bdf := strings.ToLower(strings.TrimSpace(item.BDF))
|
||||||
if slot != "" {
|
// Generic slot names (e.g. "PCIe Device" from HGX BMC) are not unique
|
||||||
|
// hardware positions — multiple distinct devices share the same name.
|
||||||
|
// Fall through to serial/BDF so they are not incorrectly collapsed.
|
||||||
|
if slot != "" && !isGenericPCIeSlotName(slot) {
|
||||||
return "slot:" + slot
|
return "slot:" + slot
|
||||||
}
|
}
|
||||||
if serial != "" {
|
if serial != "" {
|
||||||
@@ -1904,9 +1973,22 @@ func pcieDedupKey(item ReanimatorPCIe) string {
|
|||||||
if bdf != "" {
|
if bdf != "" {
|
||||||
return "bdf:" + bdf
|
return "bdf:" + bdf
|
||||||
}
|
}
|
||||||
|
if slot != "" {
|
||||||
|
return "slot:" + slot
|
||||||
|
}
|
||||||
return strings.ToLower(strings.TrimSpace(item.DeviceClass)) + "|" + strings.ToLower(strings.TrimSpace(item.Model))
|
return strings.ToLower(strings.TrimSpace(item.DeviceClass)) + "|" + strings.ToLower(strings.TrimSpace(item.Model))
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// isGenericPCIeSlotName reports whether slot is a generic device-type label
|
||||||
|
// rather than a unique hardware position identifier.
|
||||||
|
func isGenericPCIeSlotName(slot string) bool {
|
||||||
|
switch slot {
|
||||||
|
case "pcie device", "pcie slot", "pcie":
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
|
||||||
func pcieQualityScore(item ReanimatorPCIe) int {
|
func pcieQualityScore(item ReanimatorPCIe) int {
|
||||||
score := 0
|
score := 0
|
||||||
if strings.TrimSpace(item.SerialNumber) != "" {
|
if strings.TrimSpace(item.SerialNumber) != "" {
|
||||||
@@ -2180,10 +2262,8 @@ func normalizePCIeDeviceClass(d models.HardwareDevice) string {
|
|||||||
|
|
||||||
func normalizeLegacyPCIeDeviceClass(deviceClass string) string {
|
func normalizeLegacyPCIeDeviceClass(deviceClass string) string {
|
||||||
switch strings.ToLower(strings.TrimSpace(deviceClass)) {
|
switch strings.ToLower(strings.TrimSpace(deviceClass)) {
|
||||||
case "", "network", "network controller", "networkcontroller":
|
case "", "network", "network controller", "networkcontroller", "ethernet", "ethernet controller", "ethernetcontroller":
|
||||||
return "NetworkController"
|
return "NetworkController"
|
||||||
case "ethernet", "ethernet controller", "ethernetcontroller":
|
|
||||||
return "EthernetController"
|
|
||||||
case "fibre channel", "fibre channel controller", "fibrechannelcontroller", "fc":
|
case "fibre channel", "fibre channel controller", "fibrechannelcontroller", "fc":
|
||||||
return "FibreChannelController"
|
return "FibreChannelController"
|
||||||
case "display", "displaycontroller", "display controller", "vga":
|
case "display", "displaycontroller", "display controller", "vga":
|
||||||
@@ -2204,8 +2284,6 @@ func normalizeLegacyPCIeDeviceClass(deviceClass string) string {
|
|||||||
func normalizeNetworkDeviceClass(portType, model, description string) string {
|
func normalizeNetworkDeviceClass(portType, model, description string) string {
|
||||||
joined := strings.ToLower(strings.TrimSpace(strings.Join([]string{portType, model, description}, " ")))
|
joined := strings.ToLower(strings.TrimSpace(strings.Join([]string{portType, model, description}, " ")))
|
||||||
switch {
|
switch {
|
||||||
case strings.Contains(joined, "ethernet"):
|
|
||||||
return "EthernetController"
|
|
||||||
case strings.Contains(joined, "fibre channel") || strings.Contains(joined, " fibrechannel") || strings.Contains(joined, "fc "):
|
case strings.Contains(joined, "fibre channel") || strings.Contains(joined, " fibrechannel") || strings.Contains(joined, "fc "):
|
||||||
return "FibreChannelController"
|
return "FibreChannelController"
|
||||||
default:
|
default:
|
||||||
|
|||||||
@@ -259,6 +259,29 @@ func TestConvertMemory(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestConvertMemory_KeepsInstalledDIMMWithUnknownSize(t *testing.T) {
|
||||||
|
memory := []models.MemoryDIMM{
|
||||||
|
{
|
||||||
|
Slot: "PROC 1 DIMM 3",
|
||||||
|
Present: true,
|
||||||
|
SizeMB: 0,
|
||||||
|
Manufacturer: "Hynix",
|
||||||
|
PartNumber: "HMCG88AEBRA115N",
|
||||||
|
SerialNumber: "2B5F92C6",
|
||||||
|
Status: "OK",
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
result := convertMemory(memory, "2026-03-30T10:00:00Z")
|
||||||
|
|
||||||
|
if len(result) != 1 {
|
||||||
|
t.Fatalf("expected 1 inventory-only DIMM, got %d", len(result))
|
||||||
|
}
|
||||||
|
if result[0].PartNumber != "HMCG88AEBRA115N" || result[0].SerialNumber != "2B5F92C6" || result[0].SizeMB != 0 {
|
||||||
|
t.Fatalf("unexpected converted memory: %+v", result[0])
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func TestConvertToReanimator_CPUSerialIsNotSynthesizedAndSocketIsDeduped(t *testing.T) {
|
func TestConvertToReanimator_CPUSerialIsNotSynthesizedAndSocketIsDeduped(t *testing.T) {
|
||||||
input := &models.AnalysisResult{
|
input := &models.AnalysisResult{
|
||||||
Filename: "cpu-dedupe.json",
|
Filename: "cpu-dedupe.json",
|
||||||
@@ -424,20 +447,26 @@ func TestConvertStorage(t *testing.T) {
|
|||||||
Slot: "OB02",
|
Slot: "OB02",
|
||||||
Type: "NVMe",
|
Type: "NVMe",
|
||||||
Model: "INTEL SSDPF2KX076T1",
|
Model: "INTEL SSDPF2KX076T1",
|
||||||
SerialNumber: "", // No serial - should be skipped
|
SerialNumber: "",
|
||||||
Present: true,
|
Present: true,
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
|
|
||||||
result := convertStorage(storage, "2026-02-10T15:30:00Z")
|
result := convertStorage(storage, "2026-02-10T15:30:00Z")
|
||||||
|
|
||||||
if len(result) != 1 {
|
if len(result) != 2 {
|
||||||
t.Fatalf("expected 1 storage device (skipped one without serial), got %d", len(result))
|
t.Fatalf("expected both inventory slots to be exported, got %d", len(result))
|
||||||
}
|
}
|
||||||
|
|
||||||
if result[0].Status != "Unknown" {
|
if result[0].Status != "Unknown" {
|
||||||
t.Errorf("expected Unknown status, got %q", result[0].Status)
|
t.Errorf("expected Unknown status, got %q", result[0].Status)
|
||||||
}
|
}
|
||||||
|
if result[1].SerialNumber != "" {
|
||||||
|
t.Errorf("expected empty serial for second storage slot, got %q", result[1].SerialNumber)
|
||||||
|
}
|
||||||
|
if result[1].Present == nil || !*result[1].Present {
|
||||||
|
t.Fatalf("expected present=true to be preserved for populated slot without serial")
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
func TestConvertToReanimator_SkipsAMIVirtualStorageDevices(t *testing.T) {
|
func TestConvertToReanimator_SkipsAMIVirtualStorageDevices(t *testing.T) {
|
||||||
@@ -704,6 +733,42 @@ func TestConvertPCIeDevices_SkipsDisplayControllerDuplicates(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestConvertPCIeDevices_PreservesAllGPUsWithGenericSlot(t *testing.T) {
|
||||||
|
// Supermicro HGX BMC reports all GPU PCIe devices with Name "PCIe Device" —
|
||||||
|
// a generic label that is not a unique hardware position. All 8 GPUs must
|
||||||
|
// be preserved; dedup by generic slot name must not collapse them into one.
|
||||||
|
gpus := make([]models.GPU, 8)
|
||||||
|
serials := []string{
|
||||||
|
"1654925165720", "1654925166160", "1654925165942", "1654925165271",
|
||||||
|
"1654925165719", "1654925165252", "1654925165304", "1654925165587",
|
||||||
|
}
|
||||||
|
for i, sn := range serials {
|
||||||
|
gpus[i] = models.GPU{
|
||||||
|
Slot: "PCIe Device",
|
||||||
|
Model: "B200 180GB HBM3e",
|
||||||
|
Manufacturer: "NVIDIA",
|
||||||
|
SerialNumber: sn,
|
||||||
|
PartNumber: "2901-886-A1",
|
||||||
|
Status: "OK",
|
||||||
|
}
|
||||||
|
}
|
||||||
|
hw := &models.HardwareConfig{GPUs: gpus}
|
||||||
|
result := convertPCIeDevices(hw, "2026-04-13T10:00:00Z")
|
||||||
|
if len(result) != 8 {
|
||||||
|
t.Fatalf("expected 8 GPU entries (one per serial), got %d", len(result))
|
||||||
|
}
|
||||||
|
seen := make(map[string]bool)
|
||||||
|
for _, r := range result {
|
||||||
|
if seen[r.SerialNumber] {
|
||||||
|
t.Fatalf("duplicate serial %q in PCIe result", r.SerialNumber)
|
||||||
|
}
|
||||||
|
seen[r.SerialNumber] = true
|
||||||
|
if r.DeviceClass != "VideoController" {
|
||||||
|
t.Fatalf("expected VideoController device class, got %q", r.DeviceClass)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func TestConvertPCIeDevices_MapsGPUStatusHistory(t *testing.T) {
|
func TestConvertPCIeDevices_MapsGPUStatusHistory(t *testing.T) {
|
||||||
hw := &models.HardwareConfig{
|
hw := &models.HardwareConfig{
|
||||||
GPUs: []models.GPU{
|
GPUs: []models.GPU{
|
||||||
@@ -971,6 +1036,52 @@ func TestConvertToReanimator_StatusFallbackUsesCollectedAt(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestConvertToReanimator_ExportsStorageInventoryWithoutSerial(t *testing.T) {
|
||||||
|
collectedAt := time.Date(2026, 4, 1, 9, 0, 0, 0, time.UTC)
|
||||||
|
input := &models.AnalysisResult{
|
||||||
|
Filename: "nvme-inventory.json",
|
||||||
|
CollectedAt: collectedAt,
|
||||||
|
Hardware: &models.HardwareConfig{
|
||||||
|
BoardInfo: models.BoardInfo{SerialNumber: "BOARD-001"},
|
||||||
|
Storage: []models.Storage{
|
||||||
|
{
|
||||||
|
Slot: "OB01",
|
||||||
|
Type: "NVMe",
|
||||||
|
Model: "PM9A3",
|
||||||
|
SerialNumber: "SSD-001",
|
||||||
|
Present: true,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
Slot: "OB02",
|
||||||
|
Type: "NVMe",
|
||||||
|
Model: "PM9A3",
|
||||||
|
Present: true,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
Slot: "OB03",
|
||||||
|
Type: "NVMe",
|
||||||
|
Model: "PM9A3",
|
||||||
|
Present: false,
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
out, err := ConvertToReanimator(input)
|
||||||
|
if err != nil {
|
||||||
|
t.Fatalf("ConvertToReanimator() failed: %v", err)
|
||||||
|
}
|
||||||
|
if len(out.Hardware.Storage) != 3 {
|
||||||
|
t.Fatalf("expected 3 storage entries including inventory slots without serial, got %d", len(out.Hardware.Storage))
|
||||||
|
}
|
||||||
|
if out.Hardware.Storage[1].Slot != "OB02" || out.Hardware.Storage[1].SerialNumber != "" {
|
||||||
|
t.Fatalf("expected OB02 storage slot without serial to survive export, got %#v", out.Hardware.Storage[1])
|
||||||
|
}
|
||||||
|
if out.Hardware.Storage[2].Present == nil || *out.Hardware.Storage[2].Present {
|
||||||
|
t.Fatalf("expected OB03 to preserve present=false, got %#v", out.Hardware.Storage[2])
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func TestConvertToReanimator_FirmwareExcludesDeviceBoundEntries(t *testing.T) {
|
func TestConvertToReanimator_FirmwareExcludesDeviceBoundEntries(t *testing.T) {
|
||||||
input := &models.AnalysisResult{
|
input := &models.AnalysisResult{
|
||||||
Filename: "fw-filter-test.json",
|
Filename: "fw-filter-test.json",
|
||||||
@@ -1658,6 +1769,43 @@ func TestConvertToReanimator_ExportsContractV24Telemetry(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestConvertToReanimator_UnifiesEthernetAndNetworkControllers(t *testing.T) {
|
||||||
|
input := &models.AnalysisResult{
|
||||||
|
Hardware: &models.HardwareConfig{
|
||||||
|
BoardInfo: models.BoardInfo{SerialNumber: "BOARD-123"},
|
||||||
|
Devices: []models.HardwareDevice{
|
||||||
|
{
|
||||||
|
Kind: models.DeviceKindPCIe,
|
||||||
|
Slot: "PCIe1",
|
||||||
|
DeviceClass: "EthernetController",
|
||||||
|
Present: boolPtr(true),
|
||||||
|
SerialNumber: "ETH-001",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
Kind: models.DeviceKindNetwork,
|
||||||
|
Slot: "NIC1",
|
||||||
|
Model: "Ethernet Adapter",
|
||||||
|
Present: boolPtr(true),
|
||||||
|
SerialNumber: "NIC-001",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
out, err := ConvertToReanimator(input)
|
||||||
|
if err != nil {
|
||||||
|
t.Fatalf("ConvertToReanimator() failed: %v", err)
|
||||||
|
}
|
||||||
|
if len(out.Hardware.PCIeDevices) != 2 {
|
||||||
|
t.Fatalf("expected two pcie-class exports, got %d", len(out.Hardware.PCIeDevices))
|
||||||
|
}
|
||||||
|
for _, dev := range out.Hardware.PCIeDevices {
|
||||||
|
if dev.DeviceClass != "NetworkController" {
|
||||||
|
t.Fatalf("expected unified NetworkController class, got %+v", dev)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func TestConvertToReanimator_PreservesLegacyStorageAndPSUDetails(t *testing.T) {
|
func TestConvertToReanimator_PreservesLegacyStorageAndPSUDetails(t *testing.T) {
|
||||||
input := &models.AnalysisResult{
|
input := &models.AnalysisResult{
|
||||||
Filename: "legacy-details.json",
|
Filename: "legacy-details.json",
|
||||||
|
|||||||
29
internal/models/memory.go
Normal file
29
internal/models/memory.go
Normal file
@@ -0,0 +1,29 @@
|
|||||||
|
package models
|
||||||
|
|
||||||
|
import "strings"
|
||||||
|
|
||||||
|
// HasInventoryIdentity reports whether the DIMM has enough identifying
|
||||||
|
// inventory data to treat it as a populated module even when size is unknown.
|
||||||
|
func (m MemoryDIMM) HasInventoryIdentity() bool {
|
||||||
|
return strings.TrimSpace(m.SerialNumber) != "" ||
|
||||||
|
strings.TrimSpace(m.PartNumber) != "" ||
|
||||||
|
strings.TrimSpace(m.Type) != "" ||
|
||||||
|
strings.TrimSpace(m.Technology) != "" ||
|
||||||
|
strings.TrimSpace(m.Description) != ""
|
||||||
|
}
|
||||||
|
|
||||||
|
// IsInstalledInventory reports whether the DIMM represents an installed module
|
||||||
|
// that should be kept in canonical inventory and exports.
|
||||||
|
func (m MemoryDIMM) IsInstalledInventory() bool {
|
||||||
|
if !m.Present {
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
|
||||||
|
status := strings.ToLower(strings.TrimSpace(m.Status))
|
||||||
|
switch status {
|
||||||
|
case "empty", "absent", "not installed":
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
|
||||||
|
return m.SizeMB > 0 || m.HasInventoryIdentity()
|
||||||
|
}
|
||||||
455
internal/parser/vendors/hpe_ilo_ahs/parser.go
vendored
455
internal/parser/vendors/hpe_ilo_ahs/parser.go
vendored
@@ -25,12 +25,17 @@ const (
|
|||||||
)
|
)
|
||||||
|
|
||||||
var (
|
var (
|
||||||
partNumberPattern = regexp.MustCompile(`(?i)^[a-z0-9]{1,4}\d{4,6}-[a-z0-9]{2,4}$`)
|
partNumberPattern = regexp.MustCompile(`(?i)^[a-z0-9]{1,4}\d{4,6}-[a-z0-9]{2,4}$`)
|
||||||
serverSerialRE = regexp.MustCompile(`(?i)(?:^|[_-])([a-z0-9]{10})(?:[_-]|\.)`)
|
serverSerialRE = regexp.MustCompile(`(?i)(?:^|[_-])([a-z0-9]{10})(?:[_-]|\.)`)
|
||||||
dimmSlotRE = regexp.MustCompile(`^PROC\s+(\d+)\s+DIMM\s+(\d+)$`)
|
dimmSlotRE = regexp.MustCompile(`^PROC\s+(\d+)\s+DIMM\s+(\d+)$`)
|
||||||
procSlotRE = regexp.MustCompile(`^Proc\s+(\d+)$`)
|
procSlotRE = regexp.MustCompile(`^Proc\s+(\d+)$`)
|
||||||
psuSlotRE = regexp.MustCompile(`^Power Supply\s+(\d+)$`)
|
psuSlotRE = regexp.MustCompile(`^Power Supply\s+(\d+)$`)
|
||||||
eventTimeRE = regexp.MustCompile(`^\d{2}/\d{2}/\d{4} \d{2}:\d{2}:\d{2}$`)
|
eventTimeRE = regexp.MustCompile(`^\d{2}/\d{2}/\d{4} \d{2}:\d{2}:\d{2}$`)
|
||||||
|
psuXMLRE = regexp.MustCompile(`(?s)<PowerSupplySlot id="(\d+)">(.*?)</PowerSupplySlot>`)
|
||||||
|
firmwareLockdownRE = regexp.MustCompile(`(?s)<FirmwareLockdown>(.*?)</FirmwareLockdown>`)
|
||||||
|
xmlFieldRE = regexp.MustCompile(`(?s)<([A-Za-z0-9_-]+)>([^<]*)</[A-Za-z0-9_-]+>`)
|
||||||
|
psuLogRE = regexp.MustCompile(`Update bay (\d+) (SPN|Serial Number|Model Number|fw ver\.), value = ([A-Za-z0-9._-]+)`)
|
||||||
|
versionFragmentRE = regexp.MustCompile(`\d+(?:\.\d+)+`)
|
||||||
)
|
)
|
||||||
|
|
||||||
func init() {
|
func init() {
|
||||||
@@ -129,6 +134,13 @@ func (p *Parser) Parse(files []parser.ExtractedFile) (*models.AnalysisResult, er
|
|||||||
result.Hardware.NetworkAdapters = dedupeNetworkAdapters(parseNetworkAdapters(tokens))
|
result.Hardware.NetworkAdapters = dedupeNetworkAdapters(parseNetworkAdapters(tokens))
|
||||||
result.Hardware.Firmware = dedupeFirmware(parseFirmware(tokens))
|
result.Hardware.Firmware = dedupeFirmware(parseFirmware(tokens))
|
||||||
|
|
||||||
|
psuSupplements := parsePSUSupplements(entries)
|
||||||
|
result.Hardware.PowerSupply = dedupePSUs(mergePSUs(result.Hardware.PowerSupply, psuSupplements))
|
||||||
|
|
||||||
|
lockdownFW, nicFirmwareByVendor := parseBCertFirmware(entries)
|
||||||
|
result.Hardware.NetworkAdapters = dedupeNetworkAdapters(enrichNetworkAdapters(result.Hardware.NetworkAdapters, nicFirmwareByVendor))
|
||||||
|
result.Hardware.Firmware = dedupeFirmware(append(result.Hardware.Firmware, lockdownFW...))
|
||||||
|
|
||||||
storage, volumes, controllerDevices, controllerFW := parseRedfishStorage(redfishDocs)
|
storage, volumes, controllerDevices, controllerFW := parseRedfishStorage(redfishDocs)
|
||||||
result.Hardware.Storage = dedupeStorage(storage)
|
result.Hardware.Storage = dedupeStorage(storage)
|
||||||
result.Hardware.Volumes = volumes
|
result.Hardware.Volumes = volumes
|
||||||
@@ -446,22 +458,37 @@ func parseDIMMs(tokens []string) []models.MemoryDIMM {
|
|||||||
|
|
||||||
func parsePSUs(tokens []string) []models.PSU {
|
func parsePSUs(tokens []string) []models.PSU {
|
||||||
out := make([]models.PSU, 0, 4)
|
out := make([]models.PSU, 0, 4)
|
||||||
for i := 0; i+2 < len(tokens); i++ {
|
for i := 0; i < len(tokens); i++ {
|
||||||
match := psuSlotRE.FindStringSubmatch(tokens[i])
|
match := psuSlotRE.FindStringSubmatch(tokens[i])
|
||||||
if len(match) != 2 {
|
if len(match) != 2 {
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
slot := "PSU " + match[1]
|
slot := "PSU " + match[1]
|
||||||
serial := tokens[i+1]
|
vendor := ""
|
||||||
partNumber := tokens[i+2]
|
serial := ""
|
||||||
if isUnavailable(serial) && isUnavailable(partNumber) {
|
partNumber := ""
|
||||||
|
for j := i + 1; j < len(tokens) && j <= i+5; j++ {
|
||||||
|
field := strings.TrimSpace(tokens[j])
|
||||||
|
if strings.HasPrefix(field, "PciRoot(") || psuSlotRE.MatchString(field) || dimmSlotRE.MatchString(field) || procSlotRE.MatchString(field) || eventTimeRE.MatchString(field) {
|
||||||
|
break
|
||||||
|
}
|
||||||
|
switch {
|
||||||
|
case vendor == "" && looksLikePSUVendor(field):
|
||||||
|
vendor = field
|
||||||
|
case partNumber == "" && looksLikePartNumber(field):
|
||||||
|
partNumber = field
|
||||||
|
case serial == "" && isLikelySerial(field):
|
||||||
|
serial = field
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if serial == "" && partNumber == "" {
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
psu := models.PSU{
|
psu := models.PSU{
|
||||||
Slot: slot,
|
Slot: slot,
|
||||||
Present: true,
|
Present: true,
|
||||||
Model: valueOr(partNumber, "Power Supply"),
|
Model: valueOr(partNumber, "Power Supply"),
|
||||||
Vendor: "HPE",
|
Vendor: valueOr(cleanUnavailable(vendor), "HPE"),
|
||||||
SerialNumber: cleanUnavailable(serial),
|
SerialNumber: cleanUnavailable(serial),
|
||||||
PartNumber: cleanUnavailable(partNumber),
|
PartNumber: cleanUnavailable(partNumber),
|
||||||
Status: "ok",
|
Status: "ok",
|
||||||
@@ -471,6 +498,80 @@ func parsePSUs(tokens []string) []models.PSU {
|
|||||||
return out
|
return out
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func parsePSUSupplements(entries []ahsEntry) []models.PSU {
|
||||||
|
bySlot := make(map[string]models.PSU)
|
||||||
|
|
||||||
|
for _, entry := range entries {
|
||||||
|
text := string(entry.Content)
|
||||||
|
if text == "" {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
|
||||||
|
if strings.EqualFold(entry.Name, "bcert.pkg") {
|
||||||
|
for _, match := range psuXMLRE.FindAllStringSubmatch(text, -1) {
|
||||||
|
slotNum, _ := strconv.Atoi(match[1])
|
||||||
|
slot := fmt.Sprintf("PSU %d", slotNum+1)
|
||||||
|
fields := parseXMLFields(match[2])
|
||||||
|
item := bySlot[slot]
|
||||||
|
item.Slot = slot
|
||||||
|
item.Present = strings.EqualFold(fields["Present"], "Yes") || item.Present
|
||||||
|
if serial := strings.TrimSpace(fields["SerialNumber"]); serial != "" {
|
||||||
|
item.SerialNumber = serial
|
||||||
|
}
|
||||||
|
if fw := strings.TrimSpace(fields["FirmwareVersion"]); fw != "" {
|
||||||
|
item.Firmware = fw
|
||||||
|
}
|
||||||
|
if spare := strings.TrimSpace(fields["SparePartNumber"]); spare != "" {
|
||||||
|
if item.Details == nil {
|
||||||
|
item.Details = make(map[string]any)
|
||||||
|
}
|
||||||
|
item.Details["spare_part_number"] = spare
|
||||||
|
}
|
||||||
|
bySlot[slot] = item
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
for _, match := range psuLogRE.FindAllStringSubmatch(text, -1) {
|
||||||
|
slotNum, _ := strconv.Atoi(match[1])
|
||||||
|
slot := fmt.Sprintf("PSU %d", slotNum+1)
|
||||||
|
item := bySlot[slot]
|
||||||
|
item.Slot = slot
|
||||||
|
item.Present = true
|
||||||
|
value := strings.TrimSpace(match[3])
|
||||||
|
switch match[2] {
|
||||||
|
case "SPN":
|
||||||
|
if item.Details == nil {
|
||||||
|
item.Details = make(map[string]any)
|
||||||
|
}
|
||||||
|
item.Details["spare_part_number"] = value
|
||||||
|
case "Serial Number":
|
||||||
|
item.SerialNumber = value
|
||||||
|
case "Model Number":
|
||||||
|
item.Model = value
|
||||||
|
item.PartNumber = value
|
||||||
|
case "fw ver.":
|
||||||
|
item.Firmware = normalizeLooseVersion(value)
|
||||||
|
}
|
||||||
|
bySlot[slot] = item
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
out := make([]models.PSU, 0, len(bySlot))
|
||||||
|
for _, item := range bySlot {
|
||||||
|
if item.Slot == "" {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
item.Vendor = valueOr(item.Vendor, "HPE")
|
||||||
|
item.Status = valueOr(item.Status, "ok")
|
||||||
|
if item.Model == "" {
|
||||||
|
item.Model = valueOr(item.PartNumber, "Power Supply")
|
||||||
|
}
|
||||||
|
out = append(out, item)
|
||||||
|
}
|
||||||
|
sort.Slice(out, func(i, j int) bool { return out[i].Slot < out[j].Slot })
|
||||||
|
return out
|
||||||
|
}
|
||||||
|
|
||||||
type pcieSequence struct {
|
type pcieSequence struct {
|
||||||
UEFIPath string
|
UEFIPath string
|
||||||
Code string
|
Code string
|
||||||
@@ -621,13 +722,53 @@ func parseRedfishStorage(docs map[string]map[string]any) ([]models.Storage, []mo
|
|||||||
|
|
||||||
storage := make([]models.Storage, 0, 8)
|
storage := make([]models.Storage, 0, 8)
|
||||||
volumes := make([]models.StorageVolume, 0, 4)
|
volumes := make([]models.StorageVolume, 0, 4)
|
||||||
devices := make([]models.HardwareDevice, 0, 4)
|
devices := make([]models.HardwareDevice, 0, 6)
|
||||||
firmware := make([]models.FirmwareInfo, 0, 4)
|
firmware := make([]models.FirmwareInfo, 0, 8)
|
||||||
|
fabricNames := make(map[string]string)
|
||||||
|
fabricTypes := make(map[string]string)
|
||||||
|
|
||||||
for _, path := range paths {
|
for _, path := range paths {
|
||||||
doc := docs[path]
|
doc := docs[path]
|
||||||
docType := asString(doc["@odata.type"])
|
docType := asString(doc["@odata.type"])
|
||||||
switch {
|
switch {
|
||||||
|
case strings.Contains(docType, "#Fabric."):
|
||||||
|
fabricID := redfishID(path)
|
||||||
|
fabricNames[fabricID] = strings.TrimSpace(asString(doc["Name"]))
|
||||||
|
fabricTypes[fabricID] = strings.TrimSpace(asString(doc["FabricType"]))
|
||||||
|
|
||||||
|
case strings.Contains(docType, "#Switch."):
|
||||||
|
fabricID := fabricIDFromPath(path)
|
||||||
|
name := valueOr(fabricNames[fabricID], strings.TrimSpace(asString(doc["Name"])))
|
||||||
|
model := strings.TrimSpace(asString(doc["Model"]))
|
||||||
|
fw := strings.TrimSpace(asString(doc["FirmwareVersion"]))
|
||||||
|
device := models.HardwareDevice{
|
||||||
|
ID: "hpe-fabric-" + redfishID(path),
|
||||||
|
Kind: models.DeviceKindStorage,
|
||||||
|
Source: "redfish",
|
||||||
|
Slot: valueOr(fabricID, redfishID(path)),
|
||||||
|
DeviceClass: "storage_backplane",
|
||||||
|
Model: valueOr(name, model),
|
||||||
|
PartNumber: model,
|
||||||
|
Firmware: fw,
|
||||||
|
Status: redfishStatus(doc["Status"]),
|
||||||
|
Details: map[string]any{
|
||||||
|
"odata_id": path,
|
||||||
|
"fabric_type": valueOr(fabricTypes[fabricID], strings.TrimSpace(asString(doc["FabricType"]))),
|
||||||
|
"switch_type": strings.TrimSpace(asString(doc["SwitchType"])),
|
||||||
|
"supported_protocols": stringSlice(doc["SupportedProtocols"]),
|
||||||
|
"domain_id": asInt64(doc["DomainID"]),
|
||||||
|
"fabric_name": fabricNames[fabricID],
|
||||||
|
"connected_chassis_id": asString(nested(doc, "Links", "Chassis", "@odata.id")),
|
||||||
|
},
|
||||||
|
}
|
||||||
|
devices = append(devices, device)
|
||||||
|
if fw != "" {
|
||||||
|
firmware = append(firmware, models.FirmwareInfo{
|
||||||
|
DeviceName: valueOr(name, model),
|
||||||
|
Version: fw,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
case strings.Contains(docType, "#StorageController."):
|
case strings.Contains(docType, "#StorageController."):
|
||||||
slot := redfishServiceLabel(doc, "Location", "PartLocation", "ServiceLabel")
|
slot := redfishServiceLabel(doc, "Location", "PartLocation", "ServiceLabel")
|
||||||
model := valueOr(asString(doc["Model"]), asString(doc["Name"]))
|
model := valueOr(asString(doc["Model"]), asString(doc["Name"]))
|
||||||
@@ -649,9 +790,16 @@ func parseRedfishStorage(docs map[string]map[string]any) ([]models.Storage, []mo
|
|||||||
Firmware: fw,
|
Firmware: fw,
|
||||||
Status: redfishStatus(doc["Status"]),
|
Status: redfishStatus(doc["Status"]),
|
||||||
Details: map[string]any{
|
Details: map[string]any{
|
||||||
"odata_id": path,
|
"odata_id": path,
|
||||||
"part_number": partNumber,
|
"part_number": partNumber,
|
||||||
"sku": sku,
|
"sku": sku,
|
||||||
|
"speed_gbps": asFloat64(doc["SpeedGbps"]),
|
||||||
|
"supported_controller_protocols": stringSlice(doc["SupportedControllerProtocols"]),
|
||||||
|
"supported_device_protocols": stringSlice(doc["SupportedDeviceProtocols"]),
|
||||||
|
"supported_raid_types": stringSlice(doc["SupportedRAIDTypes"]),
|
||||||
|
"cache_total_mib": asInt64(nested(doc, "CacheSummary", "TotalCacheSizeMiB")),
|
||||||
|
"persistent_cache_mib": asInt64(nested(doc, "CacheSummary", "PersistentCacheSizeMiB")),
|
||||||
|
"durable_name": firstDurableName(doc),
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
if width := asInt(doc, "PCIeInterface", "LanesInUse"); width > 0 {
|
if width := asInt(doc, "PCIeInterface", "LanesInUse"); width > 0 {
|
||||||
@@ -692,8 +840,12 @@ func parseRedfishStorage(docs map[string]map[string]any) ([]models.Storage, []mo
|
|||||||
RemainingEndurancePct: endurance,
|
RemainingEndurancePct: endurance,
|
||||||
Status: redfishStatus(doc["Status"]),
|
Status: redfishStatus(doc["Status"]),
|
||||||
Details: map[string]any{
|
Details: map[string]any{
|
||||||
"odata_id": path,
|
"odata_id": path,
|
||||||
"capacity_bytes": capacity,
|
"capacity_bytes": capacity,
|
||||||
|
"failure_predicted": asBool(doc["FailurePredicted"]),
|
||||||
|
"negotiated_speed_gbps": asFloat64(doc["NegotiatedSpeedGbs"]),
|
||||||
|
"capable_speed_gbps": asFloat64(doc["CapableSpeedGbs"]),
|
||||||
|
"location_indicator_active": asBool(doc["LocationIndicatorActive"]),
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
storage = append(storage, entry)
|
storage = append(storage, entry)
|
||||||
@@ -1005,6 +1157,16 @@ func isHPEManufacturer(v string) bool {
|
|||||||
return v == "HPE" || v == "HP"
|
return v == "HPE" || v == "HP"
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func looksLikePSUVendor(v string) bool {
|
||||||
|
v = strings.TrimSpace(strings.ToUpper(v))
|
||||||
|
switch v {
|
||||||
|
case "HPE", "HP", "DELTA", "LITEON", "LTEON":
|
||||||
|
return true
|
||||||
|
default:
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func looksLikeServerModel(v string) bool {
|
func looksLikeServerModel(v string) bool {
|
||||||
v = sanitizeModel(v)
|
v = sanitizeModel(v)
|
||||||
if v == "" {
|
if v == "" {
|
||||||
@@ -1115,6 +1277,163 @@ func inferVendor(model string) string {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func mergePSUs(base, extra []models.PSU) []models.PSU {
|
||||||
|
merged := make(map[string]models.PSU)
|
||||||
|
order := make([]string, 0, len(base)+len(extra))
|
||||||
|
mergeOne := func(item models.PSU) {
|
||||||
|
key := strings.ToLower(strings.TrimSpace(item.Slot))
|
||||||
|
if key == "" {
|
||||||
|
key = strings.ToLower(strings.TrimSpace(valueOr(item.SerialNumber, item.Model+"|"+item.PartNumber)))
|
||||||
|
}
|
||||||
|
if key == "" {
|
||||||
|
return
|
||||||
|
}
|
||||||
|
current, exists := merged[key]
|
||||||
|
if !exists {
|
||||||
|
merged[key] = item
|
||||||
|
order = append(order, key)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if current.Slot == "" {
|
||||||
|
current.Slot = item.Slot
|
||||||
|
}
|
||||||
|
current.Present = current.Present || item.Present
|
||||||
|
current.Model = valueOr(current.Model, item.Model)
|
||||||
|
current.Description = valueOr(current.Description, item.Description)
|
||||||
|
current.Vendor = valueOr(current.Vendor, item.Vendor)
|
||||||
|
if current.WattageW == 0 {
|
||||||
|
current.WattageW = item.WattageW
|
||||||
|
}
|
||||||
|
current.SerialNumber = valueOr(current.SerialNumber, item.SerialNumber)
|
||||||
|
current.PartNumber = valueOr(current.PartNumber, item.PartNumber)
|
||||||
|
current.Firmware = valueOr(current.Firmware, item.Firmware)
|
||||||
|
current.Status = valueOr(current.Status, item.Status)
|
||||||
|
current.InputType = valueOr(current.InputType, item.InputType)
|
||||||
|
if current.InputPowerW == 0 {
|
||||||
|
current.InputPowerW = item.InputPowerW
|
||||||
|
}
|
||||||
|
if current.OutputPowerW == 0 {
|
||||||
|
current.OutputPowerW = item.OutputPowerW
|
||||||
|
}
|
||||||
|
if current.InputVoltage == 0 {
|
||||||
|
current.InputVoltage = item.InputVoltage
|
||||||
|
}
|
||||||
|
if current.OutputVoltage == 0 {
|
||||||
|
current.OutputVoltage = item.OutputVoltage
|
||||||
|
}
|
||||||
|
if current.TemperatureC == 0 {
|
||||||
|
current.TemperatureC = item.TemperatureC
|
||||||
|
}
|
||||||
|
current.Details = mergeDetailMaps(current.Details, item.Details)
|
||||||
|
merged[key] = current
|
||||||
|
}
|
||||||
|
for _, item := range base {
|
||||||
|
mergeOne(item)
|
||||||
|
}
|
||||||
|
for _, item := range extra {
|
||||||
|
mergeOne(item)
|
||||||
|
}
|
||||||
|
out := make([]models.PSU, 0, len(order))
|
||||||
|
for _, key := range order {
|
||||||
|
out = append(out, merged[key])
|
||||||
|
}
|
||||||
|
return out
|
||||||
|
}
|
||||||
|
|
||||||
|
func enrichNetworkAdapters(items []models.NetworkAdapter, firmwareByVendor map[string]string) []models.NetworkAdapter {
|
||||||
|
out := make([]models.NetworkAdapter, 0, len(items))
|
||||||
|
for _, item := range items {
|
||||||
|
if item.Firmware == "" {
|
||||||
|
if fw := firmwareByVendor[strings.ToLower(strings.TrimSpace(item.Vendor))]; fw != "" {
|
||||||
|
item.Firmware = fw
|
||||||
|
}
|
||||||
|
}
|
||||||
|
out = append(out, item)
|
||||||
|
}
|
||||||
|
return out
|
||||||
|
}
|
||||||
|
|
||||||
|
func parseBCertFirmware(entries []ahsEntry) ([]models.FirmwareInfo, map[string]string) {
|
||||||
|
out := make([]models.FirmwareInfo, 0, 8)
|
||||||
|
nicFirmwareByVendor := make(map[string]string)
|
||||||
|
seen := make(map[string]bool)
|
||||||
|
|
||||||
|
tagNames := map[string]string{
|
||||||
|
"SystemProgrammableLogicDevice": "System Programmable Logic Device",
|
||||||
|
"ServerPlatformServicesSPSFirmware": "Server Platform Services (SPS) Firmware",
|
||||||
|
"STMicroGen11TPM": "TPM Firmware",
|
||||||
|
"PrimaryR012U3x16slotsriserx8-x16-x8": "PCIe Riser 1 Programmable Logic Device",
|
||||||
|
"HPEMR408i-oGen11": "HPE MR408i-o Gen11",
|
||||||
|
"UBM3": "8 SFF 24G x1NVMe/SAS UBM3 BC BP",
|
||||||
|
"BCM57191Gb4pBASE-T": "BCM 5719 1Gb 4p BASE-T OCP Adptr",
|
||||||
|
"BCM57191Gb4pBASE-TOCP3": "BCM 5719 1Gb 4p BASE-T OCP Adptr",
|
||||||
|
}
|
||||||
|
|
||||||
|
for _, entry := range entries {
|
||||||
|
if !strings.EqualFold(entry.Name, "bcert.pkg") {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
text := string(entry.Content)
|
||||||
|
for _, match := range firmwareLockdownRE.FindAllStringSubmatch(text, -1) {
|
||||||
|
fields := parseXMLFields(match[1])
|
||||||
|
for tag, value := range fields {
|
||||||
|
name := tagNames[tag]
|
||||||
|
if name == "" {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
version := normalizeBCertVersion(tag, value)
|
||||||
|
if version == "" {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
appendFirmware(&out, seen, models.FirmwareInfo{
|
||||||
|
DeviceName: name,
|
||||||
|
Version: version,
|
||||||
|
})
|
||||||
|
if strings.Contains(name, "BCM 5719") {
|
||||||
|
nicFirmwareByVendor["broadcom"] = version
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return out, nicFirmwareByVendor
|
||||||
|
}
|
||||||
|
|
||||||
|
func parseXMLFields(block string) map[string]string {
|
||||||
|
out := make(map[string]string)
|
||||||
|
for _, match := range xmlFieldRE.FindAllStringSubmatch(block, -1) {
|
||||||
|
out[match[1]] = strings.TrimSpace(match[2])
|
||||||
|
}
|
||||||
|
return out
|
||||||
|
}
|
||||||
|
|
||||||
|
func normalizeBCertVersion(tag, value string) string {
|
||||||
|
value = strings.TrimSpace(value)
|
||||||
|
if value == "" || strings.EqualFold(value, "NA") {
|
||||||
|
return ""
|
||||||
|
}
|
||||||
|
switch tag {
|
||||||
|
case "UBM3":
|
||||||
|
if idx := strings.LastIndex(value, "/"); idx >= 0 && idx+1 < len(value) {
|
||||||
|
return strings.TrimSpace(value[idx+1:])
|
||||||
|
}
|
||||||
|
case "IntegratedLights-OutVI":
|
||||||
|
if idx := strings.Index(value, " - "); idx > 0 {
|
||||||
|
return strings.TrimSpace(value[:idx])
|
||||||
|
}
|
||||||
|
case "U54":
|
||||||
|
return value
|
||||||
|
}
|
||||||
|
return value
|
||||||
|
}
|
||||||
|
|
||||||
|
func normalizeLooseVersion(value string) string {
|
||||||
|
if match := versionFragmentRE.FindString(strings.TrimSpace(value)); match != "" {
|
||||||
|
return match
|
||||||
|
}
|
||||||
|
return strings.TrimSpace(value)
|
||||||
|
}
|
||||||
|
|
||||||
func slotLabelFromCode(code string) string {
|
func slotLabelFromCode(code string) string {
|
||||||
parts := strings.Split(code, ".")
|
parts := strings.Split(code, ".")
|
||||||
if len(parts) < 3 {
|
if len(parts) < 3 {
|
||||||
@@ -1132,6 +1451,16 @@ func slotLabelFromCode(code string) string {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func fabricIDFromPath(path string) string {
|
||||||
|
parts := strings.Split(strings.Trim(path, "/"), "/")
|
||||||
|
for i := 0; i+1 < len(parts); i++ {
|
||||||
|
if parts[i] == "Fabrics" {
|
||||||
|
return parts[i+1]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return ""
|
||||||
|
}
|
||||||
|
|
||||||
func inferSeverity(message string) models.Severity {
|
func inferSeverity(message string) models.Severity {
|
||||||
lower := strings.ToLower(message)
|
lower := strings.ToLower(message)
|
||||||
switch {
|
switch {
|
||||||
@@ -1261,6 +1590,24 @@ func asInt64(v any) int64 {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func asFloat64(v any) float64 {
|
||||||
|
switch t := v.(type) {
|
||||||
|
case float64:
|
||||||
|
return t
|
||||||
|
case float32:
|
||||||
|
return float64(t)
|
||||||
|
case int:
|
||||||
|
return float64(t)
|
||||||
|
case int64:
|
||||||
|
return float64(t)
|
||||||
|
case json.Number:
|
||||||
|
f, _ := t.Float64()
|
||||||
|
return f
|
||||||
|
default:
|
||||||
|
return 0
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func asOptionalInt(v any) *int {
|
func asOptionalInt(v any) *int {
|
||||||
switch value := v.(type) {
|
switch value := v.(type) {
|
||||||
case float64:
|
case float64:
|
||||||
@@ -1274,6 +1621,11 @@ func asOptionalInt(v any) *int {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func asBool(v any) bool {
|
||||||
|
b, ok := v.(bool)
|
||||||
|
return ok && b
|
||||||
|
}
|
||||||
|
|
||||||
func valueOr(v, fallback string) string {
|
func valueOr(v, fallback string) string {
|
||||||
if strings.TrimSpace(v) != "" {
|
if strings.TrimSpace(v) != "" {
|
||||||
return strings.TrimSpace(v)
|
return strings.TrimSpace(v)
|
||||||
@@ -1281,6 +1633,73 @@ func valueOr(v, fallback string) string {
|
|||||||
return strings.TrimSpace(fallback)
|
return strings.TrimSpace(fallback)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func stringSlice(v any) []string {
|
||||||
|
items, ok := v.([]any)
|
||||||
|
if !ok {
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
out := make([]string, 0, len(items))
|
||||||
|
for _, item := range items {
|
||||||
|
value := strings.TrimSpace(asString(item))
|
||||||
|
if value == "" {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
out = append(out, value)
|
||||||
|
}
|
||||||
|
return out
|
||||||
|
}
|
||||||
|
|
||||||
|
func firstDurableName(doc map[string]any) string {
|
||||||
|
items, ok := doc["Identifiers"].([]any)
|
||||||
|
if !ok {
|
||||||
|
return ""
|
||||||
|
}
|
||||||
|
for _, item := range items {
|
||||||
|
entry, ok := item.(map[string]any)
|
||||||
|
if !ok {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
if value := strings.TrimSpace(asString(entry["DurableName"])); value != "" {
|
||||||
|
return value
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return ""
|
||||||
|
}
|
||||||
|
|
||||||
|
func mergeDetailMaps(base, extra map[string]any) map[string]any {
|
||||||
|
if len(extra) == 0 {
|
||||||
|
return base
|
||||||
|
}
|
||||||
|
if base == nil {
|
||||||
|
base = make(map[string]any, len(extra))
|
||||||
|
}
|
||||||
|
for key, value := range extra {
|
||||||
|
if _, exists := base[key]; !exists || isZeroValue(base[key]) {
|
||||||
|
base[key] = value
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return base
|
||||||
|
}
|
||||||
|
|
||||||
|
func isZeroValue(v any) bool {
|
||||||
|
switch t := v.(type) {
|
||||||
|
case nil:
|
||||||
|
return true
|
||||||
|
case string:
|
||||||
|
return strings.TrimSpace(t) == ""
|
||||||
|
case int:
|
||||||
|
return t == 0
|
||||||
|
case int64:
|
||||||
|
return t == 0
|
||||||
|
case float64:
|
||||||
|
return t == 0
|
||||||
|
case bool:
|
||||||
|
return !t
|
||||||
|
default:
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func boolPtr(v bool) *bool {
|
func boolPtr(v bool) *bool {
|
||||||
out := v
|
out := v
|
||||||
return &out
|
return &out
|
||||||
|
|||||||
@@ -27,6 +27,7 @@ func TestParseAHSInventory(t *testing.T) {
|
|||||||
content := makeAHSArchive(t, []ahsTestEntry{
|
content := makeAHSArchive(t, []ahsTestEntry{
|
||||||
{Name: "CUST_INFO.DAT", Payload: make([]byte, 16)},
|
{Name: "CUST_INFO.DAT", Payload: make([]byte, 16)},
|
||||||
{Name: "0000088-2026-03-30.zbb", Payload: gzipBytes(t, []byte(sampleInventoryBlob()))},
|
{Name: "0000088-2026-03-30.zbb", Payload: gzipBytes(t, []byte(sampleInventoryBlob()))},
|
||||||
|
{Name: "bcert.pkg", Payload: []byte(sampleBCertBlob())},
|
||||||
})
|
})
|
||||||
|
|
||||||
result, err := p.Parse([]parser.ExtractedFile{{
|
result, err := p.Parse([]parser.ExtractedFile{{
|
||||||
@@ -73,6 +74,9 @@ func TestParseAHSInventory(t *testing.T) {
|
|||||||
if result.Hardware.PowerSupply[0].SerialNumber != "5XUWB0C4DJG4BV" {
|
if result.Hardware.PowerSupply[0].SerialNumber != "5XUWB0C4DJG4BV" {
|
||||||
t.Fatalf("unexpected PSU serial: %q", result.Hardware.PowerSupply[0].SerialNumber)
|
t.Fatalf("unexpected PSU serial: %q", result.Hardware.PowerSupply[0].SerialNumber)
|
||||||
}
|
}
|
||||||
|
if result.Hardware.PowerSupply[0].Firmware != "2.00" {
|
||||||
|
t.Fatalf("unexpected PSU firmware: %q", result.Hardware.PowerSupply[0].Firmware)
|
||||||
|
}
|
||||||
|
|
||||||
if len(result.Hardware.Storage) != 1 {
|
if len(result.Hardware.Storage) != 1 {
|
||||||
t.Fatalf("expected one physical drive, got %d", len(result.Hardware.Storage))
|
t.Fatalf("expected one physical drive, got %d", len(result.Hardware.Storage))
|
||||||
@@ -93,6 +97,8 @@ func TestParseAHSInventory(t *testing.T) {
|
|||||||
}
|
}
|
||||||
foundILO := false
|
foundILO := false
|
||||||
foundControllerFW := false
|
foundControllerFW := false
|
||||||
|
foundNICFW := false
|
||||||
|
foundBackplaneFW := false
|
||||||
for _, item := range result.Hardware.Firmware {
|
for _, item := range result.Hardware.Firmware {
|
||||||
if item.DeviceName == "iLO 6" && item.Version == "v1.63p20" {
|
if item.DeviceName == "iLO 6" && item.Version == "v1.63p20" {
|
||||||
foundILO = true
|
foundILO = true
|
||||||
@@ -100,6 +106,12 @@ func TestParseAHSInventory(t *testing.T) {
|
|||||||
if item.DeviceName == "HPE MR408i-o Gen11" && item.Version == "52.26.3-5379" {
|
if item.DeviceName == "HPE MR408i-o Gen11" && item.Version == "52.26.3-5379" {
|
||||||
foundControllerFW = true
|
foundControllerFW = true
|
||||||
}
|
}
|
||||||
|
if item.DeviceName == "BCM 5719 1Gb 4p BASE-T OCP Adptr" && item.Version == "20.28.41" {
|
||||||
|
foundNICFW = true
|
||||||
|
}
|
||||||
|
if item.DeviceName == "8 SFF 24G x1NVMe/SAS UBM3 BC BP" && item.Version == "1.24" {
|
||||||
|
foundBackplaneFW = true
|
||||||
|
}
|
||||||
}
|
}
|
||||||
if !foundILO {
|
if !foundILO {
|
||||||
t.Fatalf("expected iLO firmware entry")
|
t.Fatalf("expected iLO firmware entry")
|
||||||
@@ -107,6 +119,31 @@ func TestParseAHSInventory(t *testing.T) {
|
|||||||
if !foundControllerFW {
|
if !foundControllerFW {
|
||||||
t.Fatalf("expected controller firmware entry")
|
t.Fatalf("expected controller firmware entry")
|
||||||
}
|
}
|
||||||
|
if !foundNICFW {
|
||||||
|
t.Fatalf("expected broadcom firmware entry")
|
||||||
|
}
|
||||||
|
if !foundBackplaneFW {
|
||||||
|
t.Fatalf("expected backplane firmware entry")
|
||||||
|
}
|
||||||
|
|
||||||
|
broadcomFound := false
|
||||||
|
backplaneFound := false
|
||||||
|
for _, nic := range result.Hardware.NetworkAdapters {
|
||||||
|
if nic.SerialNumber == "1CH0150001" && nic.Firmware == "20.28.41" {
|
||||||
|
broadcomFound = true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
for _, dev := range result.Hardware.Devices {
|
||||||
|
if dev.DeviceClass == "storage_backplane" && dev.Firmware == "1.24" {
|
||||||
|
backplaneFound = true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if !broadcomFound {
|
||||||
|
t.Fatalf("expected broadcom adapter firmware to be enriched")
|
||||||
|
}
|
||||||
|
if !backplaneFound {
|
||||||
|
t.Fatalf("expected backplane canonical device")
|
||||||
|
}
|
||||||
|
|
||||||
if len(result.Hardware.Devices) < 6 {
|
if len(result.Hardware.Devices) < 6 {
|
||||||
t.Fatalf("expected canonical devices, got %d", len(result.Hardware.Devices))
|
t.Fatalf("expected canonical devices, got %d", len(result.Hardware.Devices))
|
||||||
@@ -146,17 +183,35 @@ func TestParseExampleAHS(t *testing.T) {
|
|||||||
if len(result.Hardware.Storage) < 2 {
|
if len(result.Hardware.Storage) < 2 {
|
||||||
t.Fatalf("expected at least two drives, got %d", len(result.Hardware.Storage))
|
t.Fatalf("expected at least two drives, got %d", len(result.Hardware.Storage))
|
||||||
}
|
}
|
||||||
|
if len(result.Hardware.PowerSupply) != 2 {
|
||||||
|
t.Fatalf("expected exactly two PSUs, got %d: %+v", len(result.Hardware.PowerSupply), result.Hardware.PowerSupply)
|
||||||
|
}
|
||||||
|
|
||||||
foundController := false
|
foundController := false
|
||||||
|
foundBackplaneFW := false
|
||||||
|
foundNICFW := false
|
||||||
for _, device := range result.Hardware.Devices {
|
for _, device := range result.Hardware.Devices {
|
||||||
if device.Model == "HPE MR408i-o Gen11" && device.SerialNumber == "PXSFQ0BBIJY3B3" {
|
if device.Model == "HPE MR408i-o Gen11" && device.SerialNumber == "PXSFQ0BBIJY3B3" {
|
||||||
foundController = true
|
foundController = true
|
||||||
break
|
}
|
||||||
|
if device.DeviceClass == "storage_backplane" && device.Firmware == "1.24" {
|
||||||
|
foundBackplaneFW = true
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
if !foundController {
|
if !foundController {
|
||||||
t.Fatalf("expected MR408i-o controller in canonical devices")
|
t.Fatalf("expected MR408i-o controller in canonical devices")
|
||||||
}
|
}
|
||||||
|
for _, fw := range result.Hardware.Firmware {
|
||||||
|
if fw.DeviceName == "BCM 5719 1Gb 4p BASE-T OCP Adptr" && fw.Version == "20.28.41" {
|
||||||
|
foundNICFW = true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if !foundBackplaneFW {
|
||||||
|
t.Fatalf("expected backplane device in canonical devices")
|
||||||
|
}
|
||||||
|
if !foundNICFW {
|
||||||
|
t.Fatalf("expected broadcom firmware from bcert/pkg lockdown")
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
type ahsTestEntry struct {
|
type ahsTestEntry struct {
|
||||||
@@ -239,11 +294,17 @@ func sampleInventoryBlob() string {
|
|||||||
"03/30/2026 09:47:33",
|
"03/30/2026 09:47:33",
|
||||||
"iLO network link down.",
|
"iLO network link down.",
|
||||||
`{"@odata.id":"/redfish/v1/Systems/1/Storage/DE00A000/Controllers/0","@odata.type":"#StorageController.v1_7_0.StorageController","Id":"0","Name":"HPE MR408i-o Gen11","FirmwareVersion":"52.26.3-5379","Manufacturer":"HPE","Model":"HPE MR408i-o Gen11","PartNumber":"P58543-001","SKU":"P58335-B21","SerialNumber":"PXSFQ0BBIJY3B3","Status":{"State":"Enabled","Health":"OK"},"Location":{"PartLocation":{"ServiceLabel":"Slot=14","LocationType":"Slot","LocationOrdinalValue":14}},"PCIeInterface":{"PCIeType":"Gen4","LanesInUse":8}}`,
|
`{"@odata.id":"/redfish/v1/Systems/1/Storage/DE00A000/Controllers/0","@odata.type":"#StorageController.v1_7_0.StorageController","Id":"0","Name":"HPE MR408i-o Gen11","FirmwareVersion":"52.26.3-5379","Manufacturer":"HPE","Model":"HPE MR408i-o Gen11","PartNumber":"P58543-001","SKU":"P58335-B21","SerialNumber":"PXSFQ0BBIJY3B3","Status":{"State":"Enabled","Health":"OK"},"Location":{"PartLocation":{"ServiceLabel":"Slot=14","LocationType":"Slot","LocationOrdinalValue":14}},"PCIeInterface":{"PCIeType":"Gen4","LanesInUse":8}}`,
|
||||||
|
`{"@odata.id":"/redfish/v1/Fabrics/DE00A000","@odata.type":"#Fabric.v1_3_0.Fabric","Id":"DE00A000","Name":"8 SFF 24G x1NVMe/SAS UBM3 BC BP","FabricType":"MultiProtocol"}`,
|
||||||
|
`{"@odata.id":"/redfish/v1/Fabrics/DE00A000/Switches/1","@odata.type":"#Switch.v1_9_1.Switch","Id":"1","Name":"Direct Attached","Model":"UBM3","FirmwareVersion":"1.24","SupportedProtocols":["SAS","SATA","NVMe"],"SwitchType":"MultiProtocol","Status":{"State":"Enabled","Health":"OK"}}`,
|
||||||
`{"@odata.id":"/redfish/v1/Chassis/DE00A000/Drives/0","@odata.type":"#Drive.v1_17_0.Drive","Id":"0","Name":"480GB 6G SATA SSD","Status":{"State":"StandbyOffline","Health":"OK"},"PhysicalLocation":{"PartLocation":{"ServiceLabel":"Slot=14:Port=1:Box=3:Bay=1","LocationType":"Bay","LocationOrdinalValue":1}},"CapacityBytes":480103981056,"MediaType":"SSD","Model":"SAMSUNGMZ7L3480HCHQ-00A07","Protocol":"SATA","Revision":"JXTC604Q","SerialNumber":"S664NC0Y502720","PredictedMediaLifeLeftPercent":100}`,
|
`{"@odata.id":"/redfish/v1/Chassis/DE00A000/Drives/0","@odata.type":"#Drive.v1_17_0.Drive","Id":"0","Name":"480GB 6G SATA SSD","Status":{"State":"StandbyOffline","Health":"OK"},"PhysicalLocation":{"PartLocation":{"ServiceLabel":"Slot=14:Port=1:Box=3:Bay=1","LocationType":"Bay","LocationOrdinalValue":1}},"CapacityBytes":480103981056,"MediaType":"SSD","Model":"SAMSUNGMZ7L3480HCHQ-00A07","Protocol":"SATA","Revision":"JXTC604Q","SerialNumber":"S664NC0Y502720","PredictedMediaLifeLeftPercent":100}`,
|
||||||
`{"@odata.id":"/redfish/v1/Chassis/DE00A000/Drives/64515","@odata.type":"#Drive.v1_17_0.Drive","Id":"64515","Name":"Empty Bay","Status":{"State":"Absent","Health":"OK"}}`,
|
`{"@odata.id":"/redfish/v1/Chassis/DE00A000/Drives/64515","@odata.type":"#Drive.v1_17_0.Drive","Id":"64515","Name":"Empty Bay","Status":{"State":"Absent","Health":"OK"}}`,
|
||||||
)
|
)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func sampleBCertBlob() string {
|
||||||
|
return `<BC><MfgRecord><PowerSupplySlot id="0"><Present>Yes</Present><SerialNumber>5XUWB0C4DJG4BV</SerialNumber><FirmwareVersion>2.00</FirmwareVersion><SparePartNumber>P44412-001</SparePartNumber></PowerSupplySlot><FirmwareLockdown><SystemProgrammableLogicDevice>0x12</SystemProgrammableLogicDevice><ServerPlatformServicesSPSFirmware>6.1.4.47</ServerPlatformServicesSPSFirmware><STMicroGen11TPM>1.512</STMicroGen11TPM><HPEMR408i-oGen11>52.26.3-5379</HPEMR408i-oGen11><UBM3>UBM3/1.24</UBM3><BCM57191Gb4pBASE-TOCP3>20.28.41</BCM57191Gb4pBASE-TOCP3></FirmwareLockdown></MfgRecord></BC>`
|
||||||
|
}
|
||||||
|
|
||||||
func stringsJoin(parts ...string) string {
|
func stringsJoin(parts ...string) string {
|
||||||
return string(bytes.Join(func() [][]byte {
|
return string(bytes.Join(func() [][]byte {
|
||||||
out := make([][]byte, 0, len(parts))
|
out := make([][]byte, 0, len(parts))
|
||||||
|
|||||||
450
internal/parser/vendors/xfusion/hardware.go
vendored
450
internal/parser/vendors/xfusion/hardware.go
vendored
@@ -10,6 +10,33 @@ import (
|
|||||||
"git.mchus.pro/mchus/logpile/internal/parser"
|
"git.mchus.pro/mchus/logpile/internal/parser"
|
||||||
)
|
)
|
||||||
|
|
||||||
|
type xfusionNICCard struct {
|
||||||
|
Slot string
|
||||||
|
Model string
|
||||||
|
ProductName string
|
||||||
|
Vendor string
|
||||||
|
VendorID int
|
||||||
|
DeviceID int
|
||||||
|
BDF string
|
||||||
|
SerialNumber string
|
||||||
|
PartNumber string
|
||||||
|
}
|
||||||
|
|
||||||
|
type xfusionNetcardPort struct {
|
||||||
|
BDF string
|
||||||
|
MAC string
|
||||||
|
ActualMAC string
|
||||||
|
}
|
||||||
|
|
||||||
|
type xfusionNetcardSnapshot struct {
|
||||||
|
Timestamp time.Time
|
||||||
|
Slot string
|
||||||
|
ProductName string
|
||||||
|
Manufacturer string
|
||||||
|
Firmware string
|
||||||
|
Ports []xfusionNetcardPort
|
||||||
|
}
|
||||||
|
|
||||||
// ── FRU ──────────────────────────────────────────────────────────────────────
|
// ── FRU ──────────────────────────────────────────────────────────────────────
|
||||||
|
|
||||||
// parseFRUInfo parses fruinfo.txt and populates result.FRU and result.Hardware.BoardInfo.
|
// parseFRUInfo parses fruinfo.txt and populates result.FRU and result.Hardware.BoardInfo.
|
||||||
@@ -232,15 +259,15 @@ func parseCPUInfo(content []byte) []models.CPU {
|
|||||||
}
|
}
|
||||||
|
|
||||||
cpus = append(cpus, models.CPU{
|
cpus = append(cpus, models.CPU{
|
||||||
Socket: socketNum,
|
Socket: socketNum,
|
||||||
Model: model,
|
Model: model,
|
||||||
Cores: cores,
|
Cores: cores,
|
||||||
Threads: threads,
|
Threads: threads,
|
||||||
L1CacheKB: l1,
|
L1CacheKB: l1,
|
||||||
L2CacheKB: l2,
|
L2CacheKB: l2,
|
||||||
L3CacheKB: l3,
|
L3CacheKB: l3,
|
||||||
SerialNumber: sn,
|
SerialNumber: sn,
|
||||||
Status: "ok",
|
Status: "ok",
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
return cpus
|
return cpus
|
||||||
@@ -338,9 +365,9 @@ func parseMemInfo(content []byte) []models.MemoryDIMM {
|
|||||||
|
|
||||||
// ── Card Info (GPU + NIC) ─────────────────────────────────────────────────────
|
// ── Card Info (GPU + NIC) ─────────────────────────────────────────────────────
|
||||||
|
|
||||||
// parseCardInfo parses card_info file, extracting GPU and NIC entries.
|
// parseCardInfo parses card_info file, extracting GPU and OCP NIC card inventory.
|
||||||
// The file has named sections ("GPU Card Info", "OCP Card Info", etc.) each with a pipe-table.
|
// The file has named sections ("GPU Card Info", "OCP Card Info", etc.) each with a pipe-table.
|
||||||
func parseCardInfo(content []byte) (gpus []models.GPU, nics []models.NIC) {
|
func parseCardInfo(content []byte) (gpus []models.GPU, nicCards []xfusionNICCard) {
|
||||||
sections := splitPipeSections(content)
|
sections := splitPipeSections(content)
|
||||||
|
|
||||||
// Build BDF and VendorID/DeviceID map from PCIe Card Info: slot → info
|
// Build BDF and VendorID/DeviceID map from PCIe Card Info: slot → info
|
||||||
@@ -396,17 +423,22 @@ func parseCardInfo(content []byte) (gpus []models.GPU, nics []models.NIC) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
// OCP Card Info: NIC cards
|
// OCP Card Info: NIC cards
|
||||||
for i, row := range sections["ocp card info"] {
|
for _, row := range sections["ocp card info"] {
|
||||||
desc := strings.TrimSpace(row["card desc"])
|
slot := strings.TrimSpace(row["slot"])
|
||||||
sn := strings.TrimSpace(row["serialnumber"])
|
pcie := slotPCIe[slot]
|
||||||
nics = append(nics, models.NIC{
|
nicCards = append(nicCards, xfusionNICCard{
|
||||||
Name: fmt.Sprintf("OCP%d", i+1),
|
Slot: slot,
|
||||||
Model: desc,
|
Model: strings.TrimSpace(row["card desc"]),
|
||||||
SerialNumber: sn,
|
ProductName: strings.TrimSpace(row["card desc"]),
|
||||||
|
VendorID: parseHexInt(row["vender id"]),
|
||||||
|
DeviceID: parseHexInt(row["device id"]),
|
||||||
|
BDF: pcie.bdf,
|
||||||
|
SerialNumber: strings.TrimSpace(row["serialnumber"]),
|
||||||
|
PartNumber: strings.TrimSpace(row["partnum"]),
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
|
||||||
return gpus, nics
|
return gpus, nicCards
|
||||||
}
|
}
|
||||||
|
|
||||||
// splitPipeSections parses a multi-section file where each section starts with a
|
// splitPipeSections parses a multi-section file where each section starts with a
|
||||||
@@ -462,6 +494,301 @@ func parseHexInt(s string) int {
|
|||||||
return int(n)
|
return int(n)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func parseNetcardInfo(content []byte) []xfusionNetcardSnapshot {
|
||||||
|
if len(content) == 0 {
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
|
||||||
|
var snapshots []xfusionNetcardSnapshot
|
||||||
|
var current *xfusionNetcardSnapshot
|
||||||
|
var currentPort *xfusionNetcardPort
|
||||||
|
|
||||||
|
flushPort := func() {
|
||||||
|
if current == nil || currentPort == nil {
|
||||||
|
return
|
||||||
|
}
|
||||||
|
current.Ports = append(current.Ports, *currentPort)
|
||||||
|
currentPort = nil
|
||||||
|
}
|
||||||
|
flushSnapshot := func() {
|
||||||
|
if current == nil || !current.hasData() {
|
||||||
|
return
|
||||||
|
}
|
||||||
|
flushPort()
|
||||||
|
snapshots = append(snapshots, *current)
|
||||||
|
current = nil
|
||||||
|
}
|
||||||
|
|
||||||
|
for _, rawLine := range strings.Split(string(content), "\n") {
|
||||||
|
line := strings.TrimSpace(rawLine)
|
||||||
|
if line == "" {
|
||||||
|
flushPort()
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
if ts, ok := parseXFusionUTCTimestamp(line); ok {
|
||||||
|
if current == nil {
|
||||||
|
current = &xfusionNetcardSnapshot{Timestamp: ts}
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
if current.hasData() {
|
||||||
|
flushSnapshot()
|
||||||
|
current = &xfusionNetcardSnapshot{Timestamp: ts}
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
current.Timestamp = ts
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
if current == nil {
|
||||||
|
current = &xfusionNetcardSnapshot{}
|
||||||
|
}
|
||||||
|
if port := parseNetcardPortHeader(line); port != nil {
|
||||||
|
flushPort()
|
||||||
|
currentPort = port
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
if currentPort != nil {
|
||||||
|
if value, ok := parseSimpleKV(line, "MacAddr"); ok {
|
||||||
|
currentPort.MAC = value
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
if value, ok := parseSimpleKV(line, "ActualMac"); ok {
|
||||||
|
currentPort.ActualMAC = value
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if value, ok := parseSimpleKV(line, "ProductName"); ok {
|
||||||
|
current.ProductName = value
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
if value, ok := parseSimpleKV(line, "Manufacture"); ok {
|
||||||
|
current.Manufacturer = value
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
if value, ok := parseSimpleKV(line, "FirmwareVersion"); ok {
|
||||||
|
current.Firmware = value
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
if value, ok := parseSimpleKV(line, "SlotId"); ok {
|
||||||
|
current.Slot = value
|
||||||
|
}
|
||||||
|
}
|
||||||
|
flushSnapshot()
|
||||||
|
|
||||||
|
bestIndexBySlot := make(map[string]int)
|
||||||
|
for i, snapshot := range snapshots {
|
||||||
|
slot := strings.TrimSpace(snapshot.Slot)
|
||||||
|
if slot == "" {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
prevIdx, exists := bestIndexBySlot[slot]
|
||||||
|
if !exists || snapshot.isBetterThan(snapshots[prevIdx]) {
|
||||||
|
bestIndexBySlot[slot] = i
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
ordered := make([]xfusionNetcardSnapshot, 0, len(bestIndexBySlot))
|
||||||
|
for i, snapshot := range snapshots {
|
||||||
|
slot := strings.TrimSpace(snapshot.Slot)
|
||||||
|
bestIdx, ok := bestIndexBySlot[slot]
|
||||||
|
if !ok || bestIdx != i {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
ordered = append(ordered, snapshot)
|
||||||
|
delete(bestIndexBySlot, slot)
|
||||||
|
}
|
||||||
|
return ordered
|
||||||
|
}
|
||||||
|
|
||||||
|
func mergeNetworkAdapters(cards []xfusionNICCard, snapshots []xfusionNetcardSnapshot) ([]models.NetworkAdapter, []models.NIC) {
|
||||||
|
bySlotCard := make(map[string]xfusionNICCard, len(cards))
|
||||||
|
bySlotSnapshot := make(map[string]xfusionNetcardSnapshot, len(snapshots))
|
||||||
|
orderedSlots := make([]string, 0, len(cards)+len(snapshots))
|
||||||
|
seenSlots := make(map[string]struct{}, len(cards)+len(snapshots))
|
||||||
|
|
||||||
|
for _, card := range cards {
|
||||||
|
slot := strings.TrimSpace(card.Slot)
|
||||||
|
if slot == "" {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
bySlotCard[slot] = card
|
||||||
|
if _, seen := seenSlots[slot]; !seen {
|
||||||
|
orderedSlots = append(orderedSlots, slot)
|
||||||
|
seenSlots[slot] = struct{}{}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
for _, snapshot := range snapshots {
|
||||||
|
slot := strings.TrimSpace(snapshot.Slot)
|
||||||
|
if slot == "" {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
bySlotSnapshot[slot] = snapshot
|
||||||
|
if _, seen := seenSlots[slot]; !seen {
|
||||||
|
orderedSlots = append(orderedSlots, slot)
|
||||||
|
seenSlots[slot] = struct{}{}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
adapters := make([]models.NetworkAdapter, 0, len(orderedSlots))
|
||||||
|
legacyNICs := make([]models.NIC, 0, len(orderedSlots))
|
||||||
|
for _, slot := range orderedSlots {
|
||||||
|
card := bySlotCard[slot]
|
||||||
|
snapshot := bySlotSnapshot[slot]
|
||||||
|
|
||||||
|
model := firstNonEmpty(card.Model, snapshot.ProductName)
|
||||||
|
description := ""
|
||||||
|
if !strings.EqualFold(strings.TrimSpace(model), strings.TrimSpace(snapshot.ProductName)) {
|
||||||
|
description = strings.TrimSpace(snapshot.ProductName)
|
||||||
|
}
|
||||||
|
macs := snapshot.macAddresses()
|
||||||
|
bdf := firstNonEmpty(snapshot.primaryBDF(), card.BDF)
|
||||||
|
firmware := normalizeXFusionValue(snapshot.Firmware)
|
||||||
|
manufacturer := firstNonEmpty(snapshot.Manufacturer, card.Vendor)
|
||||||
|
portCount := len(snapshot.Ports)
|
||||||
|
if portCount == 0 && len(macs) > 0 {
|
||||||
|
portCount = len(macs)
|
||||||
|
}
|
||||||
|
if portCount == 0 {
|
||||||
|
portCount = 1
|
||||||
|
}
|
||||||
|
|
||||||
|
adapters = append(adapters, models.NetworkAdapter{
|
||||||
|
Slot: slot,
|
||||||
|
Location: "OCP",
|
||||||
|
Present: true,
|
||||||
|
BDF: bdf,
|
||||||
|
Model: model,
|
||||||
|
Description: description,
|
||||||
|
Vendor: manufacturer,
|
||||||
|
VendorID: card.VendorID,
|
||||||
|
DeviceID: card.DeviceID,
|
||||||
|
SerialNumber: card.SerialNumber,
|
||||||
|
PartNumber: card.PartNumber,
|
||||||
|
Firmware: firmware,
|
||||||
|
PortCount: portCount,
|
||||||
|
PortType: "ethernet",
|
||||||
|
MACAddresses: macs,
|
||||||
|
Status: "ok",
|
||||||
|
})
|
||||||
|
legacyNICs = append(legacyNICs, models.NIC{
|
||||||
|
Name: fmt.Sprintf("OCP%s", slot),
|
||||||
|
Model: model,
|
||||||
|
Description: description,
|
||||||
|
MACAddress: firstNonEmpty(macs...),
|
||||||
|
SerialNumber: card.SerialNumber,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
|
return adapters, legacyNICs
|
||||||
|
}
|
||||||
|
|
||||||
|
func parseXFusionUTCTimestamp(line string) (time.Time, bool) {
|
||||||
|
ts, err := time.Parse("2006-01-02 15:04:05 MST", strings.TrimSpace(line))
|
||||||
|
if err != nil {
|
||||||
|
return time.Time{}, false
|
||||||
|
}
|
||||||
|
return ts, true
|
||||||
|
}
|
||||||
|
|
||||||
|
func parseNetcardPortHeader(line string) *xfusionNetcardPort {
|
||||||
|
fields := strings.Fields(strings.TrimSpace(line))
|
||||||
|
if len(fields) < 2 || !strings.HasPrefix(strings.ToLower(fields[0]), "port") {
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
joined := strings.Join(fields[1:], " ")
|
||||||
|
if !strings.HasPrefix(strings.ToLower(joined), "bdf:") {
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
return &xfusionNetcardPort{BDF: strings.TrimSpace(joined[len("BDF:"):])}
|
||||||
|
}
|
||||||
|
|
||||||
|
func parseSimpleKV(line, key string) (string, bool) {
|
||||||
|
idx := strings.Index(line, ":")
|
||||||
|
if idx < 0 {
|
||||||
|
return "", false
|
||||||
|
}
|
||||||
|
gotKey := strings.TrimSpace(line[:idx])
|
||||||
|
if !strings.EqualFold(gotKey, key) {
|
||||||
|
return "", false
|
||||||
|
}
|
||||||
|
return strings.TrimSpace(line[idx+1:]), true
|
||||||
|
}
|
||||||
|
|
||||||
|
func normalizeXFusionValue(value string) string {
|
||||||
|
value = strings.TrimSpace(value)
|
||||||
|
switch strings.ToUpper(value) {
|
||||||
|
case "", "N/A", "NA", "UNKNOWN":
|
||||||
|
return ""
|
||||||
|
default:
|
||||||
|
return value
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func (s xfusionNetcardSnapshot) hasData() bool {
|
||||||
|
return strings.TrimSpace(s.Slot) != "" ||
|
||||||
|
strings.TrimSpace(s.ProductName) != "" ||
|
||||||
|
strings.TrimSpace(s.Manufacturer) != "" ||
|
||||||
|
strings.TrimSpace(s.Firmware) != "" ||
|
||||||
|
len(s.Ports) > 0
|
||||||
|
}
|
||||||
|
|
||||||
|
func (s xfusionNetcardSnapshot) score() int {
|
||||||
|
score := len(s.Ports)
|
||||||
|
if normalizeXFusionValue(s.Firmware) != "" {
|
||||||
|
score += 10
|
||||||
|
}
|
||||||
|
score += len(s.macAddresses()) * 2
|
||||||
|
return score
|
||||||
|
}
|
||||||
|
|
||||||
|
func (s xfusionNetcardSnapshot) isBetterThan(other xfusionNetcardSnapshot) bool {
|
||||||
|
if s.score() != other.score() {
|
||||||
|
return s.score() > other.score()
|
||||||
|
}
|
||||||
|
if !s.Timestamp.Equal(other.Timestamp) {
|
||||||
|
return s.Timestamp.After(other.Timestamp)
|
||||||
|
}
|
||||||
|
return len(s.Ports) > len(other.Ports)
|
||||||
|
}
|
||||||
|
|
||||||
|
func (s xfusionNetcardSnapshot) primaryBDF() string {
|
||||||
|
for _, port := range s.Ports {
|
||||||
|
if bdf := strings.TrimSpace(port.BDF); bdf != "" {
|
||||||
|
return bdf
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return ""
|
||||||
|
}
|
||||||
|
|
||||||
|
func (s xfusionNetcardSnapshot) macAddresses() []string {
|
||||||
|
out := make([]string, 0, len(s.Ports))
|
||||||
|
seen := make(map[string]struct{}, len(s.Ports))
|
||||||
|
for _, port := range s.Ports {
|
||||||
|
for _, candidate := range []string{port.ActualMAC, port.MAC} {
|
||||||
|
mac := normalizeMAC(candidate)
|
||||||
|
if mac == "" {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
if _, exists := seen[mac]; exists {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
seen[mac] = struct{}{}
|
||||||
|
out = append(out, mac)
|
||||||
|
break
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return out
|
||||||
|
}
|
||||||
|
|
||||||
|
func normalizeMAC(value string) string {
|
||||||
|
value = strings.ToUpper(strings.TrimSpace(value))
|
||||||
|
switch value {
|
||||||
|
case "", "N/A", "NA", "UNKNOWN", "00:00:00:00:00:00":
|
||||||
|
return ""
|
||||||
|
default:
|
||||||
|
return value
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
// ── PSU ───────────────────────────────────────────────────────────────────────
|
// ── PSU ───────────────────────────────────────────────────────────────────────
|
||||||
|
|
||||||
// parsePSUInfo parses the pipe-delimited psu_info.txt.
|
// parsePSUInfo parses the pipe-delimited psu_info.txt.
|
||||||
@@ -525,6 +852,11 @@ func parsePSUInfo(content []byte) []models.PSU {
|
|||||||
func parseStorageControllerInfo(content []byte, result *models.AnalysisResult) {
|
func parseStorageControllerInfo(content []byte, result *models.AnalysisResult) {
|
||||||
// File may contain multiple controller blocks; parse key:value pairs from each.
|
// File may contain multiple controller blocks; parse key:value pairs from each.
|
||||||
// We only look at the first occurrence of each key (first controller).
|
// We only look at the first occurrence of each key (first controller).
|
||||||
|
seen := make(map[string]struct{}, len(result.Hardware.Firmware))
|
||||||
|
for _, fw := range result.Hardware.Firmware {
|
||||||
|
key := strings.ToLower(strings.TrimSpace(fw.DeviceName + "\x00" + fw.Version + "\x00" + fw.Description))
|
||||||
|
seen[key] = struct{}{}
|
||||||
|
}
|
||||||
text := string(content)
|
text := string(content)
|
||||||
blocks := strings.Split(text, "RAID Controller #")
|
blocks := strings.Split(text, "RAID Controller #")
|
||||||
for _, block := range blocks[1:] { // skip pre-block preamble
|
for _, block := range blocks[1:] { // skip pre-block preamble
|
||||||
@@ -532,7 +864,7 @@ func parseStorageControllerInfo(content []byte, result *models.AnalysisResult) {
|
|||||||
name := firstNonEmpty(fields["Component Name"], fields["Controller Name"], fields["Controller Type"])
|
name := firstNonEmpty(fields["Component Name"], fields["Controller Name"], fields["Controller Type"])
|
||||||
firmware := fields["Firmware Version"]
|
firmware := fields["Firmware Version"]
|
||||||
if name != "" && firmware != "" {
|
if name != "" && firmware != "" {
|
||||||
result.Hardware.Firmware = append(result.Hardware.Firmware, models.FirmwareInfo{
|
appendXFusionFirmware(result, seen, models.FirmwareInfo{
|
||||||
DeviceName: name,
|
DeviceName: name,
|
||||||
Description: fields["Controller Name"],
|
Description: fields["Controller Name"],
|
||||||
Version: firmware,
|
Version: firmware,
|
||||||
@@ -541,6 +873,86 @@ func parseStorageControllerInfo(content []byte, result *models.AnalysisResult) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func parseAppRevision(content []byte, result *models.AnalysisResult) {
|
||||||
|
type firmwareLine struct {
|
||||||
|
deviceName string
|
||||||
|
description string
|
||||||
|
buildKey string
|
||||||
|
}
|
||||||
|
|
||||||
|
known := map[string]firmwareLine{
|
||||||
|
"Active iBMC Version": {deviceName: "iBMC", description: "active iBMC", buildKey: "Active iBMC Built"},
|
||||||
|
"Active BIOS Version": {deviceName: "BIOS", description: "active BIOS", buildKey: "Active BIOS Built"},
|
||||||
|
"CPLD Version": {deviceName: "CPLD", description: "mainboard CPLD"},
|
||||||
|
"SDK Version": {deviceName: "SDK", description: "iBMC SDK", buildKey: "SDK Built"},
|
||||||
|
"Active Uboot Version": {deviceName: "U-Boot", description: "active U-Boot"},
|
||||||
|
"Active Secure Bootloader Version": {deviceName: "Secure Bootloader", description: "active secure bootloader"},
|
||||||
|
"Active Secure Firmware Version": {deviceName: "Secure Firmware", description: "active secure firmware"},
|
||||||
|
}
|
||||||
|
|
||||||
|
values := parseAlignedKeyValues(content)
|
||||||
|
if result.Hardware.BoardInfo.ProductName == "" {
|
||||||
|
if productName := values["Product Name"]; productName != "" {
|
||||||
|
result.Hardware.BoardInfo.ProductName = productName
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
seen := make(map[string]struct{}, len(result.Hardware.Firmware))
|
||||||
|
for _, fw := range result.Hardware.Firmware {
|
||||||
|
key := strings.ToLower(strings.TrimSpace(fw.DeviceName + "\x00" + fw.Version + "\x00" + fw.Description))
|
||||||
|
seen[key] = struct{}{}
|
||||||
|
}
|
||||||
|
|
||||||
|
for key, meta := range known {
|
||||||
|
version := normalizeXFusionValue(values[key])
|
||||||
|
if version == "" {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
appendXFusionFirmware(result, seen, models.FirmwareInfo{
|
||||||
|
DeviceName: meta.deviceName,
|
||||||
|
Description: meta.description,
|
||||||
|
Version: version,
|
||||||
|
BuildTime: normalizeXFusionValue(values[meta.buildKey]),
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func parseAlignedKeyValues(content []byte) map[string]string {
|
||||||
|
values := make(map[string]string)
|
||||||
|
for _, rawLine := range strings.Split(string(content), "\n") {
|
||||||
|
line := strings.TrimRight(rawLine, "\r")
|
||||||
|
if !strings.Contains(line, ":") {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
idx := strings.Index(line, ":")
|
||||||
|
if idx < 0 {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
key := strings.TrimRight(line[:idx], " \t")
|
||||||
|
value := strings.TrimSpace(line[idx+1:])
|
||||||
|
if key == "" || value == "" || values[key] != "" {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
values[key] = value
|
||||||
|
}
|
||||||
|
return values
|
||||||
|
}
|
||||||
|
|
||||||
|
func appendXFusionFirmware(result *models.AnalysisResult, seen map[string]struct{}, fw models.FirmwareInfo) {
|
||||||
|
if result == nil || result.Hardware == nil {
|
||||||
|
return
|
||||||
|
}
|
||||||
|
key := strings.ToLower(strings.TrimSpace(fw.DeviceName + "\x00" + fw.Version + "\x00" + fw.Description))
|
||||||
|
if key == "" {
|
||||||
|
return
|
||||||
|
}
|
||||||
|
if _, exists := seen[key]; exists {
|
||||||
|
return
|
||||||
|
}
|
||||||
|
seen[key] = struct{}{}
|
||||||
|
result.Hardware.Firmware = append(result.Hardware.Firmware, fw)
|
||||||
|
}
|
||||||
|
|
||||||
// parseDiskInfo parses a single PhysicalDrivesInfo/DiskN/disk_info file.
|
// parseDiskInfo parses a single PhysicalDrivesInfo/DiskN/disk_info file.
|
||||||
func parseDiskInfo(content []byte) *models.Storage {
|
func parseDiskInfo(content []byte) *models.Storage {
|
||||||
fields := parseKeyValueBlock(content)
|
fields := parseKeyValueBlock(content)
|
||||||
|
|||||||
57
internal/parser/vendors/xfusion/parser.go
vendored
57
internal/parser/vendors/xfusion/parser.go
vendored
@@ -13,7 +13,7 @@ import (
|
|||||||
"git.mchus.pro/mchus/logpile/internal/parser"
|
"git.mchus.pro/mchus/logpile/internal/parser"
|
||||||
)
|
)
|
||||||
|
|
||||||
const parserVersion = "1.0"
|
const parserVersion = "1.1"
|
||||||
|
|
||||||
func init() {
|
func init() {
|
||||||
parser.Register(&Parser{})
|
parser.Register(&Parser{})
|
||||||
@@ -34,11 +34,15 @@ func (p *Parser) Detect(files []parser.ExtractedFile) int {
|
|||||||
path := strings.ToLower(f.Path)
|
path := strings.ToLower(f.Path)
|
||||||
switch {
|
switch {
|
||||||
case strings.Contains(path, "appdump/frudata/fruinfo.txt"):
|
case strings.Contains(path, "appdump/frudata/fruinfo.txt"):
|
||||||
confidence += 60
|
confidence += 50
|
||||||
|
case strings.Contains(path, "rtosdump/versioninfo/app_revision.txt"):
|
||||||
|
confidence += 30
|
||||||
case strings.Contains(path, "appdump/sensor_alarm/sensor_info.txt"):
|
case strings.Contains(path, "appdump/sensor_alarm/sensor_info.txt"):
|
||||||
confidence += 20
|
confidence += 10
|
||||||
case strings.Contains(path, "appdump/card_manage/card_info"):
|
case strings.Contains(path, "appdump/card_manage/card_info"):
|
||||||
confidence += 20
|
confidence += 20
|
||||||
|
case strings.Contains(path, "logdump/netcard/netcard_info.txt"):
|
||||||
|
confidence += 20
|
||||||
}
|
}
|
||||||
if confidence >= 100 {
|
if confidence >= 100 {
|
||||||
return 100
|
return 100
|
||||||
@@ -54,17 +58,21 @@ func (p *Parser) Parse(files []parser.ExtractedFile) (*models.AnalysisResult, er
|
|||||||
FRU: make([]models.FRUInfo, 0),
|
FRU: make([]models.FRUInfo, 0),
|
||||||
Sensors: make([]models.SensorReading, 0),
|
Sensors: make([]models.SensorReading, 0),
|
||||||
Hardware: &models.HardwareConfig{
|
Hardware: &models.HardwareConfig{
|
||||||
CPUs: make([]models.CPU, 0),
|
Firmware: make([]models.FirmwareInfo, 0),
|
||||||
Memory: make([]models.MemoryDIMM, 0),
|
Devices: make([]models.HardwareDevice, 0),
|
||||||
Storage: make([]models.Storage, 0),
|
CPUs: make([]models.CPU, 0),
|
||||||
GPUs: make([]models.GPU, 0),
|
Memory: make([]models.MemoryDIMM, 0),
|
||||||
NetworkCards: make([]models.NIC, 0),
|
Storage: make([]models.Storage, 0),
|
||||||
PowerSupply: make([]models.PSU, 0),
|
Volumes: make([]models.StorageVolume, 0),
|
||||||
Firmware: make([]models.FirmwareInfo, 0),
|
PCIeDevices: make([]models.PCIeDevice, 0),
|
||||||
|
GPUs: make([]models.GPU, 0),
|
||||||
|
NetworkCards: make([]models.NIC, 0),
|
||||||
|
NetworkAdapters: make([]models.NetworkAdapter, 0),
|
||||||
|
PowerSupply: make([]models.PSU, 0),
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
|
|
||||||
if f := findByPath(files, "appdump/frudata/fruinfo.txt"); f != nil {
|
if f := findByAnyPath(files, "appdump/frudata/fruinfo.txt", "rtosdump/versioninfo/fruinfo.txt"); f != nil {
|
||||||
parseFRUInfo(f.Content, result)
|
parseFRUInfo(f.Content, result)
|
||||||
}
|
}
|
||||||
if f := findByPath(files, "appdump/sensor_alarm/sensor_info.txt"); f != nil {
|
if f := findByPath(files, "appdump/sensor_alarm/sensor_info.txt"); f != nil {
|
||||||
@@ -76,10 +84,20 @@ func (p *Parser) Parse(files []parser.ExtractedFile) (*models.AnalysisResult, er
|
|||||||
if f := findByPath(files, "appdump/cpumem/mem_info"); f != nil {
|
if f := findByPath(files, "appdump/cpumem/mem_info"); f != nil {
|
||||||
result.Hardware.Memory = parseMemInfo(f.Content)
|
result.Hardware.Memory = parseMemInfo(f.Content)
|
||||||
}
|
}
|
||||||
|
var nicCards []xfusionNICCard
|
||||||
if f := findByPath(files, "appdump/card_manage/card_info"); f != nil {
|
if f := findByPath(files, "appdump/card_manage/card_info"); f != nil {
|
||||||
gpus, nics := parseCardInfo(f.Content)
|
gpus, cards := parseCardInfo(f.Content)
|
||||||
result.Hardware.GPUs = gpus
|
result.Hardware.GPUs = gpus
|
||||||
result.Hardware.NetworkCards = nics
|
nicCards = cards
|
||||||
|
}
|
||||||
|
if f := findByPath(files, "logdump/netcard/netcard_info.txt"); f != nil || len(nicCards) > 0 {
|
||||||
|
var content []byte
|
||||||
|
if f != nil {
|
||||||
|
content = f.Content
|
||||||
|
}
|
||||||
|
adapters, legacyNICs := mergeNetworkAdapters(nicCards, parseNetcardInfo(content))
|
||||||
|
result.Hardware.NetworkAdapters = adapters
|
||||||
|
result.Hardware.NetworkCards = legacyNICs
|
||||||
}
|
}
|
||||||
if f := findByPath(files, "appdump/bmc/psu_info.txt"); f != nil {
|
if f := findByPath(files, "appdump/bmc/psu_info.txt"); f != nil {
|
||||||
result.Hardware.PowerSupply = parsePSUInfo(f.Content)
|
result.Hardware.PowerSupply = parsePSUInfo(f.Content)
|
||||||
@@ -87,6 +105,9 @@ func (p *Parser) Parse(files []parser.ExtractedFile) (*models.AnalysisResult, er
|
|||||||
if f := findByPath(files, "appdump/storagemgnt/raid_controller_info.txt"); f != nil {
|
if f := findByPath(files, "appdump/storagemgnt/raid_controller_info.txt"); f != nil {
|
||||||
parseStorageControllerInfo(f.Content, result)
|
parseStorageControllerInfo(f.Content, result)
|
||||||
}
|
}
|
||||||
|
if f := findByPath(files, "rtosdump/versioninfo/app_revision.txt"); f != nil {
|
||||||
|
parseAppRevision(f.Content, result)
|
||||||
|
}
|
||||||
for _, f := range findDiskInfoFiles(files) {
|
for _, f := range findDiskInfoFiles(files) {
|
||||||
disk := parseDiskInfo(f.Content)
|
disk := parseDiskInfo(f.Content)
|
||||||
if disk != nil {
|
if disk != nil {
|
||||||
@@ -99,6 +120,7 @@ func (p *Parser) Parse(files []parser.ExtractedFile) (*models.AnalysisResult, er
|
|||||||
|
|
||||||
result.Protocol = "ipmi"
|
result.Protocol = "ipmi"
|
||||||
result.SourceType = models.SourceTypeArchive
|
result.SourceType = models.SourceTypeArchive
|
||||||
|
parser.ApplyManufacturedYearWeekFromFRU(result.FRU, result.Hardware)
|
||||||
|
|
||||||
return result, nil
|
return result, nil
|
||||||
}
|
}
|
||||||
@@ -113,6 +135,15 @@ func findByPath(files []parser.ExtractedFile, substring string) *parser.Extracte
|
|||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func findByAnyPath(files []parser.ExtractedFile, substrings ...string) *parser.ExtractedFile {
|
||||||
|
for _, substring := range substrings {
|
||||||
|
if f := findByPath(files, substring); f != nil {
|
||||||
|
return f
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
|
||||||
// findDiskInfoFiles returns all PhysicalDrivesInfo disk_info files.
|
// findDiskInfoFiles returns all PhysicalDrivesInfo disk_info files.
|
||||||
func findDiskInfoFiles(files []parser.ExtractedFile) []parser.ExtractedFile {
|
func findDiskInfoFiles(files []parser.ExtractedFile) []parser.ExtractedFile {
|
||||||
var out []parser.ExtractedFile
|
var out []parser.ExtractedFile
|
||||||
|
|||||||
113
internal/parser/vendors/xfusion/parser_test.go
vendored
113
internal/parser/vendors/xfusion/parser_test.go
vendored
@@ -1,8 +1,10 @@
|
|||||||
package xfusion
|
package xfusion
|
||||||
|
|
||||||
import (
|
import (
|
||||||
|
"strings"
|
||||||
"testing"
|
"testing"
|
||||||
|
|
||||||
|
"git.mchus.pro/mchus/logpile/internal/models"
|
||||||
"git.mchus.pro/mchus/logpile/internal/parser"
|
"git.mchus.pro/mchus/logpile/internal/parser"
|
||||||
)
|
)
|
||||||
|
|
||||||
@@ -26,6 +28,29 @@ func TestDetect_G5500V7(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestDetect_ServerFileExportMarkers(t *testing.T) {
|
||||||
|
p := &Parser{}
|
||||||
|
score := p.Detect([]parser.ExtractedFile{
|
||||||
|
{Path: "dump_info/RTOSDump/versioninfo/app_revision.txt", Content: []byte("Product Name: G5500 V7")},
|
||||||
|
{Path: "dump_info/LogDump/netcard/netcard_info.txt", Content: []byte("2026-02-04 03:54:06 UTC")},
|
||||||
|
{Path: "dump_info/AppDump/card_manage/card_info", Content: []byte("OCP Card Info")},
|
||||||
|
})
|
||||||
|
if score < 70 {
|
||||||
|
t.Fatalf("expected Detect score >= 70 for xFusion file export markers, got %d", score)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestDetect_Negative(t *testing.T) {
|
||||||
|
p := &Parser{}
|
||||||
|
score := p.Detect([]parser.ExtractedFile{
|
||||||
|
{Path: "logs/messages.txt", Content: []byte("plain text")},
|
||||||
|
{Path: "inventory.json", Content: []byte(`{"vendor":"other"}`)},
|
||||||
|
})
|
||||||
|
if score != 0 {
|
||||||
|
t.Fatalf("expected Detect score 0 for non-xFusion input, got %d", score)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func TestParse_G5500V7_BoardInfo(t *testing.T) {
|
func TestParse_G5500V7_BoardInfo(t *testing.T) {
|
||||||
files := loadTestArchive(t, "../../../../example/G5500V7_210619KUGGXGS2000015_20260318-1128.tar.gz")
|
files := loadTestArchive(t, "../../../../example/G5500V7_210619KUGGXGS2000015_20260318-1128.tar.gz")
|
||||||
p := &Parser{}
|
p := &Parser{}
|
||||||
@@ -126,6 +151,94 @@ func TestParse_G5500V7_NICs(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestParse_ServerFileExport_NetworkAdaptersAndFirmware(t *testing.T) {
|
||||||
|
p := &Parser{}
|
||||||
|
files := []parser.ExtractedFile{
|
||||||
|
{
|
||||||
|
Path: "dump_info/AppDump/card_manage/card_info",
|
||||||
|
Content: []byte(strings.TrimSpace(`
|
||||||
|
Pcie Card Info
|
||||||
|
Slot | Vender Id | Device Id | Sub Vender Id | Sub Device Id | Segment Number | Bus Number | Device Number | Function Number | Card Desc | Board Id | PCB Version | CPLD Version | Sub Card Bom Id | PartNum | SerialNumber | OriginalPartNum
|
||||||
|
1 | 0x15b3 | 0x101f | 0x1f24 | 0x2011 | 0x00 | 0x27 | 0x00 | 0x00 | MT2894 Family [ConnectX-6 Lx] | N/A | N/A | N/A | N/A | 0302Y238 | 02Y238X6RC000058 |
|
||||||
|
|
||||||
|
OCP Card Info
|
||||||
|
Slot | Vender Id | Device Id | Sub Vender Id | Sub Device Id | Segment Number | Bus Number | Device Number | Function Number | Card Desc | Board Id | PCB Version | CPLD Version | Sub Card Bom Id | PartNum | SerialNumber | OriginalPartNum
|
||||||
|
1 | 0x15b3 | 0x101f | 0x1f24 | 0x2011 | 0x00 | 0x27 | 0x00 | 0x00 | MT2894 Family [ConnectX-6 Lx] | N/A | N/A | N/A | N/A | 0302Y238 | 02Y238X6RC000058 |
|
||||||
|
`)),
|
||||||
|
},
|
||||||
|
{
|
||||||
|
Path: "dump_info/LogDump/netcard/netcard_info.txt",
|
||||||
|
Content: []byte(strings.TrimSpace(`
|
||||||
|
2026-02-04 03:54:06 UTC
|
||||||
|
ProductName :XC385
|
||||||
|
Manufacture :XFUSION
|
||||||
|
FirmwareVersion :26.39.2048
|
||||||
|
SlotId :1
|
||||||
|
Port0 BDF:0000:27:00.0
|
||||||
|
MacAddr:44:1A:4C:16:E8:03
|
||||||
|
ActualMac:44:1A:4C:16:E8:03
|
||||||
|
Port1 BDF:0000:27:00.1
|
||||||
|
MacAddr:00:00:00:00:00:00
|
||||||
|
ActualMac:44:1A:4C:16:E8:04
|
||||||
|
`)),
|
||||||
|
},
|
||||||
|
{
|
||||||
|
Path: "dump_info/RTOSDump/versioninfo/app_revision.txt",
|
||||||
|
Content: []byte(strings.TrimSpace(`
|
||||||
|
------------------- iBMC INFO -------------------
|
||||||
|
Active iBMC Version: (U68)3.08.05.85
|
||||||
|
Active iBMC Built: 16:46:26 Jan 4 2026
|
||||||
|
SDK Version: 13.16.30.16
|
||||||
|
SDK Built: 07:55:18 Dec 12 2025
|
||||||
|
Active BIOS Version: (U6216)01.02.08.17
|
||||||
|
Active BIOS Built: 00:00:00 Jan 05 2026
|
||||||
|
Product Name: G5500 V7
|
||||||
|
`)),
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
result, err := p.Parse(files)
|
||||||
|
if err != nil {
|
||||||
|
t.Fatalf("Parse: %v", err)
|
||||||
|
}
|
||||||
|
if result.Protocol != "ipmi" || result.SourceType != models.SourceTypeArchive {
|
||||||
|
t.Fatalf("unexpected source metadata: protocol=%q source_type=%q", result.Protocol, result.SourceType)
|
||||||
|
}
|
||||||
|
if result.Hardware == nil {
|
||||||
|
t.Fatal("Hardware is nil")
|
||||||
|
}
|
||||||
|
if len(result.Hardware.NetworkAdapters) != 1 {
|
||||||
|
t.Fatalf("expected 1 network adapter, got %d", len(result.Hardware.NetworkAdapters))
|
||||||
|
}
|
||||||
|
adapter := result.Hardware.NetworkAdapters[0]
|
||||||
|
if adapter.BDF != "0000:27:00.0" {
|
||||||
|
t.Fatalf("expected network adapter BDF 0000:27:00.0, got %q", adapter.BDF)
|
||||||
|
}
|
||||||
|
if adapter.Firmware != "26.39.2048" {
|
||||||
|
t.Fatalf("expected network adapter firmware 26.39.2048, got %q", adapter.Firmware)
|
||||||
|
}
|
||||||
|
if adapter.SerialNumber != "02Y238X6RC000058" {
|
||||||
|
t.Fatalf("expected network adapter serial from card_info, got %q", adapter.SerialNumber)
|
||||||
|
}
|
||||||
|
if len(adapter.MACAddresses) != 2 || adapter.MACAddresses[0] != "44:1A:4C:16:E8:03" || adapter.MACAddresses[1] != "44:1A:4C:16:E8:04" {
|
||||||
|
t.Fatalf("unexpected MAC addresses: %#v", adapter.MACAddresses)
|
||||||
|
}
|
||||||
|
|
||||||
|
fwByDevice := make(map[string]models.FirmwareInfo)
|
||||||
|
for _, fw := range result.Hardware.Firmware {
|
||||||
|
fwByDevice[fw.DeviceName] = fw
|
||||||
|
}
|
||||||
|
if fwByDevice["iBMC"].Version != "(U68)3.08.05.85" {
|
||||||
|
t.Fatalf("expected iBMC firmware from app_revision.txt, got %#v", fwByDevice["iBMC"])
|
||||||
|
}
|
||||||
|
if fwByDevice["BIOS"].Version != "(U6216)01.02.08.17" {
|
||||||
|
t.Fatalf("expected BIOS firmware from app_revision.txt, got %#v", fwByDevice["BIOS"])
|
||||||
|
}
|
||||||
|
if result.Hardware.BoardInfo.ProductName != "G5500 V7" {
|
||||||
|
t.Fatalf("expected board product fallback from app_revision.txt, got %q", result.Hardware.BoardInfo.ProductName)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func TestParse_G5500V7_PSUs(t *testing.T) {
|
func TestParse_G5500V7_PSUs(t *testing.T) {
|
||||||
files := loadTestArchive(t, "../../../../example/G5500V7_210619KUGGXGS2000015_20260318-1128.tar.gz")
|
files := loadTestArchive(t, "../../../../example/G5500V7_210619KUGGXGS2000015_20260318-1128.tar.gz")
|
||||||
p := &Parser{}
|
p := &Parser{}
|
||||||
|
|||||||
@@ -44,6 +44,9 @@ func TestParserParseExample(t *testing.T) {
|
|||||||
examplePath := filepath.Join("..", "..", "..", "..", "example", "xigmanas.txt")
|
examplePath := filepath.Join("..", "..", "..", "..", "example", "xigmanas.txt")
|
||||||
raw, err := os.ReadFile(examplePath)
|
raw, err := os.ReadFile(examplePath)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
|
if os.IsNotExist(err) {
|
||||||
|
t.Skipf("example file %s not present", examplePath)
|
||||||
|
}
|
||||||
t.Fatalf("read example file: %v", err)
|
t.Fatalf("read example file: %v", err)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -3,6 +3,8 @@ package server
|
|||||||
import (
|
import (
|
||||||
"bytes"
|
"bytes"
|
||||||
"encoding/json"
|
"encoding/json"
|
||||||
|
"fmt"
|
||||||
|
"net"
|
||||||
"net/http"
|
"net/http"
|
||||||
"net/http/httptest"
|
"net/http/httptest"
|
||||||
"strings"
|
"strings"
|
||||||
@@ -22,6 +24,7 @@ func newCollectTestServer() (*Server, *httptest.Server) {
|
|||||||
mux.HandleFunc("POST /api/collect", s.handleCollectStart)
|
mux.HandleFunc("POST /api/collect", s.handleCollectStart)
|
||||||
mux.HandleFunc("GET /api/collect/{id}", s.handleCollectStatus)
|
mux.HandleFunc("GET /api/collect/{id}", s.handleCollectStatus)
|
||||||
mux.HandleFunc("POST /api/collect/{id}/cancel", s.handleCollectCancel)
|
mux.HandleFunc("POST /api/collect/{id}/cancel", s.handleCollectCancel)
|
||||||
|
mux.HandleFunc("POST /api/collect/{id}/skip", s.handleCollectSkip)
|
||||||
return s, httptest.NewServer(mux)
|
return s, httptest.NewServer(mux)
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -29,7 +32,17 @@ func TestCollectProbe(t *testing.T) {
|
|||||||
_, ts := newCollectTestServer()
|
_, ts := newCollectTestServer()
|
||||||
defer ts.Close()
|
defer ts.Close()
|
||||||
|
|
||||||
body := `{"host":"bmc-off.local","protocol":"redfish","port":443,"username":"admin","auth_type":"password","password":"secret","tls_mode":"strict"}`
|
ln, err := net.Listen("tcp", "127.0.0.1:0")
|
||||||
|
if err != nil {
|
||||||
|
t.Fatalf("listen probe target: %v", err)
|
||||||
|
}
|
||||||
|
defer ln.Close()
|
||||||
|
addr, ok := ln.Addr().(*net.TCPAddr)
|
||||||
|
if !ok {
|
||||||
|
t.Fatalf("unexpected listener address type: %T", ln.Addr())
|
||||||
|
}
|
||||||
|
|
||||||
|
body := fmt.Sprintf(`{"host":"127.0.0.1","protocol":"redfish","port":%d,"username":"admin-off","auth_type":"password","password":"secret","tls_mode":"strict"}`, addr.Port)
|
||||||
resp, err := http.Post(ts.URL+"/api/collect/probe", "application/json", bytes.NewBufferString(body))
|
resp, err := http.Post(ts.URL+"/api/collect/probe", "application/json", bytes.NewBufferString(body))
|
||||||
if err != nil {
|
if err != nil {
|
||||||
t.Fatalf("post collect probe failed: %v", err)
|
t.Fatalf("post collect probe failed: %v", err)
|
||||||
@@ -53,9 +66,6 @@ func TestCollectProbe(t *testing.T) {
|
|||||||
if payload.HostPowerState != "Off" {
|
if payload.HostPowerState != "Off" {
|
||||||
t.Fatalf("expected host power state Off, got %q", payload.HostPowerState)
|
t.Fatalf("expected host power state Off, got %q", payload.HostPowerState)
|
||||||
}
|
}
|
||||||
if !payload.PowerControlAvailable {
|
|
||||||
t.Fatalf("expected power control to be available")
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
func TestCollectLifecycleToTerminal(t *testing.T) {
|
func TestCollectLifecycleToTerminal(t *testing.T) {
|
||||||
|
|||||||
@@ -21,13 +21,16 @@ func (c *mockConnector) Probe(ctx context.Context, req collector.Request) (*coll
|
|||||||
if strings.Contains(strings.ToLower(req.Host), "fail") {
|
if strings.Contains(strings.ToLower(req.Host), "fail") {
|
||||||
return nil, context.DeadlineExceeded
|
return nil, context.DeadlineExceeded
|
||||||
}
|
}
|
||||||
|
hostPoweredOn := true
|
||||||
|
if strings.Contains(strings.ToLower(req.Host), "off") || strings.Contains(strings.ToLower(req.Username), "off") {
|
||||||
|
hostPoweredOn = false
|
||||||
|
}
|
||||||
return &collector.ProbeResult{
|
return &collector.ProbeResult{
|
||||||
Reachable: true,
|
Reachable: true,
|
||||||
Protocol: c.protocol,
|
Protocol: c.protocol,
|
||||||
HostPowerState: map[bool]string{true: "On", false: "Off"}[!strings.Contains(strings.ToLower(req.Host), "off")],
|
HostPowerState: map[bool]string{true: "On", false: "Off"}[hostPoweredOn],
|
||||||
HostPoweredOn: !strings.Contains(strings.ToLower(req.Host), "off"),
|
HostPoweredOn: hostPoweredOn,
|
||||||
PowerControlAvailable: true,
|
SystemPath: "/redfish/v1/Systems/1",
|
||||||
SystemPath: "/redfish/v1/Systems/1",
|
|
||||||
}, nil
|
}, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -19,18 +19,15 @@ type CollectRequest struct {
|
|||||||
Password string `json:"password,omitempty"`
|
Password string `json:"password,omitempty"`
|
||||||
Token string `json:"token,omitempty"`
|
Token string `json:"token,omitempty"`
|
||||||
TLSMode string `json:"tls_mode"`
|
TLSMode string `json:"tls_mode"`
|
||||||
PowerOnIfHostOff bool `json:"power_on_if_host_off,omitempty"`
|
DebugPayloads bool `json:"debug_payloads,omitempty"`
|
||||||
StopHostAfterCollect bool `json:"stop_host_after_collect,omitempty"`
|
|
||||||
DebugPayloads bool `json:"debug_payloads,omitempty"`
|
|
||||||
}
|
}
|
||||||
|
|
||||||
type CollectProbeResponse struct {
|
type CollectProbeResponse struct {
|
||||||
Reachable bool `json:"reachable"`
|
Reachable bool `json:"reachable"`
|
||||||
Protocol string `json:"protocol,omitempty"`
|
Protocol string `json:"protocol,omitempty"`
|
||||||
HostPowerState string `json:"host_power_state,omitempty"`
|
HostPowerState string `json:"host_power_state,omitempty"`
|
||||||
HostPoweredOn bool `json:"host_powered_on"`
|
HostPoweredOn bool `json:"host_powered_on"`
|
||||||
PowerControlAvailable bool `json:"power_control_available"`
|
Message string `json:"message,omitempty"`
|
||||||
Message string `json:"message,omitempty"`
|
|
||||||
}
|
}
|
||||||
|
|
||||||
type CollectJobResponse struct {
|
type CollectJobResponse struct {
|
||||||
@@ -78,7 +75,8 @@ type Job struct {
|
|||||||
CreatedAt time.Time
|
CreatedAt time.Time
|
||||||
UpdatedAt time.Time
|
UpdatedAt time.Time
|
||||||
RequestMeta CollectRequestMeta
|
RequestMeta CollectRequestMeta
|
||||||
cancel func()
|
cancel func()
|
||||||
|
skipFn func()
|
||||||
}
|
}
|
||||||
|
|
||||||
type CollectModuleStatus struct {
|
type CollectModuleStatus struct {
|
||||||
|
|||||||
@@ -81,7 +81,7 @@ func BuildHardwareDevices(hw *models.HardwareConfig) []models.HardwareDevice {
|
|||||||
}
|
}
|
||||||
|
|
||||||
for _, mem := range hw.Memory {
|
for _, mem := range hw.Memory {
|
||||||
if !mem.Present || mem.SizeMB == 0 {
|
if !mem.IsInstalledInventory() {
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
present := mem.Present
|
present := mem.Present
|
||||||
@@ -243,6 +243,8 @@ func BuildHardwareDevices(hw *models.HardwareConfig) []models.HardwareDevice {
|
|||||||
Source: "network_adapters",
|
Source: "network_adapters",
|
||||||
Slot: nic.Slot,
|
Slot: nic.Slot,
|
||||||
Location: nic.Location,
|
Location: nic.Location,
|
||||||
|
BDF: nic.BDF,
|
||||||
|
DeviceClass: "NetworkController",
|
||||||
VendorID: nic.VendorID,
|
VendorID: nic.VendorID,
|
||||||
DeviceID: nic.DeviceID,
|
DeviceID: nic.DeviceID,
|
||||||
Model: nic.Model,
|
Model: nic.Model,
|
||||||
@@ -253,6 +255,11 @@ func BuildHardwareDevices(hw *models.HardwareConfig) []models.HardwareDevice {
|
|||||||
PortCount: nic.PortCount,
|
PortCount: nic.PortCount,
|
||||||
PortType: nic.PortType,
|
PortType: nic.PortType,
|
||||||
MACAddresses: nic.MACAddresses,
|
MACAddresses: nic.MACAddresses,
|
||||||
|
LinkWidth: nic.LinkWidth,
|
||||||
|
LinkSpeed: nic.LinkSpeed,
|
||||||
|
MaxLinkWidth: nic.MaxLinkWidth,
|
||||||
|
MaxLinkSpeed: nic.MaxLinkSpeed,
|
||||||
|
NUMANode: nic.NUMANode,
|
||||||
Present: &present,
|
Present: &present,
|
||||||
Status: nic.Status,
|
Status: nic.Status,
|
||||||
StatusCheckedAt: nic.StatusCheckedAt,
|
StatusCheckedAt: nic.StatusCheckedAt,
|
||||||
|
|||||||
@@ -90,6 +90,98 @@ func TestBuildHardwareDevices_MemorySameSerialDifferentSlots_NotDeduped(t *testi
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestBuildHardwareDevices_ZeroSizeMemoryWithInventoryIsIncluded(t *testing.T) {
|
||||||
|
hw := &models.HardwareConfig{
|
||||||
|
Memory: []models.MemoryDIMM{
|
||||||
|
{
|
||||||
|
Slot: "PROC 1 DIMM 3",
|
||||||
|
Location: "PROC 1 DIMM 3",
|
||||||
|
Present: true,
|
||||||
|
SizeMB: 0,
|
||||||
|
Manufacturer: "Hynix",
|
||||||
|
SerialNumber: "2B5F92C6",
|
||||||
|
PartNumber: "HMCG88AEBRA115N",
|
||||||
|
Status: "ok",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
devices := BuildHardwareDevices(hw)
|
||||||
|
memoryCount := 0
|
||||||
|
for _, d := range devices {
|
||||||
|
if d.Kind != models.DeviceKindMemory {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
memoryCount++
|
||||||
|
if d.Slot != "PROC 1 DIMM 3" || d.PartNumber != "HMCG88AEBRA115N" || d.SerialNumber != "2B5F92C6" {
|
||||||
|
t.Fatalf("unexpected memory device: %+v", d)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if memoryCount != 1 {
|
||||||
|
t.Fatalf("expected 1 installed zero-size memory record, got %d", memoryCount)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestBuildHardwareDevices_NetworkAdapterPreservesPCIeMetadata(t *testing.T) {
|
||||||
|
hw := &models.HardwareConfig{
|
||||||
|
NetworkAdapters: []models.NetworkAdapter{
|
||||||
|
{
|
||||||
|
Slot: "1",
|
||||||
|
Location: "OCP",
|
||||||
|
Present: true,
|
||||||
|
BDF: "0000:27:00.0",
|
||||||
|
Model: "ConnectX-6 Lx",
|
||||||
|
VendorID: 0x15b3,
|
||||||
|
DeviceID: 0x101f,
|
||||||
|
SerialNumber: "NIC-001",
|
||||||
|
Firmware: "26.39.2048",
|
||||||
|
MACAddresses: []string{"44:1A:4C:16:E8:03", "44:1A:4C:16:E8:04"},
|
||||||
|
LinkWidth: 16,
|
||||||
|
LinkSpeed: "32 GT/s",
|
||||||
|
NUMANode: 1,
|
||||||
|
Status: "ok",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
devices := BuildHardwareDevices(hw)
|
||||||
|
for _, d := range devices {
|
||||||
|
if d.Kind != models.DeviceKindNetwork {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
if d.BDF != "0000:27:00.0" || d.LinkWidth != 16 || d.LinkSpeed != "32 GT/s" || d.NUMANode != 1 {
|
||||||
|
t.Fatalf("expected network PCIe metadata to be preserved, got %+v", d)
|
||||||
|
}
|
||||||
|
return
|
||||||
|
}
|
||||||
|
t.Fatal("expected network device in canonical inventory")
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestBuildSpecification_ZeroSizeMemoryWithInventoryIsShown(t *testing.T) {
|
||||||
|
hw := &models.HardwareConfig{
|
||||||
|
Memory: []models.MemoryDIMM{
|
||||||
|
{
|
||||||
|
Slot: "PROC 1 DIMM 3",
|
||||||
|
Present: true,
|
||||||
|
SizeMB: 0,
|
||||||
|
Manufacturer: "Hynix",
|
||||||
|
PartNumber: "HMCG88AEBRA115N",
|
||||||
|
SerialNumber: "2B5F92C6",
|
||||||
|
Status: "ok",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
spec := buildSpecification(hw)
|
||||||
|
for _, line := range spec {
|
||||||
|
if line.Category == "Память" && line.Name == "Hynix HMCG88AEBRA115N (size unknown)" && line.Quantity == 1 {
|
||||||
|
return
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
t.Fatalf("expected memory spec line for zero-size identified DIMM, got %+v", spec)
|
||||||
|
}
|
||||||
|
|
||||||
func TestBuildHardwareDevices_DuplicateSerials_AreAnnotated(t *testing.T) {
|
func TestBuildHardwareDevices_DuplicateSerials_AreAnnotated(t *testing.T) {
|
||||||
hw := &models.HardwareConfig{
|
hw := &models.HardwareConfig{
|
||||||
Memory: []models.MemoryDIMM{
|
Memory: []models.MemoryDIMM{
|
||||||
@@ -166,6 +258,31 @@ func TestBuildHardwareDevices_SkipsFirmwareOnlyNumericSlots(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestBuildHardwareDevices_NetworkDevicesUseUnifiedControllerClass(t *testing.T) {
|
||||||
|
hw := &models.HardwareConfig{
|
||||||
|
NetworkAdapters: []models.NetworkAdapter{
|
||||||
|
{
|
||||||
|
Slot: "NIC1",
|
||||||
|
Model: "Ethernet Adapter",
|
||||||
|
Vendor: "Intel",
|
||||||
|
Present: true,
|
||||||
|
},
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
devices := BuildHardwareDevices(hw)
|
||||||
|
for _, d := range devices {
|
||||||
|
if d.Kind != models.DeviceKindNetwork {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
if d.DeviceClass != "NetworkController" {
|
||||||
|
t.Fatalf("expected unified network controller class, got %+v", d)
|
||||||
|
}
|
||||||
|
return
|
||||||
|
}
|
||||||
|
t.Fatalf("expected one canonical network device")
|
||||||
|
}
|
||||||
|
|
||||||
func TestHandleGetConfig_ReturnsCanonicalHardware(t *testing.T) {
|
func TestHandleGetConfig_ReturnsCanonicalHardware(t *testing.T) {
|
||||||
srv := &Server{}
|
srv := &Server{}
|
||||||
srv.SetResult(&models.AnalysisResult{
|
srv.SetResult(&models.AnalysisResult{
|
||||||
|
|||||||
@@ -18,6 +18,7 @@ import (
|
|||||||
"sort"
|
"sort"
|
||||||
"strconv"
|
"strconv"
|
||||||
"strings"
|
"strings"
|
||||||
|
"sync"
|
||||||
"sync/atomic"
|
"sync/atomic"
|
||||||
"time"
|
"time"
|
||||||
|
|
||||||
@@ -530,11 +531,21 @@ func buildSpecification(hw *models.HardwareConfig) []SpecLine {
|
|||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
present := mem.Present != nil && *mem.Present
|
present := mem.Present != nil && *mem.Present
|
||||||
// Skip empty slots (not present or 0 size)
|
if !present {
|
||||||
if !present || mem.SizeMB == 0 {
|
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
// Include frequency if available
|
|
||||||
|
if mem.SizeMB == 0 {
|
||||||
|
name := strings.TrimSpace(strings.Join(nonEmptyStrings(mem.Manufacturer, mem.PartNumber, mem.Type), " "))
|
||||||
|
if name == "" {
|
||||||
|
name = "Installed DIMM (size unknown)"
|
||||||
|
} else {
|
||||||
|
name += " (size unknown)"
|
||||||
|
}
|
||||||
|
memGroups[name]++
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
|
||||||
key := ""
|
key := ""
|
||||||
currentSpeed := intFromDetails(mem.Details, "current_speed_mhz")
|
currentSpeed := intFromDetails(mem.Details, "current_speed_mhz")
|
||||||
if currentSpeed > 0 {
|
if currentSpeed > 0 {
|
||||||
@@ -626,6 +637,18 @@ func buildSpecification(hw *models.HardwareConfig) []SpecLine {
|
|||||||
return spec
|
return spec
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func nonEmptyStrings(values ...string) []string {
|
||||||
|
out := make([]string, 0, len(values))
|
||||||
|
for _, value := range values {
|
||||||
|
value = strings.TrimSpace(value)
|
||||||
|
if value == "" {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
out = append(out, value)
|
||||||
|
}
|
||||||
|
return out
|
||||||
|
}
|
||||||
|
|
||||||
func (s *Server) handleGetSerials(w http.ResponseWriter, r *http.Request) {
|
func (s *Server) handleGetSerials(w http.ResponseWriter, r *http.Request) {
|
||||||
result := s.GetResult()
|
result := s.GetResult()
|
||||||
if result == nil {
|
if result == nil {
|
||||||
@@ -1652,34 +1675,28 @@ func (s *Server) handleCollectProbe(w http.ResponseWriter, r *http.Request) {
|
|||||||
|
|
||||||
message := "Связь с BMC установлена"
|
message := "Связь с BMC установлена"
|
||||||
if result != nil {
|
if result != nil {
|
||||||
switch {
|
if result.HostPoweredOn {
|
||||||
case !result.HostPoweredOn && result.PowerControlAvailable:
|
message = "Связь с BMC установлена, host включён."
|
||||||
message = "Связь с BMC установлена, host выключен. Можно включить перед сбором."
|
} else {
|
||||||
case !result.HostPoweredOn:
|
message = "Связь с BMC установлена, host выключен. Данные инвентаря могут быть неполными."
|
||||||
message = "Связь с BMC установлена, host выключен."
|
|
||||||
default:
|
|
||||||
message = "Связь с BMC установлена, host включен."
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
hostPowerState := ""
|
hostPowerState := ""
|
||||||
hostPoweredOn := false
|
hostPoweredOn := false
|
||||||
powerControlAvailable := false
|
|
||||||
reachable := false
|
reachable := false
|
||||||
if result != nil {
|
if result != nil {
|
||||||
reachable = result.Reachable
|
reachable = result.Reachable
|
||||||
hostPowerState = strings.TrimSpace(result.HostPowerState)
|
hostPowerState = strings.TrimSpace(result.HostPowerState)
|
||||||
hostPoweredOn = result.HostPoweredOn
|
hostPoweredOn = result.HostPoweredOn
|
||||||
powerControlAvailable = result.PowerControlAvailable
|
|
||||||
}
|
}
|
||||||
|
|
||||||
jsonResponse(w, CollectProbeResponse{
|
jsonResponse(w, CollectProbeResponse{
|
||||||
Reachable: reachable,
|
Reachable: reachable,
|
||||||
Protocol: req.Protocol,
|
Protocol: req.Protocol,
|
||||||
HostPowerState: hostPowerState,
|
HostPowerState: hostPowerState,
|
||||||
HostPoweredOn: hostPoweredOn,
|
HostPoweredOn: hostPoweredOn,
|
||||||
PowerControlAvailable: powerControlAvailable,
|
Message: message,
|
||||||
Message: message,
|
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -1715,6 +1732,22 @@ func (s *Server) handleCollectCancel(w http.ResponseWriter, r *http.Request) {
|
|||||||
jsonResponse(w, job.toStatusResponse())
|
jsonResponse(w, job.toStatusResponse())
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func (s *Server) handleCollectSkip(w http.ResponseWriter, r *http.Request) {
|
||||||
|
jobID := strings.TrimSpace(r.PathValue("id"))
|
||||||
|
if !isValidCollectJobID(jobID) {
|
||||||
|
jsonError(w, "Invalid collect job id", http.StatusBadRequest)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
job, ok := s.jobManager.SkipJob(jobID)
|
||||||
|
if !ok {
|
||||||
|
jsonError(w, "Collect job not found", http.StatusNotFound)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
jsonResponse(w, job.toStatusResponse())
|
||||||
|
}
|
||||||
|
|
||||||
func (s *Server) startCollectionJob(jobID string, req CollectRequest) {
|
func (s *Server) startCollectionJob(jobID string, req CollectRequest) {
|
||||||
ctx, cancel := context.WithCancel(context.Background())
|
ctx, cancel := context.WithCancel(context.Background())
|
||||||
if attached := s.jobManager.AttachJobCancel(jobID, cancel); !attached {
|
if attached := s.jobManager.AttachJobCancel(jobID, cancel); !attached {
|
||||||
@@ -1722,6 +1755,11 @@ func (s *Server) startCollectionJob(jobID string, req CollectRequest) {
|
|||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
|
skipCh := make(chan struct{})
|
||||||
|
var skipOnce sync.Once
|
||||||
|
skipFn := func() { skipOnce.Do(func() { close(skipCh) }) }
|
||||||
|
s.jobManager.AttachJobSkip(jobID, skipFn)
|
||||||
|
|
||||||
go func() {
|
go func() {
|
||||||
connector, ok := s.getCollector(req.Protocol)
|
connector, ok := s.getCollector(req.Protocol)
|
||||||
if !ok {
|
if !ok {
|
||||||
@@ -1789,7 +1827,9 @@ func (s *Server) startCollectionJob(jobID string, req CollectRequest) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
result, err := connector.Collect(ctx, toCollectorRequest(req), emitProgress)
|
collectorReq := toCollectorRequest(req)
|
||||||
|
collectorReq.SkipHungCh = skipCh
|
||||||
|
result, err := connector.Collect(ctx, collectorReq, emitProgress)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
if ctx.Err() != nil {
|
if ctx.Err() != nil {
|
||||||
return
|
return
|
||||||
@@ -2013,9 +2053,7 @@ func toCollectorRequest(req CollectRequest) collector.Request {
|
|||||||
Password: req.Password,
|
Password: req.Password,
|
||||||
Token: req.Token,
|
Token: req.Token,
|
||||||
TLSMode: req.TLSMode,
|
TLSMode: req.TLSMode,
|
||||||
PowerOnIfHostOff: req.PowerOnIfHostOff,
|
DebugPayloads: req.DebugPayloads,
|
||||||
StopHostAfterCollect: req.StopHostAfterCollect,
|
|
||||||
DebugPayloads: req.DebugPayloads,
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -175,6 +175,43 @@ func (m *JobManager) UpdateJobDebugInfo(id string, info *CollectDebugInfo) (*Job
|
|||||||
return cloned, true
|
return cloned, true
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func (m *JobManager) AttachJobSkip(id string, skipFn func()) bool {
|
||||||
|
m.mu.Lock()
|
||||||
|
defer m.mu.Unlock()
|
||||||
|
|
||||||
|
job, ok := m.jobs[id]
|
||||||
|
if !ok || job == nil || isTerminalCollectStatus(job.Status) {
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
job.skipFn = skipFn
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
|
||||||
|
func (m *JobManager) SkipJob(id string) (*Job, bool) {
|
||||||
|
m.mu.Lock()
|
||||||
|
job, ok := m.jobs[id]
|
||||||
|
if !ok || job == nil {
|
||||||
|
m.mu.Unlock()
|
||||||
|
return nil, false
|
||||||
|
}
|
||||||
|
if isTerminalCollectStatus(job.Status) {
|
||||||
|
cloned := cloneJob(job)
|
||||||
|
m.mu.Unlock()
|
||||||
|
return cloned, true
|
||||||
|
}
|
||||||
|
skipFn := job.skipFn
|
||||||
|
job.skipFn = nil
|
||||||
|
job.UpdatedAt = time.Now().UTC()
|
||||||
|
job.Logs = append(job.Logs, formatCollectLogLine(job.UpdatedAt, "Пропуск зависших запросов по команде пользователя"))
|
||||||
|
cloned := cloneJob(job)
|
||||||
|
m.mu.Unlock()
|
||||||
|
|
||||||
|
if skipFn != nil {
|
||||||
|
skipFn()
|
||||||
|
}
|
||||||
|
return cloned, true
|
||||||
|
}
|
||||||
|
|
||||||
func (m *JobManager) AttachJobCancel(id string, cancelFn context.CancelFunc) bool {
|
func (m *JobManager) AttachJobCancel(id string, cancelFn context.CancelFunc) bool {
|
||||||
m.mu.Lock()
|
m.mu.Lock()
|
||||||
defer m.mu.Unlock()
|
defer m.mu.Unlock()
|
||||||
@@ -229,5 +266,6 @@ func cloneJob(job *Job) *Job {
|
|||||||
cloned.CurrentPhase = job.CurrentPhase
|
cloned.CurrentPhase = job.CurrentPhase
|
||||||
cloned.ETASeconds = job.ETASeconds
|
cloned.ETASeconds = job.ETASeconds
|
||||||
cloned.cancel = nil
|
cloned.cancel = nil
|
||||||
|
cloned.skipFn = nil
|
||||||
return &cloned
|
return &cloned
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -99,6 +99,7 @@ func (s *Server) setupRoutes() {
|
|||||||
s.mux.HandleFunc("POST /api/collect/probe", s.handleCollectProbe)
|
s.mux.HandleFunc("POST /api/collect/probe", s.handleCollectProbe)
|
||||||
s.mux.HandleFunc("GET /api/collect/{id}", s.handleCollectStatus)
|
s.mux.HandleFunc("GET /api/collect/{id}", s.handleCollectStatus)
|
||||||
s.mux.HandleFunc("POST /api/collect/{id}/cancel", s.handleCollectCancel)
|
s.mux.HandleFunc("POST /api/collect/{id}/cancel", s.handleCollectCancel)
|
||||||
|
s.mux.HandleFunc("POST /api/collect/{id}/skip", s.handleCollectSkip)
|
||||||
}
|
}
|
||||||
|
|
||||||
func (s *Server) Run() error {
|
func (s *Server) Run() error {
|
||||||
|
|||||||
@@ -24,6 +24,7 @@ func newFlowTestServer() (*Server, *httptest.Server) {
|
|||||||
mux.HandleFunc("POST /api/collect", s.handleCollectStart)
|
mux.HandleFunc("POST /api/collect", s.handleCollectStart)
|
||||||
mux.HandleFunc("GET /api/collect/{id}", s.handleCollectStatus)
|
mux.HandleFunc("GET /api/collect/{id}", s.handleCollectStatus)
|
||||||
mux.HandleFunc("POST /api/collect/{id}/cancel", s.handleCollectCancel)
|
mux.HandleFunc("POST /api/collect/{id}/cancel", s.handleCollectCancel)
|
||||||
|
mux.HandleFunc("POST /api/collect/{id}/skip", s.handleCollectSkip)
|
||||||
return s, httptest.NewServer(mux)
|
return s, httptest.NewServer(mux)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -211,8 +211,6 @@ main {
|
|||||||
}
|
}
|
||||||
|
|
||||||
#api-connect-btn,
|
#api-connect-btn,
|
||||||
#api-power-on-collect-btn,
|
|
||||||
#api-collect-off-btn,
|
|
||||||
#convert-folder-btn,
|
#convert-folder-btn,
|
||||||
#convert-run-btn,
|
#convert-run-btn,
|
||||||
#cancel-job-btn,
|
#cancel-job-btn,
|
||||||
@@ -229,8 +227,6 @@ main {
|
|||||||
}
|
}
|
||||||
|
|
||||||
#api-connect-btn:hover,
|
#api-connect-btn:hover,
|
||||||
#api-power-on-collect-btn:hover,
|
|
||||||
#api-collect-off-btn:hover,
|
|
||||||
#convert-folder-btn:hover,
|
#convert-folder-btn:hover,
|
||||||
#convert-run-btn:hover,
|
#convert-run-btn:hover,
|
||||||
#cancel-job-btn:hover,
|
#cancel-job-btn:hover,
|
||||||
@@ -241,8 +237,6 @@ main {
|
|||||||
#convert-run-btn:disabled,
|
#convert-run-btn:disabled,
|
||||||
#convert-folder-btn:disabled,
|
#convert-folder-btn:disabled,
|
||||||
#api-connect-btn:disabled,
|
#api-connect-btn:disabled,
|
||||||
#api-power-on-collect-btn:disabled,
|
|
||||||
#api-collect-off-btn:disabled,
|
|
||||||
#cancel-job-btn:disabled,
|
#cancel-job-btn:disabled,
|
||||||
.upload-area button:disabled {
|
.upload-area button:disabled {
|
||||||
opacity: 0.6;
|
opacity: 0.6;
|
||||||
@@ -311,64 +305,19 @@ main {
|
|||||||
border-top: 1px solid #e2e8f0;
|
border-top: 1px solid #e2e8f0;
|
||||||
}
|
}
|
||||||
|
|
||||||
.api-confirm-modal-backdrop {
|
.api-host-off-warning {
|
||||||
position: fixed;
|
|
||||||
inset: 0;
|
|
||||||
background: rgba(0, 0, 0, 0.45);
|
|
||||||
display: flex;
|
display: flex;
|
||||||
align-items: center;
|
align-items: center;
|
||||||
justify-content: center;
|
gap: 0.4rem;
|
||||||
z-index: 1000;
|
padding: 0.5rem 0.75rem;
|
||||||
}
|
background: #fef3c7;
|
||||||
|
border: 1px solid #f59e0b;
|
||||||
.api-confirm-modal {
|
|
||||||
background: #fff;
|
|
||||||
border-radius: 10px;
|
|
||||||
padding: 1.5rem 1.75rem;
|
|
||||||
max-width: 380px;
|
|
||||||
width: 90%;
|
|
||||||
box-shadow: 0 8px 32px rgba(0,0,0,0.18);
|
|
||||||
}
|
|
||||||
|
|
||||||
.api-confirm-modal p {
|
|
||||||
margin-bottom: 1.1rem;
|
|
||||||
font-size: 0.95rem;
|
|
||||||
color: #333;
|
|
||||||
line-height: 1.5;
|
|
||||||
}
|
|
||||||
|
|
||||||
.api-confirm-modal-actions {
|
|
||||||
display: flex;
|
|
||||||
gap: 0.6rem;
|
|
||||||
justify-content: flex-end;
|
|
||||||
}
|
|
||||||
|
|
||||||
.api-confirm-modal-actions button {
|
|
||||||
border: none;
|
|
||||||
border-radius: 6px;
|
border-radius: 6px;
|
||||||
padding: 0.5rem 1rem;
|
font-size: 0.875rem;
|
||||||
font-size: 0.9rem;
|
color: #92400e;
|
||||||
font-weight: 600;
|
font-weight: 500;
|
||||||
cursor: pointer;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
.api-confirm-modal-actions .btn-cancel {
|
|
||||||
background: #e2e8f0;
|
|
||||||
color: #333;
|
|
||||||
}
|
|
||||||
|
|
||||||
.api-confirm-modal-actions .btn-cancel:hover {
|
|
||||||
background: #cbd5e1;
|
|
||||||
}
|
|
||||||
|
|
||||||
.api-confirm-modal-actions .btn-confirm {
|
|
||||||
background: #dc3545;
|
|
||||||
color: #fff;
|
|
||||||
}
|
|
||||||
|
|
||||||
.api-confirm-modal-actions .btn-confirm:hover {
|
|
||||||
background: #b02a37;
|
|
||||||
}
|
|
||||||
|
|
||||||
.api-connect-status {
|
.api-connect-status {
|
||||||
margin-top: 0.75rem;
|
margin-top: 0.75rem;
|
||||||
@@ -445,6 +394,33 @@ main {
|
|||||||
cursor: default;
|
cursor: default;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
.job-status-actions {
|
||||||
|
display: flex;
|
||||||
|
gap: 0.5rem;
|
||||||
|
align-items: center;
|
||||||
|
}
|
||||||
|
|
||||||
|
#skip-hung-btn {
|
||||||
|
background: #f59e0b;
|
||||||
|
color: #fff;
|
||||||
|
border: none;
|
||||||
|
border-radius: 6px;
|
||||||
|
padding: 0.5rem 0.9rem;
|
||||||
|
font-size: 0.875rem;
|
||||||
|
font-weight: 600;
|
||||||
|
cursor: pointer;
|
||||||
|
transition: background-color 0.2s ease, opacity 0.2s ease;
|
||||||
|
}
|
||||||
|
|
||||||
|
#skip-hung-btn:hover {
|
||||||
|
background: #d97706;
|
||||||
|
}
|
||||||
|
|
||||||
|
#skip-hung-btn:disabled {
|
||||||
|
opacity: 0.6;
|
||||||
|
cursor: not-allowed;
|
||||||
|
}
|
||||||
|
|
||||||
.job-status-meta {
|
.job-status-meta {
|
||||||
display: grid;
|
display: grid;
|
||||||
grid-template-columns: repeat(auto-fit, minmax(230px, 1fr));
|
grid-template-columns: repeat(auto-fit, minmax(230px, 1fr));
|
||||||
|
|||||||
@@ -91,9 +91,9 @@ function initApiSource() {
|
|||||||
}
|
}
|
||||||
|
|
||||||
const cancelJobButton = document.getElementById('cancel-job-btn');
|
const cancelJobButton = document.getElementById('cancel-job-btn');
|
||||||
|
const skipHungButton = document.getElementById('skip-hung-btn');
|
||||||
const connectButton = document.getElementById('api-connect-btn');
|
const connectButton = document.getElementById('api-connect-btn');
|
||||||
const collectButton = document.getElementById('api-collect-btn');
|
const collectButton = document.getElementById('api-collect-btn');
|
||||||
const powerOffCheckbox = document.getElementById('api-power-off');
|
|
||||||
const fieldNames = ['host', 'port', 'username', 'password'];
|
const fieldNames = ['host', 'port', 'username', 'password'];
|
||||||
|
|
||||||
apiForm.addEventListener('submit', (event) => {
|
apiForm.addEventListener('submit', (event) => {
|
||||||
@@ -110,6 +110,11 @@ function initApiSource() {
|
|||||||
cancelCollectionJob();
|
cancelCollectionJob();
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
if (skipHungButton) {
|
||||||
|
skipHungButton.addEventListener('click', () => {
|
||||||
|
skipHungCollectionJob();
|
||||||
|
});
|
||||||
|
}
|
||||||
if (connectButton) {
|
if (connectButton) {
|
||||||
connectButton.addEventListener('click', () => {
|
connectButton.addEventListener('click', () => {
|
||||||
startApiProbe();
|
startApiProbe();
|
||||||
@@ -120,22 +125,6 @@ function initApiSource() {
|
|||||||
startCollectionWithOptions();
|
startCollectionWithOptions();
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
if (powerOffCheckbox) {
|
|
||||||
powerOffCheckbox.addEventListener('change', () => {
|
|
||||||
if (!powerOffCheckbox.checked) {
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
// If host was already on when probed, warn before enabling shutdown
|
|
||||||
if (apiProbeResult && apiProbeResult.host_powered_on) {
|
|
||||||
showConfirmModal(
|
|
||||||
'Хост был включён до начала сбора. Вы уверены, что хотите выключить его после завершения сбора?',
|
|
||||||
() => { /* confirmed — leave checked */ },
|
|
||||||
() => { powerOffCheckbox.checked = false; }
|
|
||||||
);
|
|
||||||
}
|
|
||||||
});
|
|
||||||
}
|
|
||||||
|
|
||||||
fieldNames.forEach((fieldName) => {
|
fieldNames.forEach((fieldName) => {
|
||||||
const field = apiForm.elements.namedItem(fieldName);
|
const field = apiForm.elements.namedItem(fieldName);
|
||||||
if (!field) {
|
if (!field) {
|
||||||
@@ -163,36 +152,6 @@ function initApiSource() {
|
|||||||
renderCollectionJob();
|
renderCollectionJob();
|
||||||
}
|
}
|
||||||
|
|
||||||
function showConfirmModal(message, onConfirm, onCancel) {
|
|
||||||
const backdrop = document.createElement('div');
|
|
||||||
backdrop.className = 'api-confirm-modal-backdrop';
|
|
||||||
backdrop.innerHTML = `
|
|
||||||
<div class="api-confirm-modal" role="dialog" aria-modal="true">
|
|
||||||
<p>${escapeHtml(message)}</p>
|
|
||||||
<div class="api-confirm-modal-actions">
|
|
||||||
<button class="btn-cancel">Отмена</button>
|
|
||||||
<button class="btn-confirm">Да, выключить</button>
|
|
||||||
</div>
|
|
||||||
</div>
|
|
||||||
`;
|
|
||||||
document.body.appendChild(backdrop);
|
|
||||||
|
|
||||||
const close = () => document.body.removeChild(backdrop);
|
|
||||||
backdrop.querySelector('.btn-cancel').addEventListener('click', () => {
|
|
||||||
close();
|
|
||||||
if (onCancel) onCancel();
|
|
||||||
});
|
|
||||||
backdrop.querySelector('.btn-confirm').addEventListener('click', () => {
|
|
||||||
close();
|
|
||||||
if (onConfirm) onConfirm();
|
|
||||||
});
|
|
||||||
backdrop.addEventListener('click', (e) => {
|
|
||||||
if (e.target === backdrop) {
|
|
||||||
close();
|
|
||||||
if (onCancel) onCancel();
|
|
||||||
}
|
|
||||||
});
|
|
||||||
}
|
|
||||||
|
|
||||||
function startApiProbe() {
|
function startApiProbe() {
|
||||||
const { isValid, payload, errors } = validateCollectForm();
|
const { isValid, payload, errors } = validateCollectForm();
|
||||||
@@ -255,11 +214,7 @@ function startCollectionWithOptions() {
|
|||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
const powerOnCheckbox = document.getElementById('api-power-on');
|
|
||||||
const powerOffCheckbox = document.getElementById('api-power-off');
|
|
||||||
const debugPayloads = document.getElementById('api-debug-payloads');
|
const debugPayloads = document.getElementById('api-debug-payloads');
|
||||||
payload.power_on_if_host_off = powerOnCheckbox ? powerOnCheckbox.checked : false;
|
|
||||||
payload.stop_host_after_collect = powerOffCheckbox ? powerOffCheckbox.checked : false;
|
|
||||||
payload.debug_payloads = debugPayloads ? debugPayloads.checked : false;
|
payload.debug_payloads = debugPayloads ? debugPayloads.checked : false;
|
||||||
startCollectionJob(payload);
|
startCollectionJob(payload);
|
||||||
}
|
}
|
||||||
@@ -268,8 +223,6 @@ function renderApiProbeState() {
|
|||||||
const connectButton = document.getElementById('api-connect-btn');
|
const connectButton = document.getElementById('api-connect-btn');
|
||||||
const probeOptions = document.getElementById('api-probe-options');
|
const probeOptions = document.getElementById('api-probe-options');
|
||||||
const status = document.getElementById('api-connect-status');
|
const status = document.getElementById('api-connect-status');
|
||||||
const powerOnCheckbox = document.getElementById('api-power-on');
|
|
||||||
const powerOffCheckbox = document.getElementById('api-power-off');
|
|
||||||
if (!connectButton || !probeOptions || !status) {
|
if (!connectButton || !probeOptions || !status) {
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
@@ -283,7 +236,6 @@ function renderApiProbeState() {
|
|||||||
}
|
}
|
||||||
|
|
||||||
const hostOn = apiProbeResult.host_powered_on;
|
const hostOn = apiProbeResult.host_powered_on;
|
||||||
const powerControlAvailable = apiProbeResult.power_control_available;
|
|
||||||
|
|
||||||
if (hostOn) {
|
if (hostOn) {
|
||||||
status.textContent = apiProbeResult.message || 'Связь с BMC есть, host включён.';
|
status.textContent = apiProbeResult.message || 'Связь с BMC есть, host включён.';
|
||||||
@@ -295,25 +247,15 @@ function renderApiProbeState() {
|
|||||||
|
|
||||||
probeOptions.classList.remove('hidden');
|
probeOptions.classList.remove('hidden');
|
||||||
|
|
||||||
// "Включить" checkbox
|
const hostOffWarning = document.getElementById('api-host-off-warning');
|
||||||
if (powerOnCheckbox) {
|
if (hostOffWarning) {
|
||||||
if (hostOn) {
|
if (hostOn) {
|
||||||
// Host already on — checkbox is checked and disabled
|
hostOffWarning.classList.add('hidden');
|
||||||
powerOnCheckbox.checked = true;
|
|
||||||
powerOnCheckbox.disabled = true;
|
|
||||||
} else {
|
} else {
|
||||||
// Host off — default: checked (will power on), enabled
|
hostOffWarning.classList.remove('hidden');
|
||||||
powerOnCheckbox.checked = true;
|
|
||||||
powerOnCheckbox.disabled = !powerControlAvailable;
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// "Выключить" checkbox — default: unchecked
|
|
||||||
if (powerOffCheckbox) {
|
|
||||||
powerOffCheckbox.checked = false;
|
|
||||||
powerOffCheckbox.disabled = !powerControlAvailable;
|
|
||||||
}
|
|
||||||
|
|
||||||
connectButton.textContent = 'Переподключиться';
|
connectButton.textContent = 'Переподключиться';
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -535,6 +477,36 @@ function pollCollectionJobStatus() {
|
|||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
|
function skipHungCollectionJob() {
|
||||||
|
if (!collectionJob || isCollectionJobTerminal(collectionJob.status)) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
const btn = document.getElementById('skip-hung-btn');
|
||||||
|
if (btn) {
|
||||||
|
btn.disabled = true;
|
||||||
|
btn.textContent = 'Пропуск...';
|
||||||
|
}
|
||||||
|
fetch(`/api/collect/${encodeURIComponent(collectionJob.id)}/skip`, {
|
||||||
|
method: 'POST'
|
||||||
|
})
|
||||||
|
.then(async (response) => {
|
||||||
|
const body = await response.json().catch(() => ({}));
|
||||||
|
if (!response.ok) {
|
||||||
|
throw new Error(body.error || 'Не удалось пропустить зависшие запросы');
|
||||||
|
}
|
||||||
|
syncServerLogs(body.logs);
|
||||||
|
renderCollectionJob();
|
||||||
|
})
|
||||||
|
.catch((err) => {
|
||||||
|
appendJobLog(`Ошибка пропуска: ${err.message}`);
|
||||||
|
if (btn) {
|
||||||
|
btn.disabled = false;
|
||||||
|
btn.textContent = 'Пропустить зависшие';
|
||||||
|
}
|
||||||
|
renderCollectionJob();
|
||||||
|
});
|
||||||
|
}
|
||||||
|
|
||||||
function cancelCollectionJob() {
|
function cancelCollectionJob() {
|
||||||
if (!collectionJob || isCollectionJobTerminal(collectionJob.status)) {
|
if (!collectionJob || isCollectionJobTerminal(collectionJob.status)) {
|
||||||
return;
|
return;
|
||||||
@@ -671,6 +643,19 @@ function renderCollectionJob() {
|
|||||||
)).join('');
|
)).join('');
|
||||||
|
|
||||||
cancelButton.disabled = isTerminal;
|
cancelButton.disabled = isTerminal;
|
||||||
|
|
||||||
|
const skipBtn = document.getElementById('skip-hung-btn');
|
||||||
|
if (skipBtn) {
|
||||||
|
const isCollecting = !isTerminal && collectionJob.status === 'running';
|
||||||
|
if (isCollecting) {
|
||||||
|
skipBtn.classList.remove('hidden');
|
||||||
|
} else {
|
||||||
|
skipBtn.classList.add('hidden');
|
||||||
|
skipBtn.disabled = false;
|
||||||
|
skipBtn.textContent = 'Пропустить зависшие';
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
setApiFormBlocked(!isTerminal);
|
setApiFormBlocked(!isTerminal);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -80,18 +80,12 @@
|
|||||||
</div>
|
</div>
|
||||||
<div id="api-connect-status" class="api-connect-status"></div>
|
<div id="api-connect-status" class="api-connect-status"></div>
|
||||||
<div id="api-probe-options" class="api-probe-options hidden">
|
<div id="api-probe-options" class="api-probe-options hidden">
|
||||||
<label class="api-form-checkbox" for="api-power-on">
|
<div id="api-host-off-warning" class="api-host-off-warning hidden">
|
||||||
<input id="api-power-on" name="power_on_if_host_off" type="checkbox">
|
⚠ Host выключен — данные инвентаря могут быть неполными
|
||||||
<span>Включить перед сбором</span>
|
</div>
|
||||||
</label>
|
|
||||||
<label class="api-form-checkbox" for="api-power-off">
|
|
||||||
<input id="api-power-off" name="stop_host_after_collect" type="checkbox">
|
|
||||||
<span>Выключить после сбора</span>
|
|
||||||
</label>
|
|
||||||
<div class="api-probe-options-separator"></div>
|
|
||||||
<label class="api-form-checkbox" for="api-debug-payloads">
|
<label class="api-form-checkbox" for="api-debug-payloads">
|
||||||
<input id="api-debug-payloads" name="debug_payloads" type="checkbox">
|
<input id="api-debug-payloads" name="debug_payloads" type="checkbox">
|
||||||
<span>Сбор расширенных метрик для отладки</span>
|
<span>Сбор расширенных данных для диагностики</span>
|
||||||
</label>
|
</label>
|
||||||
<div class="api-form-actions">
|
<div class="api-form-actions">
|
||||||
<button id="api-collect-btn" type="submit">Собрать</button>
|
<button id="api-collect-btn" type="submit">Собрать</button>
|
||||||
@@ -102,7 +96,10 @@
|
|||||||
<section id="api-job-status" class="job-status hidden" aria-live="polite">
|
<section id="api-job-status" class="job-status hidden" aria-live="polite">
|
||||||
<div class="job-status-header">
|
<div class="job-status-header">
|
||||||
<h4>Статус задачи сбора</h4>
|
<h4>Статус задачи сбора</h4>
|
||||||
<button id="cancel-job-btn" type="button">Отменить</button>
|
<div class="job-status-actions">
|
||||||
|
<button id="skip-hung-btn" type="button" class="hidden" title="Прервать зависшие запросы и перейти к анализу собранных данных">Пропустить зависшие</button>
|
||||||
|
<button id="cancel-job-btn" type="button">Отменить</button>
|
||||||
|
</div>
|
||||||
</div>
|
</div>
|
||||||
<div class="job-status-meta">
|
<div class="job-status-meta">
|
||||||
<div><span class="meta-label">jobId:</span> <code id="job-id-value">-</code></div>
|
<div><span class="meta-label">jobId:</span> <code id="job-id-value">-</code></div>
|
||||||
|
|||||||
Reference in New Issue
Block a user