Skip to content

Commit

Permalink
Add sas protocol to discovery (#198)
Browse files Browse the repository at this point in the history
### 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`).
  • Loading branch information
turegano-equinix authored Jan 24, 2025
1 parent 0d05689 commit 5dc9c21
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion actions/inventory.go
Original file line number Diff line number Diff line change
Expand Up @@ -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
}

Expand Down

0 comments on commit 5dc9c21

Please sign in to comment.