Skip to content

Commit e6412f4

Browse files
committed
fix test
1 parent cf7bf77 commit e6412f4

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

internal/controller/apisixconsumer_controller.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ func (r *ApisixConsumerReconciler) Reconcile(ctx context.Context, req ctrl.Reque
9696
"error", err.Error())
9797
return ctrl.Result{}, nil
9898
}
99-
defer r.updateStatus(ac, err)
99+
defer func() { r.updateStatus(ac, err) }()
100100

101101
if err = ProcessIngressClassParameters(tctx, r.Client, r.Log, ac, ingressClass); err != nil {
102102
r.Log.Error(err, "failed to process IngressClass parameters", "ingressClass", ingressClass.Name)

internal/controller/apisixroute_controller.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -157,13 +157,13 @@ func (r *ApisixRouteReconciler) Reconcile(ctx context.Context, req ctrl.Request)
157157
"error", err.Error())
158158
return ctrl.Result{}, nil
159159
}
160-
defer r.updateStatus(&ar, err)
160+
defer func() { r.updateStatus(&ar, err) }()
161161

162162
if err = ProcessIngressClassParameters(tctx, r.Client, r.Log, &ar, ic); err != nil {
163163
return ctrl.Result{}, err
164164
}
165165
if err = r.processApisixRoute(ctx, tctx, &ar); err != nil {
166-
return ctrl.Result{}, nil
166+
return ctrl.Result{}, err
167167
}
168168
if err = r.Provider.Update(ctx, tctx, &ar); err != nil {
169169
err = types.ReasonError{

0 commit comments

Comments
 (0)