Skip to content

Commit 8c8b18b

Browse files
committed
Remove unused trace log from LegacySyncHandler.
This line creates an info-level logger, writes a trace-level log to it, and discards the logger reference. This looks like cruft and is responsible for some heap churn. Signed-off-by: Ben Luddy <[email protected]>
1 parent 4ffa5c1 commit 8c8b18b

File tree

1 file changed

+0
-2
lines changed

1 file changed

+0
-2
lines changed

Diff for: pkg/lib/queueinformer/queueinformer.go

-2
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@ package queueinformer
22

33
import (
44
"context"
5-
"fmt"
65

76
"github.com/pkg/errors"
87
"github.com/sirupsen/logrus"
@@ -170,7 +169,6 @@ func (l LegacySyncHandler) ToSyncer() kubestate.Syncer {
170169
// ToSyncerWithDelete returns the Syncer equivalent of the given sync handler and delete function.
171170
func (l LegacySyncHandler) ToSyncerWithDelete(onDelete func(obj interface{})) kubestate.Syncer {
172171
var syncer kubestate.SyncFunc = func(ctx context.Context, event kubestate.ResourceEvent) error {
173-
logrus.New().WithField("event", fmt.Sprintf("%+v", event)).Trace("legacy syncer received event")
174172
switch event.Type() {
175173
case kubestate.ResourceDeleted:
176174
if onDelete != nil {

0 commit comments

Comments
 (0)