Merge pull request #731 from trustyai-explainability/fix/kserve-v17-standard-deployment-mode

fix(tas): handle KServe v0.17 Standard and KNative deployment modes

Author: sheltoncyril

controllers/tas/inference_services.go

@@ -18,7 +18,9 @@ import (
 const (
 	DEPLOYMENT_MODE_MODELMESH  = "ModelMesh"
 	DEPLOYMENT_MODE_RAW        = "RawDeployment"
+	DEPLOYMENT_MODE_STANDARD   = "Standard"
 	DEPLOYMENT_MODE_SERVERLESS = "Serverless"
+	DEPLOYMENT_MODE_KNATIVE    = "Knative"
 )
 
 // GetDeploymentsByLabel returns a list of Deployments that match a label key-value pair
@@ -243,17 +245,17 @@ func (r *TrustyAIServiceReconciler) handleInferenceServices(ctx context.Context,
 			}
 			continue
 
-		case DEPLOYMENT_MODE_RAW:
-			// Handle KServe Raw deployments
+		case DEPLOYMENT_MODE_RAW, DEPLOYMENT_MODE_STANDARD:
+			// Handle KServe Raw/Standard deployments (KServe v0.17+ renamed RawDeployment to Standard)
 			err := r.patchKServe(ctx, instance, infService, namespace, crName, remove, true)
 			if err != nil {
 				log.FromContext(ctx).Error(err, "Could not patch InferenceLogger for KServe Raw deployment")
 				return false, err
 			}
 			continue
 
-		default:
-			// Handle KServe Serverless deployments
+		case DEPLOYMENT_MODE_SERVERLESS, DEPLOYMENT_MODE_KNATIVE:
+			// Handle KServe Serverless/Knative deployments (KServe v0.17+ renamed Serverless to Knative)
 			if kServeServerlessEnabled {
 				err := r.patchKServe(ctx, instance, infService, namespace, crName, remove, false)
 				if err != nil {
@@ -262,6 +264,10 @@ func (r *TrustyAIServiceReconciler) handleInferenceServices(ctx context.Context,
 				}
 			}
 			continue
+
+		default:
+			log.FromContext(ctx).Info("Unknown deployment mode, skipping InferenceService", "deploymentMode", deploymentMode, "inferenceService", infService.Name)
+			continue
 		}
 	}
 	return true, nil