fix: cannot use ingress when disabling gateway API (#388)

Author: AlinsRan

internal/controller/httproute_controller.go

@@ -155,7 +155,7 @@ func (r *HTTPRouteReconciler) Reconcile(ctx context.Context, req ctrl.Request) (
 	if err := r.Get(ctx, req.NamespacedName, hr); err != nil {
 		if client.IgnoreNotFound(err) == nil {
 			if err := r.updateHTTPRoutePolicyStatusOnDeleting(ctx, req.NamespacedName); err != nil {
-				return ctrl.Result{}, err
+				r.Log.Error(err, "failed to update HTTPRoutePolicy status on HTTPRoute deleting", "httproute", req.NamespacedName)
 			}
 			hr.Namespace = req.Namespace
 			hr.Name = req.Name

internal/controller/indexer/indexer.go

@@ -88,6 +88,7 @@ func SetupIndexer(mgr ctrl.Manager) error {
 		&networkingv1.IngressClass{}:      setupIngressClassIndexer,
 		&networkingv1beta1.IngressClass{}: setupIngressClassV1beta1Indexer,
 		&v1alpha1.BackendTrafficPolicy{}:  setupBackendTrafficPolicyIndexer,
+		&v1alpha1.HTTPRoutePolicy{}:       setHTTPRoutePolicyIndexer,
 	} {
 		if utils.HasAPIResource(mgr, resource) {
 			if err := setup(mgr); err != nil {
@@ -275,7 +276,10 @@ func setupHTTPRouteIndexer(mgr ctrl.Manager) error {
 	); err != nil {
 		return err
 	}
+	return nil
+}
 
+func setHTTPRoutePolicyIndexer(mgr ctrl.Manager) error {
 	if err := mgr.GetFieldIndexer().IndexField(
 		context.Background(),
 		&v1alpha1.HTTPRoutePolicy{},
@@ -284,7 +288,6 @@ func setupHTTPRouteIndexer(mgr ctrl.Manager) error {
 	); err != nil {
 		return err
 	}
-
 	return nil
 }
 

internal/controller/ingress_controller.go

@@ -143,7 +143,7 @@ func (r *IngressReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ct
 	if err := r.Get(ctx, req.NamespacedName, ingress); err != nil {
 		if client.IgnoreNotFound(err) == nil {
 			if err := r.updateHTTPRoutePolicyStatusOnDeleting(ctx, req.NamespacedName); err != nil {
-				return ctrl.Result{}, err
+				r.Log.Error(err, "failed to update HTTPRoutePolicy status on Ingress deleting", "ingress", req.NamespacedName)
 			}
 
 			// Ingress was deleted, clean up corresponding resources
@@ -212,7 +212,6 @@ func (r *IngressReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ct
 	// process HTTPRoutePolicy
 	if err := r.processHTTPRoutePolicies(tctx, ingress); err != nil {
 		r.Log.Error(err, "failed to process HTTPRoutePolicy", "ingress", ingress.Name)
-		return ctrl.Result{}, err
 	}
 
 	ProcessBackendTrafficPolicy(r.Client, r.Log, tctx)