logger: fix default event subscription mask.

Let the stub determine which events the logger plugin subscribes
to. This should fix startup errors about trying to subscribe to
{Post,}UpdatePodSandbox events without handlers. Once the runtimes
start delivering those events we can add handlers for them and
that will get us subscribed to them automatically.

Signed-off-by: Krisztian Litkey <[email protected]>

Author: klihub

plugins/logger/nri-logger.go

@@ -215,7 +215,7 @@ func main() {
 
 	flag.StringVar(&pluginName, "name", "", "plugin name to register to NRI")
 	flag.StringVar(&pluginIdx, "idx", "", "plugin index to register to NRI")
-	flag.StringVar(&events, "events", "all", "comma-separated list of events to subscribe for")
+	flag.StringVar(&events, "events", "", "comma-separated list of events to subscribe for")
 	flag.StringVar(&cfg.LogFile, "log-file", "", "logfile name, if logging to a file")
 	flag.StringVar(&cfg.AddAnnotation, "add-annotation", "", "add this annotation to containers")
 	flag.StringVar(&cfg.SetAnnotation, "set-annotation", "", "set this annotation on containers")
@@ -239,8 +239,10 @@ func main() {
 	}
 
 	p := &plugin{}
-	if p.mask, err = api.ParseEventMask(events); err != nil {
-		log.Fatalf("failed to parse events: %v", err)
+	if events != "" {
+		if p.mask, err = api.ParseEventMask(events); err != nil {
+			log.Fatalf("failed to parse events: %v", err)
+		}
 	}
 	cfg.Events = strings.Split(events, ",")