CNI Telemetry enabled on CNS

Azure · Nov 1, 2023 · 8f2cedf · 8f2cedf
1 parent b3959bc
commit 8f2cedf
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/cns/service/main.go b/cns/service/main.go
@@ -305,7 +305,7 @@ var args = acn.ArgumentList{
 		Shorthand:    acn.OptTelemetryServiceAlias,
 		Description:  "Flag to start telemetry service to receive telemetry events from CNI. Default, disabled.",
 		Type:         "bool",
-		DefaultValue: true,
+		DefaultValue: false,
 	},
 	{
 		Name:         acn.OptCNIConflistFilepath,
@@ -621,8 +621,8 @@ func main() {
 			logger.InitAI(aiConfig, ts.DisableTrace, ts.DisableMetric, ts.DisableEvent)
 		}
 	}
-	telemetryDaemonEnabled = disableTelemetry
-	if !telemetryDaemonEnabled {
+	if telemetryDaemonEnabled {
+		log.Printf("CNI Telemtry is enabled")
 		go startTelemetryService(rootCtx)
 	}
 

diff --git a/network/manager.go b/network/manager.go
@@ -128,8 +128,10 @@ func (nm *networkManager) Initialize(config *common.PluginConfig, isRehydrationR
 	nm.Version = config.Version
 	nm.store = config.Store
 	if config.Stateless {
-		err := nm.SetStatelessCNIMode()
-		return errors.Wrapf(err, "Failed to initialize stateles CNI")
+		if err := nm.SetStatelessCNIMode(); err != nil {
+			return errors.Wrapf(err, "Failed to initialize stateles CNI")
+		}
+		return nil
 	}
 
 	// Restore persisted state.