From a582d444c5c53f25d7172947c690b35cad2cc176 Mon Sep 17 00:00:00 2001 From: HOLLEVILLE Matthis <99146727+matthisholleville@users.noreply.github.com> Date: Mon, 10 Apr 2023 07:49:22 +0200 Subject: [PATCH] fix: use hpa namespace instead analyzer namespace (#230) Signed-off-by: Matthis Holleville --- pkg/analyzer/hpa.go | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/pkg/analyzer/hpa.go b/pkg/analyzer/hpa.go index 6f22a44283..70a3666a5c 100644 --- a/pkg/analyzer/hpa.go +++ b/pkg/analyzer/hpa.go @@ -2,6 +2,7 @@ package analyzer import ( "fmt" + "github.com/k8sgpt-ai/k8sgpt/pkg/util" metav1 "k8s.io/apimachinery/pkg/apis/meta/v1" ) @@ -26,22 +27,22 @@ func (HpaAnalyzer) Analyze(a Analyzer) ([]Result, error) { switch scaleTargetRef.Kind { case "Deployment": - _, err := a.Client.GetClient().AppsV1().Deployments(a.Namespace).Get(a.Context, scaleTargetRef.Name, metav1.GetOptions{}) + _, err := a.Client.GetClient().AppsV1().Deployments(hpa.Namespace).Get(a.Context, scaleTargetRef.Name, metav1.GetOptions{}) if err != nil { scaleTargetRefNotFound = true } case "ReplicationController": - _, err := a.Client.GetClient().CoreV1().ReplicationControllers(a.Namespace).Get(a.Context, scaleTargetRef.Name, metav1.GetOptions{}) + _, err := a.Client.GetClient().CoreV1().ReplicationControllers(hpa.Namespace).Get(a.Context, scaleTargetRef.Name, metav1.GetOptions{}) if err != nil { scaleTargetRefNotFound = true } case "ReplicaSet": - _, err := a.Client.GetClient().AppsV1().ReplicaSets(a.Namespace).Get(a.Context, scaleTargetRef.Name, metav1.GetOptions{}) + _, err := a.Client.GetClient().AppsV1().ReplicaSets(hpa.Namespace).Get(a.Context, scaleTargetRef.Name, metav1.GetOptions{}) if err != nil { scaleTargetRefNotFound = true } case "StatefulSet": - _, err := a.Client.GetClient().AppsV1().StatefulSets(a.Namespace).Get(a.Context, scaleTargetRef.Name, metav1.GetOptions{}) + _, err := a.Client.GetClient().AppsV1().StatefulSets(hpa.Namespace).Get(a.Context, scaleTargetRef.Name, metav1.GetOptions{}) if err != nil { scaleTargetRefNotFound = true }