Skip to content

Commit f17d1bb

Browse files
committed
move oc wait outside CSV detection loop to prevent excessive timeout
Signed-off-by: Matthias Wessendorf <[email protected]>
1 parent 282d704 commit f17d1bb

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

build/acm.mk

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,11 +40,12 @@ acm-mce-install:
4040
@for i in {1..60}; do \
4141
if oc get csv -n multicluster-engine -o name 2>/dev/null | grep -q multicluster-engine; then \
4242
echo "MCE CSV found, waiting for Succeeded phase..."; \
43-
oc wait --for=jsonpath='{.status.phase}'=Succeeded csv -l operators.coreos.com/multicluster-engine.multicluster-engine -n multicluster-engine --timeout=300s && break; \
43+
break; \
4444
fi; \
4545
echo " Waiting for MCE CSV to appear ($$i/60)..."; \
4646
sleep 5; \
4747
done
48+
@oc wait --for=jsonpath='{.status.phase}'=Succeeded csv -l operators.coreos.com/multicluster-engine.multicluster-engine -n multicluster-engine --timeout=300s
4849
@echo "Creating MultiClusterEngine instance..."
4950
@printf '%s\n' \
5051
'apiVersion: multicluster.openshift.io/v1' \
@@ -72,11 +73,12 @@ acm-operator-install:
7273
@for i in $$(seq 1 60); do \
7374
if oc get csv -n open-cluster-management -o name 2>/dev/null | grep -q advanced-cluster-management; then \
7475
echo "ACM CSV found, waiting for Succeeded phase..."; \
75-
oc wait --for=jsonpath='{.status.phase}'=Succeeded csv -l operators.coreos.com/advanced-cluster-management.open-cluster-management -n open-cluster-management --timeout=300s && break; \
76+
break; \
7677
fi; \
7778
echo " Waiting for ACM CSV to appear ($$i/60)..."; \
7879
sleep 5; \
7980
done
81+
@oc wait --for=jsonpath='{.status.phase}'=Succeeded csv -l operators.coreos.com/advanced-cluster-management.open-cluster-management -n open-cluster-management --timeout=300s
8082
@echo "✓ ACM Operator installation complete"
8183

8284
# Install ACM instance (MultiClusterHub CR)

0 commit comments

Comments
 (0)