Skip to content

Commit 4b9a131

Browse files
MikeSpreitzerclaude
andcommitted
fix: wait for CRDs to be Established in OpenShift E2E workflow
Resolves #277. The previous check used `kubectl get crd` to verify CRD existence, which was redundant since a successful `kubectl apply` already guarantees that. What actually needs checking is that the CRDs become "Established", meaning the API servers have digested the definitions. Replace the existence checks with `kubectl wait --for=condition=Established` calls (60s timeout) for each of the three FMA CRDs. Signed-off-by: Mike Spreitzer <mspreitz@us.ibm.com> Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 7eb291a commit 4b9a131

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

.github/workflows/ci-e2e-openshift.yaml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -443,12 +443,12 @@ jobs:
443443
fi
444444
done
445445
446-
# Verify CRDs are registered
447-
echo "Verifying CRDs..."
448-
kubectl get crd inferenceserverconfigs.fma.llm-d.ai
449-
kubectl get crd launcherconfigs.fma.llm-d.ai
450-
kubectl get crd launcherpopulationpolicies.fma.llm-d.ai
451-
echo "All CRDs registered successfully"
446+
# Wait for CRDs to become Established (API servers have digested the definitions)
447+
echo "Waiting for CRDs to become Established..."
448+
kubectl wait --for=condition=Established crd/inferenceserverconfigs.fma.llm-d.ai --timeout=60s
449+
kubectl wait --for=condition=Established crd/launcherconfigs.fma.llm-d.ai --timeout=60s
450+
kubectl wait --for=condition=Established crd/launcherpopulationpolicies.fma.llm-d.ai --timeout=60s
451+
echo "All CRDs established"
452452
453453
- name: Create ConfigMaps
454454
run: |

0 commit comments

Comments
 (0)