Deprecated URI: redfish/v1/Managers/ {ManagerId}/VirtualMedia/

Bug #2039458 reported by lucica
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
High
Winicius Allan
Sushy
Fix Released
High
Winicius Allan
Sushy Tools
Fix Released
Medium
Dmitry Tantsur

Bug Description

The DMTF has deprecated the uri for the VirtualMedia in their latest spec, please see 6.144.2 from https://www.dmtf.org/sites/default/files/standards/documents/DSP2046_2023.2.pdf.

Suggestion is to let ironic/ironic-inspector follow the new uri, i.e., /redfish/v1/Systems/{ComputerSystemId}/VirtualMedia/ instead of /redfish/v1/Managers/
{ManagerId}/VirtualMedia/

From a compatibility point of view, we need to check that older servers' BMCs are "exposing" their virtualmedia on the new uri (which seems to be the case for some of them, because we've had some redundant information in the redfish schema wrt to the virtualmedia from some time ago...).

lucica (lg-suciu)
description: updated
Changed in ironic:
status: New → Triaged
importance: Undecided → High
Changed in sushy:
status: New → Triaged
importance: Undecided → High
Changed in sushy-tools:
status: New → Triaged
importance: Undecided → Medium
Changed in sushy:
assignee: nobody → Winicius Allan (winiciusallan)
Revision history for this message
Iury Gregory Melo Ferreira (iurygregory) wrote :

Assigning Dmitry to sushy-tools since he did the change https://github.com/openstack/sushy-tools/commit/a654a033dece53b8e6eb005025e37643d61144a5
Marking Fix Released since we have sushy-tools 1.1.0 with the code available.

Changed in sushy-tools:
assignee: nobody → Dmitry Tantsur (divius)
status: Triaged → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to sushy (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/sushy/+/904463

Changed in ironic:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to sushy (master)

Reviewed: https://review.opendev.org/c/openstack/sushy/+/904463
Committed: https://opendev.org/openstack/sushy/commit/a9c4a8e7c2e8ed242f038f0c56476d4a578a88f4
Submitter: "Zuul (22348)"
Branch: master

commit a9c4a8e7c2e8ed242f038f0c56476d4a578a88f4
Author: Winicius Silva <email address hidden>
Date: Fri Dec 29 01:02:08 2023 -0300

    Allows System to access VirtualMedia in Sushy

    After DMTF release a new version of their guide[1], there was an
    update in VirtualMedia URI.

    Deprecated: /redfish/v1/Managers/{ManagerId}/VirtualMedia/{VirtualMediaId} New:
    /redfish/v1/Systems/{ComputerSystemId}/VirtualMedia/{VirtualMediaId}

    So, it is needed that System has access to VirtualMedia in Sushy lib
    to later Ironic may be able to handle this. This way, i added a
    property in System that it can have access to VirtualMedia instance.

    [1]https://www.dmtf.org/sites/default/files/standards/documents/DSP2046_2023.2.pdf

    Partial-Bug: #2039458
    Change-Id: I7bd7db539eab158c283524aecdaa9ebebd033cf6

Changed in ironic:
assignee: nobody → Winicius Allan (winiciusallan)
Revision history for this message
Iury Gregory Melo Ferreira (iurygregory) wrote :

Sushy 4.8.0 contains the support to virtual media trough Systems

Changed in sushy:
status: Triaged → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (master)

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

commit 6d046ad7b403f3ea9c6b0fcdde52968794d4cb3a
Author: Winicius Silva <email address hidden>
Date: Sat Mar 9 19:37:04 2024 -0300

    Allow usage of virtual media via System

    DMTF now changes their Virtual Media URI to support Systems. Redfish driver now
    support this resource to boot, so it is needed that Ironic have a way to use it.

    Closes-Bug: #2039458
    Change-Id: I66e8edb847e93f96374072525222f05e7561fb07

Changed in ironic:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ironic 24.1.0

This issue was fixed in the openstack/ironic 24.1.0 release.

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

Other bug subscribers

Remote bug watches

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