Skip to content

Commit 2ce86ab

Browse files
committed
moved test to e2e
Signed-off-by: Omer Aplatony <[email protected]>
1 parent 40b2576 commit 2ce86ab

File tree

2 files changed

+18
-16
lines changed

2 files changed

+18
-16
lines changed

vertical-pod-autoscaler/e2e/v1/admission_controller.go

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -983,6 +983,24 @@ var _ = AdmissionControllerE2eDescribe("Admission-controller", func() {
983983
}
984984
}
985985
}`),
986+
`spec.resourcePolicy.containerPolicies[0].mode: Unsupported value: "bad": supported values: "Auto", "Off"`: []byte(`{
987+
"kind": "VerticalPodAutoscaler",
988+
"apiVersion": "autoscaling.k8s.io/v1",
989+
"metadata": {"name": "hamster-vpa-invalid-mode"},
990+
"spec": {
991+
"targetRef": {
992+
"apiVersion": "apps/v1",
993+
"kind": "Deployment",
994+
"name": "hamster"
995+
},
996+
"resourcePolicy": {
997+
"containerPolicies": [{
998+
"containerName": "loot box",
999+
"mode": "bad"
1000+
}]
1001+
},
1002+
}
1003+
}`),
9861004
}
9871005

9881006
var err2 error

vertical-pod-autoscaler/pkg/admission-controller/resource/vpa/handler_test.go

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -87,22 +87,6 @@ func TestValidateVPA(t *testing.T) {
8787
},
8888
},
8989
},
90-
{
91-
name: "invalid scaling mode",
92-
vpa: vpa_types.VerticalPodAutoscaler{
93-
Spec: vpa_types.VerticalPodAutoscalerSpec{
94-
ResourcePolicy: &vpa_types.PodResourcePolicy{
95-
ContainerPolicies: []vpa_types.ContainerResourcePolicy{
96-
{
97-
ContainerName: "loot box",
98-
Mode: &badScalingMode,
99-
},
100-
},
101-
},
102-
},
103-
},
104-
expectError: fmt.Errorf("unexpected Mode value bad"),
105-
},
10690
{
10791
name: "bad limits",
10892
vpa: vpa_types.VerticalPodAutoscaler{

0 commit comments

Comments
 (0)