From 4a29c4bb495aed1e6d192a692b4fed41330fdc78 Mon Sep 17 00:00:00 2001 From: Yossi Ovadia Date: Fri, 10 Oct 2025 10:04:09 -0700 Subject: [PATCH] fix: remove log tail limit in validation script for model loading detection MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The validation script was failing to detect successful model loading because it used --tail=200 to check container logs. Since model containers generate many health check logs after startup, the "model loaded" messages were beyond the 200-line limit. Changes: - Remove --tail=200 from model-a validation (line 98) - Remove --tail=200 from model-b validation (line 110) - Now scans entire log history for "model loaded" message Fixes #391 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Signed-off-by: Yossi Ovadia --- deploy/openshift/validate-deployment.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/deploy/openshift/validate-deployment.sh b/deploy/openshift/validate-deployment.sh index 234bbc0b..d0cdf382 100755 --- a/deploy/openshift/validate-deployment.sh +++ b/deploy/openshift/validate-deployment.sh @@ -95,7 +95,7 @@ fi # Test 6: Check model-a loaded successfully log "INFO" "Test 6: Checking model-a loaded successfully..." if [[ -n "$POD_NAME" ]]; then - MODEL_STATUS=$(oc logs -n "$NAMESPACE" "$POD_NAME" -c model-a --tail=200 | grep -i "model loaded" | tail -1) + MODEL_STATUS=$(oc logs -n "$NAMESPACE" "$POD_NAME" -c model-a | grep -i "model loaded" | tail -1) if [[ -n "$MODEL_STATUS" ]]; then DEVICE=$(echo "$MODEL_STATUS" | grep -oE "(cuda|cpu)" || echo "unknown") log "PASS" "Model-A loaded successfully on $DEVICE" @@ -107,7 +107,7 @@ fi # Test 7: Check model-b loaded successfully log "INFO" "Test 7: Checking model-b loaded successfully..." if [[ -n "$POD_NAME" ]]; then - MODEL_STATUS=$(oc logs -n "$NAMESPACE" "$POD_NAME" -c model-b --tail=200 | grep -i "model loaded" | tail -1) + MODEL_STATUS=$(oc logs -n "$NAMESPACE" "$POD_NAME" -c model-b | grep -i "model loaded" | tail -1) if [[ -n "$MODEL_STATUS" ]]; then DEVICE=$(echo "$MODEL_STATUS" | grep -oE "(cuda|cpu)" || echo "unknown") log "PASS" "Model-B loaded successfully on $DEVICE"