[edpm_deploy_baremetal] Wait for ProvisionServer localImageUrl - #4136
[edpm_deploy_baremetal] Wait for ProvisionServer localImageUrl#4136rebtoor wants to merge 1 commit into
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
The role queried OpenStackProvisionServer status as soon as the deployment was Available, but the baremetal operator sets status.localImageUrl asynchronously. Retry k8s_info until the URL is present and use dict.get() when building checksum fallback facts. Seen in cifmw-crc-podified-edpm-baremetal-bootc while validating ci-framework#4131 (unrelated failure). Assisted-By: Cursor Co-authored-by: Cursor <cursoragent@cursor.com> Signed-off-by: Roberto Alfieri <ralfieri@redhat.com>
775b98b to
896aff8
Compare
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 13m 54s |
|
recheck |
|
This change depends on a change that failed to merge. Change #4130 is needed. |
3 participants
Summary
OpenStackProvisionServeruntilstatus.localImageUrlis set..get()access when deriving the checksum URL fallback.Problem
cifmw-crc-podified-edpm-baremetal-bootcfailed inedpm_deploy_baremetalwith
'dict object' has no attribute 'localImageUrl'even though theProvision Server pod and deployment were ready. The operator publishes
localImageUrlafter reconciliation, not when the Deployment becomesAvailable.
Test plan
cifmw-crc-podified-edpm-baremetal-bootcon rdoproject ZuulTEST_SINGLE_ROLE=edpm_deploy_baremetal make molecule(dry-run converge)Depends-On: #4130
Made with Cursor