From 3f0964ad385390f53516904219fbfc47b989d31f Mon Sep 17 00:00:00 2001 From: Amit Amrutiya Date: Wed, 14 Feb 2024 01:58:11 +0530 Subject: [PATCH] fix: unused variable failure warning in webhooks file (#916) Signed-off-by: amitamrutiya2210 Co-authored-by: Alex Jones --- pkg/analyzer/mutating_webhook.go | 5 +++++ pkg/analyzer/validating_webhook.go | 5 +++++ 2 files changed, 10 insertions(+) diff --git a/pkg/analyzer/mutating_webhook.go b/pkg/analyzer/mutating_webhook.go index c535c306d7..0360b796e2 100644 --- a/pkg/analyzer/mutating_webhook.go +++ b/pkg/analyzer/mutating_webhook.go @@ -75,6 +75,11 @@ func (MutatingWebhookAnalyzer) Analyze(a common.Analyzer) ([]common.Result, erro }, }, }) + preAnalysis[fmt.Sprintf("%s/%s", webhookConfig.Namespace, webhook.Name)] = common.PreAnalysis{ + MutatingWebhook: webhookConfig, + FailureDetails: failures, + } + AnalyzerErrorsMetric.WithLabelValues(kind, webhook.Name, webhookConfig.Namespace).Set(float64(len(failures))) continue } diff --git a/pkg/analyzer/validating_webhook.go b/pkg/analyzer/validating_webhook.go index 5080a69423..b29cc33974 100644 --- a/pkg/analyzer/validating_webhook.go +++ b/pkg/analyzer/validating_webhook.go @@ -73,6 +73,11 @@ func (ValidatingWebhookAnalyzer) Analyze(a common.Analyzer) ([]common.Result, er }, }, }) + preAnalysis[fmt.Sprintf("%s/%s", webhookConfig.Namespace, webhook.Name)] = common.PreAnalysis{ + ValidatingWebhook: webhookConfig, + FailureDetails: failures, + } + AnalyzerErrorsMetric.WithLabelValues(kind, webhook.Name, webhookConfig.Namespace).Set(float64(len(failures))) continue }