Merge pull request #1372 from yp969803/issue1366

fix: enableConnMetric knob at monitoring handler

Author: kmesh-bot

pkg/status/status_server.go

@@ -253,17 +253,24 @@ func (s *Server) monitoringHandler(w http.ResponseWriter, r *http.Request) {
 		_, _ = w.Write([]byte(fmt.Sprintf("invalid monitoring enable=%s", info)))
 		return
 	}
-	var enableMonitoring uint32
+	enableMonitoring := constants.DISABLED
 	if enabled {
 		enableMonitoring = constants.ENABLED
-	} else {
-		enableMonitoring = constants.DISABLED
 	}
 	if err := s.loader.UpdateEnableMonitoring(enableMonitoring); err != nil {
 		http.Error(w, fmt.Sprintf("update bpf monitoring failed: %v", err), http.StatusBadRequest)
 		return
 	}
 
+	enablePeriodicReport := constants.DISABLED
+	if enabled {
+		enablePeriodicReport = constants.ENABLED
+	}
+	if err := s.loader.UpdateEnablePeriodicReport(enablePeriodicReport); err != nil {
+		http.Error(w, fmt.Sprintf("update enable periodic report failed: %v", err), http.StatusBadRequest)
+		return
+	}
+
 	s.xdsClient.WorkloadController.SetMonitoringTrigger(enabled)
 	s.xdsClient.WorkloadController.SetAccesslogTrigger(enabled)
 	s.xdsClient.WorkloadController.SetWorkloadMetricTrigger(enabled)