try to fix prow

Signed-off-by: Daniel Grimm <[email protected]>

Signed-off-by: Daniel Grimm <[email protected]>

Author: dgn

tools/crd-schema-checker.sh

@@ -50,14 +50,59 @@ fi
 
 current_branch=$(git rev-parse --abbrev-ref HEAD)
 
-# prow uses shallow-clones
-prow=false
+# Store original working directory
+original_dir="$(pwd)"
+cleanup_clone=false
+
+# Enhanced prow shallow clone handling - use temporary full clone
 if git rev-parse --is-shallow-repository | grep -q true; then
-  git fetch --unshallow origin || git fetch origin '+refs/heads/*:refs/remotes/origin/*'
-  git fetch origin '+refs/heads/release-*:refs/remotes/origin/release-*' || true
-  prow=true
+    echo "Detected shallow clone (prow environment). Creating temporary full clone..."
+    
+    # Get the clone URL
+    clone_url=$(git remote get-url origin)
+    if [[ -z "$clone_url" ]]; then
+        echo "ERROR: Could not determine remote URL"
+        exit 1
+    fi
+    
+    # Create temporary directory for full clone
+    temp_clone_dir=$(mktemp -d)
+    cleanup_clone=true
+    
+    echo "Cloning full repository to $temp_clone_dir..."
+    if ! git clone "$clone_url" "$temp_clone_dir" --quiet; then
+        echo "ERROR: Failed to clone repository"
+        rm -rf "$temp_clone_dir"
+        exit 1
+    fi
+    
+    # Get current branch name to checkout the same branch in the clone
+    current_branch_local=$(git rev-parse --abbrev-ref HEAD)
+    
+    # Change to the cloned directory
+    cd "$temp_clone_dir"
+    
+    # Checkout the same branch if it exists
+    if git show-ref --verify --quiet "refs/remotes/origin/$current_branch_local"; then
+        git checkout "$current_branch_local" --quiet
+    else
+        echo "WARNING: Current branch '$current_branch_local' not found in full clone. Using default branch."
+    fi
+    
+    # Update current_branch in case we switched
+    current_branch=$(git rev-parse --abbrev-ref HEAD)
 fi
 
+# Setup cleanup trap
+cleanup() {
+    cd "$original_dir"
+    if [[ "$cleanup_clone" == "true" && -n "${temp_clone_dir:-}" ]]; then
+        echo "Cleaning up temporary clone..."
+        rm -rf "$temp_clone_dir"
+    fi
+}
+trap cleanup EXIT
+
 # Determine what to compare
 if [[ "$current_branch" =~ ^release-[0-9]+\.[0-9]+$ ]]; then
     # On release branch: compare against previous release
@@ -78,43 +123,21 @@ fi
 
 # Find previous release branch (only if we're on a release branch)
 if [[ "$current_branch" =~ ^release-[0-9]+\.[0-9]+$ ]]; then
-    echo "Looking for previous release branch before $target_branch..."
-    
-    # Debug: show available branches
-    echo "Available local branches:"
-    git branch -a | grep -E 'release-[0-9]+\.[0-9]+$' | head -5
-    
     previous_branch=$(git branch -a | grep -E 'release-[0-9]+\.[0-9]+$' | \
         sed 's/.*release-/release-/' | sort -V | uniq | \
         awk -v target="$target_branch" '
         $0 == target { print prev; exit }
         { prev = $0 }
         ')
-
-    # Fallback: try to find in remote branches if not found locally (prow fix)
-    if [[ -z "$previous_branch" && "$prow" == "true" ]]; then
-        echo "Fallback: checking remote branches..."
-        echo "Available remote branches:"
-        git branch -r | grep -E 'origin/release-[0-9]+\.[0-9]+$' | head -5
-        
-        previous_branch=$(git branch -r | grep -E 'origin/release-[0-9]+\.[0-9]+$' | \
-            sed 's|.*origin/||' | sort -V | uniq | \
-            awk -v target="$target_branch" '
-            $0 == target { print prev; exit }
-            { prev = $0 }
-            ')
-    fi
-    
-    echo "Found previous branch: '$previous_branch'"
 fi
 
-[[ -n "$previous_branch" ]] || {
-    echo "ERROR: No previous release branch found for target: $target_branch"
+# Handle case where no previous release branch exists
+if [[ -z "$previous_branch" ]]; then
+    echo "WARNING: No previous release branch found for target: $target_branch"
     echo "This might be expected for the first release in a series."
-    echo "Available branches:"
-    git branch -a | grep -E 'release-[0-9]+\.[0-9]+' | head -10
-    exit 1
-}
+    echo "Skipping CRD compatibility check."
+    exit 0
+fi
 
 echo "Checking CRD compatibility: $previous_branch -> $current_branch"
 
@@ -127,9 +150,7 @@ mkdir -p "$previous_dir" "$current_dir"
 
 extract_crds() {
     local branch="$1" output_dir="$2"
-    if [[ "${prow}" == "true" ]]; then
-      git fetch origin "${branch}" || true
-    fi
+    # No need to fetch branches in full clone
     while IFS= read -r file; do
         [[ -z "$file" ]] && continue
         content=$(git show "$branch:bundle/manifests/$file" 2>/dev/null)
@@ -167,10 +188,18 @@ for previous_crd in "${previous_crds[@]}"; do
 
     if [[ -n "${current_crd_map[$crd_name]:-}" ]]; then
         set +e
-        output=$($CRD_SCHEMA_CHECKER check-manifests \
-            --disabled-validators=NoBools,NoMaps \
-            --existing-crd-filename="$previous_crd" \
-            --new-crd-filename="${current_crd_map[$crd_name]}" 2>&1)
+        # Use the original directory's crd-schema-checker if we're in a temp clone
+        if [[ "$cleanup_clone" == "true" ]]; then
+            output=$("$original_dir/$CRD_SCHEMA_CHECKER" check-manifests \
+                --disabled-validators=NoBools,NoMaps \
+                --existing-crd-filename="$previous_crd" \
+                --new-crd-filename="${current_crd_map[$crd_name]}" 2>&1)
+        else
+            output=$($CRD_SCHEMA_CHECKER check-manifests \
+                --disabled-validators=NoBools,NoMaps \
+                --existing-crd-filename="$previous_crd" \
+                --new-crd-filename="${current_crd_map[$crd_name]}" 2>&1)
+        fi
         exit_code=$?
         set -e