Merge pull request 'fix(ci): broaden graceful-exit on infra PR auto-merge (CAR-1438) [dev→uat]' (#48) from dev into uat
CI / build-and-push (push) Successful in 9s
CI / deploy-uat (push) Successful in 8s
CI / deploy-dev (push) Has been skipped
CI / build-and-push (pull_request) Has been skipped
CI / deploy-dev (pull_request) Has been skipped
CI / deploy-uat (pull_request) Has been skipped
CI / build-and-push (push) Successful in 9s
CI / deploy-uat (push) Successful in 8s
CI / deploy-dev (push) Has been skipped
CI / build-and-push (pull_request) Has been skipped
CI / deploy-dev (pull_request) Has been skipped
CI / deploy-uat (pull_request) Has been skipped
fix(ci): broaden graceful-exit on infra PR auto-merge (CAR-1438) [dev→uat] QA PASS Charlie review #4825. Promotes CAR-1436+CAR-1438 fixes to UAT.
This commit was merged in pull request #48.
This commit is contained in:
+9
-10
@@ -177,12 +177,11 @@ jobs:
|
|||||||
MERGED=$(echo "$MERGE_RESP" | jq -r '.merged // false')
|
MERGED=$(echo "$MERGE_RESP" | jq -r '.merged // false')
|
||||||
if [ "$MERGED" = "true" ]; then
|
if [ "$MERGED" = "true" ]; then
|
||||||
echo "PR #${PR_NUM} merged into cartsnitch/infra dev"
|
echo "PR #${PR_NUM} merged into cartsnitch/infra dev"
|
||||||
elif echo "$MERGE_RESP" | grep -qi 'does not have enough approvals'; then
|
|
||||||
echo "::notice::infra PR #${PR_NUM} opened and awaiting CTO (cs_savannah) approve+merge — GitOps approval gate, not a failure"
|
|
||||||
exit 0
|
|
||||||
else
|
else
|
||||||
echo "::error::Auto-merge of cartsnitch/infra PR #${PR_NUM} failed: $MERGE_RESP"
|
# CAR-1438: PR opened successfully; any non-merged outcome (empty body,
|
||||||
exit 1
|
# approval-gate, pending checks) is the GitOps gate — not a failure.
|
||||||
|
echo "::notice::infra PR #${PR_NUM} opened — auto-merge not available (${MERGE_RESP:-empty response}); awaiting CTO (cs_savannah) approve+merge"
|
||||||
|
exit 0
|
||||||
fi
|
fi
|
||||||
|
|
||||||
deploy-uat:
|
deploy-uat:
|
||||||
@@ -275,10 +274,10 @@ jobs:
|
|||||||
MERGED=$(echo "$MERGE_RESP" | jq -r '.merged // false')
|
MERGED=$(echo "$MERGE_RESP" | jq -r '.merged // false')
|
||||||
if [ "$MERGED" = "true" ]; then
|
if [ "$MERGED" = "true" ]; then
|
||||||
echo "PR #${PR_NUM} merged into cartsnitch/infra uat"
|
echo "PR #${PR_NUM} merged into cartsnitch/infra uat"
|
||||||
elif echo "$MERGE_RESP" | grep -qi 'does not have enough approvals'; then
|
|
||||||
echo "::notice::infra PR #${PR_NUM} opened and awaiting CTO (cs_savannah) approve+merge — GitOps approval gate, not a failure"
|
|
||||||
exit 0
|
|
||||||
else
|
else
|
||||||
echo "::error::Auto-merge of cartsnitch/infra PR #${PR_NUM} failed: $MERGE_RESP"
|
# CAR-1438: PR opened successfully; any non-merged outcome (empty body,
|
||||||
exit 1
|
# approval-gate, pending checks) is the GitOps gate — not a failure.
|
||||||
|
echo "::notice::infra PR #${PR_NUM} opened — auto-merge not available (${MERGE_RESP:-empty response}); awaiting CTO (cs_savannah) approve+merge"
|
||||||
|
exit 0
|
||||||
fi
|
fi
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user