fix: reconfigure pac if github throws error

- re-annotate the component to let it retry onboarding

Signed-off-by: Scott Hebert <[email protected]>

Author: scoheb

integration-tests/lib/test-functions.sh

@@ -239,21 +239,55 @@ create_kubernetes_resources() {
 # Modifies global variables: component_pr, pr_number
 # Relies on global variables: component_name, tenant_namespace
 wait_for_component_initialization() {
-    echo -n "Waiting for component ${component_name} in namespace ${tenant_namespace} to be initialized: "
+    echo "Waiting for component ${component_name} in namespace ${tenant_namespace} to be initialized..."
+
+    local max_attempts=60  # 10 minutes with 10-second intervals
+    local attempt=1
     local component_annotations=""
-    while [ -z "${component_annotations}" ]; do
-      sleep 1
-      echo -n "."
+    local initialization_success=false
+
+    while [ $attempt -le $max_attempts ]; do
+      echo "Initialization check attempt ${attempt}/${max_attempts}..."
+
+      # Try to get component annotations
       component_annotations=$(kubectl get component/"${component_name}" -n "${tenant_namespace}" -ojson 2>/dev/null | \
         jq -r --arg k "build.appstudio.openshift.io/status" '.metadata.annotations[$k] // ""')
+
+      if [ -n "${component_annotations}" ]; then
+        echo "✅ Component initialized successfully"
+
+        # component_pr is made global by not declaring it local
+        component_pr=$(jq -r '.pac."merge-url" // ""' <<< "${component_annotations}")
+        if [ -n "${component_pr}" ]; then
+            initialization_success=true
+            break
+        else
+            log_warning "Could not get component PR from annotations: ${component_annotations}"
+            echo "Waiting 10 seconds before retry..."
+            sleep 10
+            echo "Requesting a new configure-pac..."
+            kubectl annotate components/${component_name} build.appstudio.openshift.io/request=configure-pac -n "${tenant_namespace}"
+        fi
+
+      else
+        echo "⚠️  Component not yet initialized (attempt ${attempt}/${max_attempts})"
+
+        # Wait before retrying (except on the last attempt)
+        if [ $attempt -lt $max_attempts ]; then
+          echo "Waiting 10 seconds before retry..."
+          sleep 10
+        fi
+      fi
+
+      attempt=$((attempt + 1))
     done
-    echo ""
-    echo "️✅️ Initialized."
 
-    # component_pr is made global by not declaring it local
-    component_pr=$(jq -r '.pac."merge-url" // ""' <<< "${component_annotations}")
-    if [ -z "${component_pr}" ]; then
-      log_error "Could not get component PR from annotations: ${component_annotations}"
+    # Check if initialization ultimately succeeded
+    if [ "$initialization_success" = false ]; then
+      echo "🔴 error: component ${component_name} failed to initialize after ${max_attempts} attempts ($(($max_attempts * 10 / 60)) minutes)"
+      echo "   - Component may not exist in namespace ${tenant_namespace}"
+      echo "   - Component creation may have failed"
+      exit 1
     fi
 
     # pr_number is made global by not declaring it local
@@ -286,7 +320,7 @@ merge_github_pr() {
     # Retry loop for PR merge
     while [ $attempt -le $max_attempts ] && [ "$success" = false ]; do
         echo "Merge attempt ${attempt}/${max_attempts}..."
-        
+
         set +e
         merge_result=$(curl -L \
           -X PUT \

tasks/managed/collect-data/collect-data.yaml

@@ -6,6 +6,7 @@ metadata:
   annotations:
     tekton.dev/pipelines.minVersion: "0.12.1"
     tekton.dev/tags: release
+    test: "scott"
 spec:
   description: >-
     Tekton task to collect data from release resources and verify that they do not contain any