openstack/ironic
Code Issues Proposed changes

Fix device_type in attach/detach vmedia for Redfish

Browse Source 
We need to translate the generic device_type to sushy device_type

Change-Id: Id09e6f2b989513b41980c7b89c9d76edecc320db
This commit is contained in:
Riccardo Pittau 2024-04-18 17:18:57 +02:00
parent 5fabff87d1
commit 68fcc6b0d2
2 changed files with 11 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
ironic/drivers/modules/redfish/management.py
View File

@ -1346,7 +1346,8 @@ class RedfishManagement(base.ManagementInterface):
:param image_url: URL of the image to attach, HTTP or HTTPS.
"""
redfish_boot.insert_vmedia(task, image_url, device_type)
redfish_boot.insert_vmedia(task, image_url,
BOOT_DEVICE_MAP_REV[device_type])
@task_manager.require_exclusive_lock
def detach_virtual_media(self, task, device_types=None):
@ -1362,4 +1363,5 @@ class RedfishManagement(base.ManagementInterface):
redfish_boot.eject_vmedia(task)
else:
for device_type in device_types:
redfish_boot.eject_vmedia(task, device_type)
redfish_boot.eject_vmedia(task,
BOOT_DEVICE_MAP_REV[device_type])

7
ironic/releasenotes/notes/fix-device_type-redfish-advmedia-dff448c2f9085a47.yaml Normal file
View File

@ -0,0 +1,7 @@
---
fixes:
- |
Fixes an issue in Redfish generic attach and detach virtual media
where the virtual media devices were not recognized causing a
failure when attaching or detaching a single virtual media device.