remove extra logs and fix test

Author: wonderflow

api/pkg/filtermanager/api_impl.go

@@ -215,10 +215,6 @@ func (cb *filterManagerCallbackHandler) GetCounterMetrics(pluginName, metricName
 		api.LogErrorf("metrics not exist or not initialized for plugin %s", pluginName)
 		return nil
 	}
-	api.LogInfo("[metrics] printing:")
-	for k, v := range cb.metrics {
-		api.LogInfof("[metrics] %s: %v", k, v)
-	}
 	writer, ok := cb.metrics[pluginName]
 	if !ok {
 		api.LogErrorf("metrics writer for plugin %s not found", pluginName)

api/pkg/filtermanager/config.go

@@ -20,6 +20,7 @@ import (
 	"fmt"
 	"reflect"
 	"sort"
+	"strings"
 	"sync"
 
 	xds "github.com/cncf/xds/go/xds/type/v3"
@@ -96,7 +97,6 @@ func (conf *filterManagerConfig) Merge(another *filterManagerConfig) *filterMana
 	}
 
 	// Pass LDS metrics writers to the merged config for golang filter to use at route level
-	capi.LogInfof("[metrics] merging http filter, filtermanager config: %+v", another.metricsWriters)
 	conf.metricsWriters = another.metricsWriters
 
 	// It's tough to do the data plane merge right. We don't use shallow copy, which may share
@@ -192,11 +192,13 @@ func (p *FilterManagerConfigParser) Parse(any *anypb.Any, callbacks capi.ConfigC
 		// If callbacks is not nil, it means this filter is configured in the LDS level.
 		// We need to initialize the metrics for all golang plugins here.
 		registers := plugins.GetMetricsDefinitions()
+		registeredPlugins := []string{}
 		for pluginName, register := range registers {
-			api.LogInfof("initializing metrics for plugin %s", pluginName)
+			api.LogInfof("registering metrics for golang plugin %s", pluginName)
 			metricsWriters[pluginName] = register(callbacks)
+			registeredPlugins = append(registeredPlugins, pluginName)
 		}
-		capi.LogInfof("[metrics] initialized http filter, filtermanager config: %+v", metricsWriters)
+		capi.LogInfof("metrics registered for plugins: [%s]", strings.Join(registeredPlugins, ", "))
 	}
 
 	// No configuration

api/pkg/filtermanager/filtermanager.go

@@ -152,7 +152,7 @@ func FilterManagerFactory(c interface{}, cb capi.FilterCallbackHandler) (streamF
 	}
 
 	fm.callbacks.FilterCallbackHandler = cb
-	capi.LogInfof("[metrics] filter manager metrics Writers %v", conf.metricsWriters)
+
 	fm.callbacks.metrics = conf.metricsWriters
 
 	canSkipMethods := fm.canSkipMethods

api/plugins/tests/pkg/envoy/capi.go

@@ -583,6 +583,13 @@ func (i *filterCallbackHandler) PluginState() api.PluginState {
 	return i.pluginState
 }
 
+func (i *filterCallbackHandler) GetCounterMetrics(pluginName, metricsName string) capi.CounterMetric {
+	return nil
+}
+func (i *filterCallbackHandler) GetGaugeMetrics(pluginName, metricsName string) capi.GaugeMetric {
+	return nil
+}
+
 func (i *filterCallbackHandler) WithLogArg(key string, value any) api.StreamFilterCallbacks {
 	return i
 }