)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"0fb56ffb54c9e618cfd4f0760ecc46b79c66abc9","unresolved":true,"context_lines":[{"line_number":19,"context_line":"* redfish_virtual_media_usb_id"},{"line_number":20,"context_line":"* redfish_virtual_media_cd_id"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"NOTE: I haven\u0027t yet tested if this actually fixes"},{"line_number":23,"context_line":"my issues, it just feels like the next thing to try"},{"line_number":24,"context_line":"so I am putting it here for visibility."},{"line_number":25,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"99fcb2ef_61db7f0d","line":22,"updated":"2025-08-27 15:32:01.000000000","message":"Turns out the CDROM drive is missing the attach call via REDFISH, and it only support eject (even when the drive is empty). Seems its only for the UI.\n\nAnyways, there is a follow on patch to request we boot USB, not CDROM, when doing virtual media. It also needs the existing config adding to skip the wiping of read only drives, because the virtual media USB is appearing as a disk.\n\nGo figure... but with these two patches, and that extra config, we have IPA booting, and cleaning completes for this hardware.\n\nI am very open to what the proper way should be to describe such hardware. There doesn\u0027t appear to be a good way to detect this, except looking for virtual media devices that support both ISO and USB that have an id that is either USB1 or USB2, but that seems quite nuts, and probably best in inspector logic to auto discover the correct virtual media device... maybe? I duno.","commit_id":"963657fe432f20a8f6bd4e4c4857d134c8ea6139"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"95cab6cd6d5ab22418adddb5d26e8bd468c7f8fa","unresolved":true,"context_lines":[{"line_number":19,"context_line":"* redfish_virtual_media_usb_id"},{"line_number":20,"context_line":"* redfish_virtual_media_cd_id"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"NOTE: I haven\u0027t yet tested if this actually fixes"},{"line_number":23,"context_line":"my issues, it just feels like the next thing to try"},{"line_number":24,"context_line":"so I am putting it here for visibility."},{"line_number":25,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"b8e8e54e_3593646c","line":22,"in_reply_to":"99fcb2ef_61db7f0d","updated":"2025-08-27 16:58:00.000000000","message":"I\u0027m going to link to this comment/change from https://etherpad.opendev.org/p/ironic-ptg-2026.1 as we have a topic about this (specifically inspired by the hardware you\u0027re targetting, I believe).\n\nThat\u0027ll be a good place to talk about moving-forward fixes.","commit_id":"963657fe432f20a8f6bd4e4c4857d134c8ea6139"}],"ironic/drivers/modules/redfish/boot.py":[{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"692d2098c80cc71aa5aac09222c656f9be5f1b64","unresolved":true,"context_lines":[{"line_number":330,"context_line":"            # next v_media device, if any"},{"line_number":331,"context_line":"            # This is needed to add support to Cisco UCSB and UCSX blades"},{"line_number":332,"context_line":"            # reference: https://bugs.launchpad.net/ironic/+bug/2031595"},{"line_number":333,"context_line":"            if (boot_device \u003d\u003d sushy.VIRTUAL_MEDIA_CD"},{"line_number":334,"context_line":"                and sushy.VIRTUAL_MEDIA_DVD in v_media.media_types):"},{"line_number":335,"context_line":"                LOG.debug(\"While looking for %(requested_device)s virtual \""},{"line_number":336,"context_line":"                          \"media device, found %(available_device)s \""}],"source_content_type":"text/x-python","patch_set":3,"id":"9225dba5_e197a904","line":333,"updated":"2025-08-20 16:34:58.000000000","message":"I\u0027m wondering if we couldn\u0027t check if v_media.media_types doesn\u0027t contain sushy.USB_STICK, if it does we wouldn\u0027t use it and would move to other slot...","commit_id":"fb926cb25c82ea9dc836acb7df1ce6d960a4d085"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"bdc1e3b9e95c953c2f7f1ac51346cc3c37947680","unresolved":true,"context_lines":[{"line_number":330,"context_line":"            # next v_media device, if any"},{"line_number":331,"context_line":"            # This is needed to add support to Cisco UCSB and UCSX blades"},{"line_number":332,"context_line":"            # reference: https://bugs.launchpad.net/ironic/+bug/2031595"},{"line_number":333,"context_line":"            if (boot_device \u003d\u003d sushy.VIRTUAL_MEDIA_CD"},{"line_number":334,"context_line":"                and sushy.VIRTUAL_MEDIA_DVD in v_media.media_types):"},{"line_number":335,"context_line":"                LOG.debug(\"While looking for %(requested_device)s virtual \""},{"line_number":336,"context_line":"                          \"media device, found %(available_device)s \""}],"source_content_type":"text/x-python","patch_set":3,"id":"70c0bfdc_2e5a9d73","line":333,"in_reply_to":"8d8ddf59_5a90ac1f","updated":"2025-08-27 13:16:35.000000000","message":"The problem in this case is the hardware says the CD and USB devices support both USB_STICK and ISO, but annoyingly the CD device doesn\u0027t have an insert functional available, even if we use it :facepalm:\n\nI generally like the idea of being more declarative for these edge cases. It would be nice if we can use inspection to check auto detect the crazy hardware and add the known workarounds automagically. Will be a fun PTG discussion.","commit_id":"fb926cb25c82ea9dc836acb7df1ce6d960a4d085"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"09b2e7ea902578f49df2fb9e30ed8fdce4905e0b","unresolved":true,"context_lines":[{"line_number":330,"context_line":"            # next v_media device, if any"},{"line_number":331,"context_line":"            # This is needed to add support to Cisco UCSB and UCSX blades"},{"line_number":332,"context_line":"            # reference: https://bugs.launchpad.net/ironic/+bug/2031595"},{"line_number":333,"context_line":"            if (boot_device \u003d\u003d sushy.VIRTUAL_MEDIA_CD"},{"line_number":334,"context_line":"                and sushy.VIRTUAL_MEDIA_DVD in v_media.media_types):"},{"line_number":335,"context_line":"                LOG.debug(\"While looking for %(requested_device)s virtual \""},{"line_number":336,"context_line":"                          \"media device, found %(available_device)s \""}],"source_content_type":"text/x-python","patch_set":3,"id":"8d8ddf59_5a90ac1f","line":333,"in_reply_to":"9225dba5_e197a904","updated":"2025-08-20 16:58:29.000000000","message":"FWIW, this is the general problem I\u0027m talking about in https://etherpad.opendev.org/p/ironic-ptg-2026.1\n\nInstead of encoding tricks like this, I wonder if it\u0027s possible for us to inspect and ask the question \"can this be used as a boot device?\" and key on that, rather than specific types of objects?","commit_id":"fb926cb25c82ea9dc836acb7df1ce6d960a4d085"}]}
