Bug 2088716

Summary: [NO-OP] [4.10] Redfish set boot device failed for node in OCP 4.8/4.9 latest RC
Product: OpenShift Container Platform Reporter: Jacob Anders <janders>
Component: Bare Metal Hardware ProvisioningAssignee: Jacob Anders <janders>
Bare Metal Hardware Provisioning sub component: ironic QA Contact: Amit Ugol <augol>
Status: CLOSED CURRENTRELEASE Docs Contact:
Severity: urgent    
Priority: urgent CC: augol, dtantsur, imelofer, kurathod, manrodri, mcornea, openshift-bugs-escalate, racedoro, rpittau, shajmakh, tsedovic
Version: 4.10   
Target Milestone: ---   
Target Release: 4.10.z   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Release Note
Doc Text:
Story Points: ---
Clone Of: 2088196 Environment:
Last Closed: 2022-05-24 08:23:25 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 2088717    
Bug Blocks: 2088319    

Comment 1 Jacob Anders 2022-05-20 09:13:26 UTC 
In https://bugzilla.redhat.com/show_bug.cgi?id=2088196#c7 myself and Manuel (reporter of BZ2088196) validated that the fix proposed to the Ironic/Wallaby branch https://review.opendev.org/c/openstack/sushy/+/842462 fixes the problem. This fix adds a missing backport of this change: https://review.opendev.org/c/openstack/sushy/+/818110

OCP 4.11 and 4.10 releases use Ironic Yoga release which already contains this code:
https://opendev.org/openstack/sushy/src/branch/stable/yoga/sushy/resources/base.py#L604

This means it is not prone to the issue affecting 4.8 and 4.9 (which use older Ironic branches) and hence does not require a fix.
Comment 4 Jacob Anders 2022-05-24 00:15:50 UTC 
Further with regards to verification of this BZ, please see Manuel's comment in the 4.8 BZ https://bugzilla.redhat.com/show_bug.cgi?id=2088196#c12 confirming OCP 4.10 and 4.11 were not affected by this BZ.
Comment 5 Jacob Anders 2022-05-24 08:23:25 UTC 
Closing with CURRENTRELEASE as this is a no-op BZ which is not to be included in Errata.