Add validation step to verify /mnt storage and symlink configuration

Signed-off-by: Liav Weiss <[email protected]>

Author: Liav Weiss

.github/workflows/test-and-build.yml

@@ -145,6 +145,49 @@ jobs:
           HF_HUB_DISABLE_TELEMETRY: 1
         run: make download-models
 
+      - name: Validate /mnt storage and symlink
+        run: |
+          set -e
+          echo "=== Validating /mnt storage configuration ==="
+          
+          # Verify symlink
+          if [ ! -L "models" ] || [ "$(readlink models)" != "/mnt/models" ]; then
+            echo "ERROR: Symlink 'models' -> '/mnt/models' is not configured correctly"
+            exit 1
+          fi
+          echo "✓ Symlink configured: models -> /mnt/models"
+          
+          # Verify models accessible - check both symlink and direct path
+          echo "Checking models via symlink:"
+          ls -la models/ | head -20 || echo "No models found via symlink"
+          MODEL_COUNT=$(find models -mindepth 1 -maxdepth 1 -type d 2>/dev/null | wc -l)
+          echo "✓ Models accessible via symlink: $MODEL_COUNT directories found"
+          
+          echo "Checking models directly in /mnt/models:"
+          ls -la /mnt/models/ | head -20 || echo "No models found in /mnt/models"
+          DIRECT_COUNT=$(find /mnt/models -mindepth 1 -maxdepth 1 -type d 2>/dev/null | wc -l)
+          echo "✓ Models in /mnt/models: $DIRECT_COUNT directories found"
+          
+          # Verify test path resolution
+          cd src/semantic-router/pkg/classification
+          if [ ! -d "../../../../models" ]; then
+            echo "ERROR: Test path '../../../../models' not accessible"
+            exit 1
+          fi
+          RESOLVED=$(readlink -f ../../../../models 2>/dev/null || echo "N/A")
+          echo "✓ Test path resolves: ../../../../models -> $RESOLVED"
+          
+          # Verify models are accessible from test path
+          echo "Checking models from test path:"
+          ls -la ../../../../models/ | head -20 || echo "No models found from test path"
+          TEST_PATH_COUNT=$(find ../../../../models -mindepth 1 -maxdepth 1 -type d 2>/dev/null | wc -l)
+          echo "✓ Models accessible from test path: $TEST_PATH_COUNT directories found"
+          cd - > /dev/null
+          
+          # Disk usage
+          echo "✓ Disk usage: /mnt $(df -h /mnt | tail -1 | awk '{print $4 " available"}')"
+          echo "=== Validation complete ==="
+
       - name: Start Milvus service
         run: |
           echo "Starting Milvus vector database..."

src/semantic-router/pkg/classification/classifier_test.go

@@ -6,6 +6,7 @@ import (
 	"fmt"
 	"os"
 	"path/filepath"
+	"runtime"
 	"strings"
 	"sync"
 	"testing"
@@ -2091,10 +2092,21 @@ func TestAutoDiscoverModels_RealModels(t *testing.T) {
 
 // TestAutoInitializeUnifiedClassifier tests the full initialization process
 func TestAutoInitializeUnifiedClassifier(t *testing.T) {
-	// Test with real models directory
-	classifier, err := AutoInitializeUnifiedClassifier(testModelsDir)
+	// Resolve models directory relative to this test file's location
+	// The test runs from src/semantic-router, but the path is relative to the test file
+	_, filename, _, ok := runtime.Caller(0)
+	modelsDir := testModelsDir
+	if ok {
+		testDir := filepath.Dir(filename)
+		resolved, err := filepath.Abs(filepath.Join(testDir, "../../../../models"))
+		if err == nil {
+			modelsDir = resolved
+		}
+	}
+
+	classifier, err := AutoInitializeUnifiedClassifier(modelsDir)
 	if err != nil {
-		t.Fatalf("AutoInitializeUnifiedClassifier() failed: %v (models directory should exist at %s)", err, testModelsDir)
+		t.Fatalf("AutoInitializeUnifiedClassifier() failed: %v (models directory should exist at %s)", err, modelsDir)
 	}
 
 	if classifier == nil {