From 5dc9c219b26823cd3be03def487734d599ab6d3b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ignacio=20Tur=C3=A9gano?= <98024527+turegano-equinix@users.noreply.github.com> Date: Fri, 24 Jan 2025 10:48:20 +0100 Subject: [PATCH] Add sas protocol to discovery (#198) ### What does this PR do This PR adds `sas` drive protocol to the supported drive protocols (`sata`, `nvme`) ### The HW vendor this change applies to (if applicable) This change is vendor-agnostic. ### The HW model number, product name this change applies to (if applicable) Not specific to any particular hardware model. Applicable to any drives using `sas` protocol. ### The BMC firmware and/or BIOS versions that this change applies to (if applicable) Not dependent on any specific firmware or BIOS version. ### What version of tooling - vendor specific or opensource does this change depend on (if applicable) ### How can this change be tested by a PR reviewer? Test with drives using `sas` protocol. ### Description for changelog/release notes Added: New supported drive protocol (`sas`). --- actions/inventory.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/actions/inventory.go b/actions/inventory.go index 90da2dfb..147bb9c6 100644 --- a/actions/inventory.go +++ b/actions/inventory.go @@ -450,7 +450,7 @@ func (a *InventoryCollectorAction) CollectDriveCapabilities(ctx context.Context) // // if theres others to be supported, the driveCapabilityCollectorByLogicalName() method // is to be updated to include the required support for SAS/USB/SCSI or other kinds of transports. - if !slices.Contains([]string{"sata", "nvme"}, drive.Protocol) { + if !slices.Contains([]string{"sata", "nvme", "sas"}, drive.Protocol) { continue }