Skip to content

Commit 40c4e5c

Browse files
committed
fluentbit: do not deepcopy inner variable, simply pull it into the iteration scope
Signed-off-by: Peter Wilcsinszky <[email protected]>
1 parent b3f9b8c commit 40c4e5c

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

controllers/logging/logging_controller.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -206,8 +206,8 @@ func (r *LoggingReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ct
206206
return ctrl.Result{}, errors.New("fluentbit has to be removed from the logging resource before the new FluentbitAgent can be reconciled")
207207
}
208208
for _, f := range loggingResources.Fluentbits {
209-
f := f.DeepCopy()
210-
nameProvider := loggingv1beta1.NewStandaloneFluentbitNameProvider(f)
209+
f := f
210+
nameProvider := loggingv1beta1.NewStandaloneFluentbitNameProvider(&f)
211211
reconcilers = append(reconcilers, fluentbit.New(
212212
r.Client,
213213
r.Log.WithName(nameProvider.ComponentName("fluentbit-agent")),

0 commit comments

Comments
 (0)