From 643179cdace2b192edf3b682a954ffd71332c47a Mon Sep 17 00:00:00 2001 From: Jan Chaloupka Date: Wed, 4 Feb 2026 11:29:06 +0100 Subject: [PATCH] feat(pkg/descheduler): setup prometheus client for the dry run mode descheduler as well The secret is used for read-only operations so it can be safely wired. --- pkg/descheduler/descheduler.go | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/pkg/descheduler/descheduler.go b/pkg/descheduler/descheduler.go index 1f0ae29c3..6ebe35f46 100644 --- a/pkg/descheduler/descheduler.go +++ b/pkg/descheduler/descheduler.go @@ -564,7 +564,7 @@ func bootstrapDescheduler( return nil, nil, fmt.Errorf("failed to create new descheduler: %v", err) } - // Setup Prometheus provider (only for real client case, not for dry run) + // Setup Prometheus provider if err := setupPrometheusProvider(descheduler, namespacedSharedInformerFactory); err != nil { return nil, nil, fmt.Errorf("failed to setup Prometheus provider: %v", err) } @@ -586,6 +586,11 @@ func bootstrapDescheduler( if err != nil { return nil, nil, fmt.Errorf("failed to create dry run descheduler: %v", err) } + + // Setup Prometheus provider (with the real shared informer factory as the secret is only read) + if err := setupPrometheusProvider(descheduler, namespacedSharedInformerFactory); err != nil { + return nil, nil, fmt.Errorf("failed to setup Prometheus provider for the dry run descheduler: %v", err) + } } // init is responsible for starting all informer factories, metrics providers