Merge pull request #28 from Mellanox/fix-common-ports

Disable metrics and health checks ports in config daemon
Mellanox · Oct 16, 2024 · b1f64b3 · b1f64b3
2 parents 6e994db + ebd9c01
commit b1f64b3
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/cmd/nic-configuration-daemon/main.go b/cmd/nic-configuration-daemon/main.go
@@ -12,6 +12,7 @@ import (
 	ctrl "sigs.k8s.io/controller-runtime"
 	"sigs.k8s.io/controller-runtime/pkg/client"
 	"sigs.k8s.io/controller-runtime/pkg/log"
+	metricsserver "sigs.k8s.io/controller-runtime/pkg/metrics/server"
 
 	"github.com/Mellanox/nic-configuration-operator/api/v1alpha1"
 	"github.com/Mellanox/nic-configuration-operator/internal/controller"
@@ -36,6 +37,9 @@ func main() {
 
 	mgr, err := ctrl.NewManager(ctrl.GetConfigOrDie(), ctrl.Options{
 		Scheme: scheme,
+		// Setting bind address to 0 disables the health probe / metrics server
+		HealthProbeBindAddress: "0",
+		Metrics:                metricsserver.Options{BindAddress: "0"},
 	})
 	if err != nil {
 		log.Log.Error(err, "unable to create manager")