Remove StartEventWatcher call

The StartEventWatcher call is no longer needed since calling
StartRecordingToSink already sets up a watcher and manages it via its context.

Signed-off-by: Borja Clemente <[email protected]>

Author: clebs

pkg/internal/recorder/recorder.go

@@ -23,7 +23,6 @@ import (
 	"sync"
 
 	"github.com/go-logr/logr"
-	eventsv1 "k8s.io/api/events/v1"
 	"k8s.io/apimachinery/pkg/runtime"
 	eventsv1client "k8s.io/client-go/kubernetes/typed/events/v1"
 	"k8s.io/client-go/rest"
@@ -96,17 +95,7 @@ func (p *Provider) getBroadcaster() events.EventBroadcaster {
 			p.broadcaster = events.NewBroadcaster(&events.EventSinkImpl{Interface: p.evtClient})
 		}
 		// TODO(clebs): figure out how to manage the context/channel that StartRecordingToSink needs inside the provider.
-		p.broadcaster.StartRecordingToSink(nil)
-
-		// TODO(clebs): figure out if we still need this and how the change would make sense.
-		p.broadcaster.StartEventWatcher(
-			func(obj runtime.Object) {
-				if e, ok := obj.(*eventsv1.Event); ok {
-					p.logger.V(1).Info(e.Note, "type", e.Type, "object", e.Regarding, "related", e.Related, "reason", e.Reason)
-				} else {
-					p.logger.V(1).Info("event watcher received an unsupported object type", "gvk", obj.GetObjectKind().GroupVersionKind().String())
-				}
-			})
+		p.broadcaster.StartRecordingToSinkWithContext(context.TODO())
 	})
 
 	return p.broadcaster