mirror of
https://github.com/k8sgpt-ai/k8sgpt.git
synced 2025-08-18 07:47:05 +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>
209 lines
5.2 KiB
Go
209 lines
5.2 KiB
Go
/*
|
|
Copyright 2024 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 (
|
|
"context"
|
|
"testing"
|
|
|
|
"github.com/k8sgpt-ai/k8sgpt/pkg/common"
|
|
"github.com/k8sgpt-ai/k8sgpt/pkg/kubernetes"
|
|
"github.com/stretchr/testify/require"
|
|
policyv1 "k8s.io/api/policy/v1"
|
|
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
|
|
"k8s.io/apimachinery/pkg/util/intstr"
|
|
"k8s.io/client-go/kubernetes/fake"
|
|
)
|
|
|
|
func TestPodDisruptionBudgetAnalyzer(t *testing.T) {
|
|
config := common.Analyzer{
|
|
Client: &kubernetes.Client{
|
|
Client: fake.NewSimpleClientset(
|
|
&policyv1.PodDisruptionBudget{
|
|
ObjectMeta: metav1.ObjectMeta{
|
|
Name: "PDB1",
|
|
Namespace: "test",
|
|
},
|
|
// Status conditions are nil.
|
|
Status: policyv1.PodDisruptionBudgetStatus{
|
|
Conditions: nil,
|
|
},
|
|
},
|
|
&policyv1.PodDisruptionBudget{
|
|
ObjectMeta: metav1.ObjectMeta{
|
|
Name: "PDB2",
|
|
Namespace: "test",
|
|
},
|
|
// Status conditions are empty.
|
|
Status: policyv1.PodDisruptionBudgetStatus{
|
|
Conditions: []metav1.Condition{},
|
|
},
|
|
},
|
|
&policyv1.PodDisruptionBudget{
|
|
ObjectMeta: metav1.ObjectMeta{
|
|
Name: "PDB3",
|
|
Namespace: "test",
|
|
},
|
|
Status: policyv1.PodDisruptionBudgetStatus{
|
|
Conditions: []metav1.Condition{
|
|
{
|
|
Type: "DisruptionAllowed",
|
|
Status: "False",
|
|
Reason: "test reason",
|
|
},
|
|
},
|
|
},
|
|
Spec: policyv1.PodDisruptionBudgetSpec{
|
|
MaxUnavailable: &intstr.IntOrString{
|
|
Type: 0,
|
|
IntVal: 17,
|
|
StrVal: "17",
|
|
},
|
|
MinAvailable: &intstr.IntOrString{
|
|
Type: 0,
|
|
IntVal: 7,
|
|
StrVal: "7",
|
|
},
|
|
// MatchLabels specified.
|
|
Selector: &metav1.LabelSelector{
|
|
MatchLabels: map[string]string{
|
|
"label1": "test1",
|
|
"label2": "test2",
|
|
},
|
|
},
|
|
},
|
|
},
|
|
&policyv1.PodDisruptionBudget{
|
|
ObjectMeta: metav1.ObjectMeta{
|
|
Name: "PDB4",
|
|
Namespace: "test",
|
|
},
|
|
Status: policyv1.PodDisruptionBudgetStatus{
|
|
Conditions: []metav1.Condition{
|
|
{
|
|
Type: "DisruptionAllowed",
|
|
Status: "False",
|
|
Reason: "test reason",
|
|
},
|
|
},
|
|
},
|
|
// Match Labels Empty.
|
|
Spec: policyv1.PodDisruptionBudgetSpec{
|
|
Selector: &metav1.LabelSelector{},
|
|
},
|
|
},
|
|
),
|
|
},
|
|
Context: context.Background(),
|
|
Namespace: "test",
|
|
}
|
|
|
|
pdbAnalyzer := PdbAnalyzer{}
|
|
results, err := pdbAnalyzer.Analyze(config)
|
|
require.NoError(t, err)
|
|
require.Equal(t, 1, len(results))
|
|
require.Equal(t, "test/PDB3", results[0].Name)
|
|
}
|
|
|
|
func TestPodDisruptionBudgetAnalyzerLabelSelectorFiltering(t *testing.T) {
|
|
config := common.Analyzer{
|
|
Client: &kubernetes.Client{
|
|
Client: fake.NewSimpleClientset(
|
|
&policyv1.PodDisruptionBudget{
|
|
ObjectMeta: metav1.ObjectMeta{
|
|
Name: "PDB1",
|
|
Namespace: "default",
|
|
Labels: map[string]string{
|
|
"app": "pdb",
|
|
},
|
|
},
|
|
// Status conditions are nil.
|
|
Status: policyv1.PodDisruptionBudgetStatus{
|
|
Conditions: []metav1.Condition{
|
|
{
|
|
Type: "DisruptionAllowed",
|
|
Status: "False",
|
|
Reason: "test reason",
|
|
},
|
|
},
|
|
},
|
|
Spec: policyv1.PodDisruptionBudgetSpec{
|
|
MaxUnavailable: &intstr.IntOrString{
|
|
Type: 0,
|
|
IntVal: 17,
|
|
StrVal: "17",
|
|
},
|
|
MinAvailable: &intstr.IntOrString{
|
|
Type: 0,
|
|
IntVal: 7,
|
|
StrVal: "7",
|
|
},
|
|
// MatchLabels specified.
|
|
Selector: &metav1.LabelSelector{
|
|
MatchLabels: map[string]string{
|
|
"label1": "test1",
|
|
"label2": "test2",
|
|
},
|
|
},
|
|
},
|
|
},
|
|
&policyv1.PodDisruptionBudget{
|
|
ObjectMeta: metav1.ObjectMeta{
|
|
Name: "PDB2",
|
|
Namespace: "default",
|
|
},
|
|
// Status conditions are empty.
|
|
Status: policyv1.PodDisruptionBudgetStatus{
|
|
Conditions: []metav1.Condition{
|
|
{
|
|
Type: "DisruptionAllowed",
|
|
Status: "False",
|
|
Reason: "test reason",
|
|
},
|
|
},
|
|
},
|
|
Spec: policyv1.PodDisruptionBudgetSpec{
|
|
MaxUnavailable: &intstr.IntOrString{
|
|
Type: 0,
|
|
IntVal: 17,
|
|
StrVal: "17",
|
|
},
|
|
MinAvailable: &intstr.IntOrString{
|
|
Type: 0,
|
|
IntVal: 7,
|
|
StrVal: "7",
|
|
},
|
|
// MatchLabels specified.
|
|
Selector: &metav1.LabelSelector{
|
|
MatchLabels: map[string]string{
|
|
"label1": "test1",
|
|
"label2": "test2",
|
|
},
|
|
},
|
|
},
|
|
},
|
|
),
|
|
},
|
|
Context: context.Background(),
|
|
Namespace: "default",
|
|
LabelSelector: "app=pdb",
|
|
}
|
|
|
|
pdbAnalyzer := PdbAnalyzer{}
|
|
results, err := pdbAnalyzer.Analyze(config)
|
|
require.NoError(t, err)
|
|
require.Equal(t, 1, len(results))
|
|
require.Equal(t, "default/PDB1", results[0].Name)
|
|
}
|