Ironic

Cannot change/get indicator LED of disk on Redfish driver

Bug #2032590 reported by Vanou Ishii
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Confirmed
Medium
Vanou Ishii

Bug Description

Current Redfish driver uses Redfish SimpleStorage resource to change/get status of indicator LED on disk attached to server. However SimpleStorage resource doesn't provide such functionality. Storage resource actually does.

Vanou Ishii (vanou)
Changed in ironic:
assignee: nobody → Vanou Ishii (vanou)
Revision history for this message
Julia Kreger (juliaashleykreger) wrote :

Fun! I guess that is most likely an oversight during that features original creation, although we've not found much update of it, so really not that surprising.

Changed in ironic:
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to ironic (master)

Reviewed: https://review.opendev.org/c/openstack/ironic/+/892252
Committed: https://opendev.org/openstack/ironic/commit/ffde150aa7069b1e70540a2ab6f49a62a0853454
Submitter: "Zuul (22348)"
Branch: master

commit ffde150aa7069b1e70540a2ab6f49a62a0853454
Author: Vanou Ishii <email address hidden>
Date: Mon Aug 21 21:16:54 2023 -0400

    Transiton to Storage resource from SimpleStorage

    Current Redfish driver uses SimpleStorage resource
    to collect disk information.
    Storage resource is defined to provide more sophisticated
    functionality over SimpsleStorage resource. Some Redfish
    implementations may take transition from SimpleStorage to
    Storage.
    So Redfish driver's logic around disk should be provided
    through Storage resource with SimpleStorage compatibility.

    This commit does 2 things
    * Use Storage resource instead of SimpleStorage, if possible
    * Fix wrong disk indicator LED logic (SimpleStorage doesn't
      support indicator LED operation)

    Related-bug: 2032590
    Change-Id: I28abd75a41059c23028717db6a9766a5164313c7

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.