mirror of
https://github.com/k8sgpt-ai/k8sgpt.git
synced 2025-05-06 23:26:35 +00:00
* feat: fix the custom-analysis printing (#1195) Signed-off-by: Alex Jones <alexsimonjones@gmail.com> Signed-off-by: JuHyung-Son <sonju0427@gmail.com> * feat: add label selector Signed-off-by: JuHyung-Son <sonju0427@gmail.com> * feat: add label selector on analyzers Signed-off-by: JuHyung-Son <sonju0427@gmail.com> * chore(deps): pin goreleaser/goreleaser-action action to 286f3b1 (#1171) Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Signed-off-by: JuHyung-Son <sonju0427@gmail.com> * fix(deps): update module buf.build/gen/go/k8sgpt-ai/k8sgpt/grpc/go to v1.4.0-20240715142657-3785f0a44aae.2 (#1196) Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Signed-off-by: JuHyung-Son <sonju0427@gmail.com> * chore(deps): update actions/upload-artifact digest to 0b2256b (#1175) Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Signed-off-by: JuHyung-Son <sonju0427@gmail.com> * chore: update proto pkg version Signed-off-by: JuHyung-Son <sonju0427@gmail.com> * chore: fix typo Signed-off-by: JuHyung-Son <sonju0427@gmail.com> * feat: add label string to LabelSelector util func Signed-off-by: JuHyung-Son <sonju0427@gmail.com> * feat: add test using 2 label selector Signed-off-by: JuHyung-Son <sonju0427@gmail.com> --------- Signed-off-by: Alex Jones <alexsimonjones@gmail.com> Signed-off-by: JuHyung-Son <sonju0427@gmail.com> Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Signed-off-by: JuHyung Son <sonju0427@gmail.com> Co-authored-by: Alex Jones <alexsimonjones@gmail.com> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
175 lines
5.0 KiB
Go
175 lines
5.0 KiB
Go
/*
|
|
Copyright 2023 The K8sGPT Authors.
|
|
Licensed under the Apache License, Version 2.0 (the "License");
|
|
you may not use this file except in compliance with the License.
|
|
You may obtain a copy of the License at
|
|
http://www.apache.org/licenses/LICENSE-2.0
|
|
Unless required by applicable law or agreed to in writing, software
|
|
distributed under the License is distributed on an "AS IS" BASIS,
|
|
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
See the License for the specific language governing permissions and
|
|
limitations under the License.
|
|
*/
|
|
|
|
package analyzer
|
|
|
|
import (
|
|
"fmt"
|
|
|
|
"github.com/k8sgpt-ai/k8sgpt/pkg/common"
|
|
"github.com/k8sgpt-ai/k8sgpt/pkg/kubernetes"
|
|
"github.com/k8sgpt-ai/k8sgpt/pkg/util"
|
|
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
|
|
"k8s.io/apimachinery/pkg/runtime/schema"
|
|
)
|
|
|
|
type IngressAnalyzer struct{}
|
|
|
|
func (IngressAnalyzer) Analyze(a common.Analyzer) ([]common.Result, error) {
|
|
|
|
kind := "Ingress"
|
|
apiDoc := kubernetes.K8sApiReference{
|
|
Kind: kind,
|
|
ApiVersion: schema.GroupVersion{
|
|
Group: "networking",
|
|
Version: "v1",
|
|
},
|
|
OpenapiSchema: a.OpenapiSchema,
|
|
}
|
|
|
|
AnalyzerErrorsMetric.DeletePartialMatch(map[string]string{
|
|
"analyzer_name": kind,
|
|
})
|
|
|
|
list, err := a.Client.GetClient().NetworkingV1().Ingresses(a.Namespace).List(a.Context, metav1.ListOptions{LabelSelector: a.LabelSelector})
|
|
if err != nil {
|
|
return nil, err
|
|
}
|
|
|
|
var preAnalysis = map[string]common.PreAnalysis{}
|
|
|
|
for _, ing := range list.Items {
|
|
var failures []common.Failure
|
|
|
|
// get ingressClassName
|
|
ingressClassName := ing.Spec.IngressClassName
|
|
if ingressClassName == nil {
|
|
ingClassValue := ing.Annotations["kubernetes.io/ingress.class"]
|
|
if ingClassValue == "" {
|
|
doc := apiDoc.GetApiDocV2("spec.ingressClassName")
|
|
|
|
failures = append(failures, common.Failure{
|
|
Text: fmt.Sprintf("Ingress %s/%s does not specify an Ingress class.", ing.Namespace, ing.Name),
|
|
KubernetesDoc: doc,
|
|
Sensitive: []common.Sensitive{
|
|
{
|
|
Unmasked: ing.Namespace,
|
|
Masked: util.MaskString(ing.Namespace),
|
|
},
|
|
{
|
|
Unmasked: ing.Name,
|
|
Masked: util.MaskString(ing.Name),
|
|
},
|
|
},
|
|
})
|
|
} else {
|
|
ingressClassName = &ingClassValue
|
|
}
|
|
}
|
|
|
|
// check if ingressclass exist
|
|
if ingressClassName != nil {
|
|
_, err := a.Client.GetClient().NetworkingV1().IngressClasses().Get(a.Context, *ingressClassName, metav1.GetOptions{})
|
|
if err != nil {
|
|
doc := apiDoc.GetApiDocV2("spec.ingressClassName")
|
|
|
|
failures = append(failures, common.Failure{
|
|
Text: fmt.Sprintf("Ingress uses the ingress class %s which does not exist.", *ingressClassName),
|
|
KubernetesDoc: doc,
|
|
Sensitive: []common.Sensitive{
|
|
{
|
|
Unmasked: *ingressClassName,
|
|
Masked: util.MaskString(*ingressClassName),
|
|
},
|
|
},
|
|
})
|
|
}
|
|
}
|
|
|
|
// loop over rules
|
|
for _, rule := range ing.Spec.Rules {
|
|
// loop over HTTP paths
|
|
if rule.HTTP != nil {
|
|
for _, path := range rule.HTTP.Paths {
|
|
_, err := a.Client.GetClient().CoreV1().Services(ing.Namespace).Get(a.Context, path.Backend.Service.Name, metav1.GetOptions{})
|
|
if err != nil {
|
|
doc := apiDoc.GetApiDocV2("spec.rules.http.paths.backend.service")
|
|
|
|
failures = append(failures, common.Failure{
|
|
Text: fmt.Sprintf("Ingress uses the service %s/%s which does not exist.", ing.Namespace, path.Backend.Service.Name),
|
|
KubernetesDoc: doc,
|
|
Sensitive: []common.Sensitive{
|
|
{
|
|
Unmasked: ing.Namespace,
|
|
Masked: util.MaskString(ing.Namespace),
|
|
},
|
|
{
|
|
Unmasked: path.Backend.Service.Name,
|
|
Masked: util.MaskString(path.Backend.Service.Name),
|
|
},
|
|
},
|
|
})
|
|
}
|
|
}
|
|
}
|
|
}
|
|
|
|
for _, tls := range ing.Spec.TLS {
|
|
_, err := a.Client.GetClient().CoreV1().Secrets(ing.Namespace).Get(a.Context, tls.SecretName, metav1.GetOptions{})
|
|
if err != nil {
|
|
doc := apiDoc.GetApiDocV2("spec.tls.secretName")
|
|
|
|
failures = append(failures, common.Failure{
|
|
Text: fmt.Sprintf("Ingress uses the secret %s/%s as a TLS certificate which does not exist.", ing.Namespace, tls.SecretName),
|
|
KubernetesDoc: doc,
|
|
Sensitive: []common.Sensitive{
|
|
{
|
|
Unmasked: ing.Namespace,
|
|
Masked: util.MaskString(ing.Namespace),
|
|
},
|
|
{
|
|
Unmasked: tls.SecretName,
|
|
Masked: util.MaskString(tls.SecretName),
|
|
},
|
|
},
|
|
})
|
|
}
|
|
}
|
|
if len(failures) > 0 {
|
|
preAnalysis[fmt.Sprintf("%s/%s", ing.Namespace, ing.Name)] = common.PreAnalysis{
|
|
Ingress: ing,
|
|
FailureDetails: failures,
|
|
}
|
|
AnalyzerErrorsMetric.WithLabelValues(kind, ing.Name, ing.Namespace).Set(float64(len(failures)))
|
|
|
|
}
|
|
|
|
}
|
|
|
|
for key, value := range preAnalysis {
|
|
var currentAnalysis = common.Result{
|
|
Kind: kind,
|
|
Name: key,
|
|
Error: value.FailureDetails,
|
|
}
|
|
|
|
parent, found := util.GetParent(a.Client, value.Ingress.ObjectMeta)
|
|
if found {
|
|
currentAnalysis.ParentObject = parent
|
|
}
|
|
a.Results = append(a.Results, currentAnalysis)
|
|
}
|
|
|
|
return a.Results, nil
|
|
}
|