[cinder_multiattach] Wait for Glance reconciliation after config patch#4060
[cinder_multiattach] Wait for Glance reconciliation after config patch#4060fmount 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 |
01cbd8f to
e43ecfe
Compare
The oc wait --for=condition=Ready on the controlplane returns immediately when the condition is already true, before the StatefulSet rolling update triggered by the config change completes. This causes a race with subsequent hooks (e.g. amphora image upload) that depend on Glance being stable. Replace the inline oc wait with two explicit steps: 1. Poll the Glance CR until customServiceConfig reflects the change 2. Wait for the controlplane Ready condition after reconciliation starts Closes: OSPCIX-1454 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Francesco Pantano <fpantano@redhat.com>
e43ecfe to
3330406
Compare
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 44m 33s |
|
The failure in the precommit job looks legit, but please wait before updating this as I'm testing it |
2 participants
The
oc wait --for=condition=Readyon thecontrolplanereturns immediately when the condition is already true, before theStatefulSetrolling update triggered by the config change completes.This causes a race with subsequent hooks (e.g. amphora image upload) that depend on
Glancebeing stable.This patch replaces the inline
oc waitwith two explicit steps:Glance CRuntilcustomServiceConfigreflects the change (oc applyhappened and therolloutstarted)controlplaneReadycondition after reconciliation starts (which is what we had previously)Closes: OSPCIX-1454