From feef4bf5082442b388dce49ffd8c86cbb1ff52e9 Mon Sep 17 00:00:00 2001 From: Ryan Richard Date: Wed, 6 Nov 2024 09:56:36 -0800 Subject: [PATCH] fix test flake by removing memory limit from test pod On AKS clusters, the pod's container would exceed its memory limit, get OOMKilled, get restarted, and cause that test to flake. Co-authored-by: Joshua Casey --- .../integration/concierge_impersonation_proxy_test.go | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/test/integration/concierge_impersonation_proxy_test.go b/test/integration/concierge_impersonation_proxy_test.go index 968f322ba..96df0343c 100644 --- a/test/integration/concierge_impersonation_proxy_test.go +++ b/test/integration/concierge_impersonation_proxy_test.go @@ -41,7 +41,6 @@ import ( rbacv1 "k8s.io/api/rbac/v1" "k8s.io/apimachinery/pkg/api/equality" apierrors "k8s.io/apimachinery/pkg/api/errors" - "k8s.io/apimachinery/pkg/api/resource" metav1 "k8s.io/apimachinery/pkg/apis/meta/v1" "k8s.io/apimachinery/pkg/apis/meta/v1/unstructured/unstructuredscheme" "k8s.io/apimachinery/pkg/labels" @@ -1095,16 +1094,6 @@ func TestImpersonationProxy(t *testing.T) { //nolint:gocyclo // yeah, it's compl ImagePullPolicy: corev1.PullIfNotPresent, Command: []string{"bash", "-c", `while true; do read VAR; echo "VAR: $VAR"; done`}, Stdin: true, - Resources: corev1.ResourceRequirements{ - Limits: corev1.ResourceList{ - corev1.ResourceMemory: resource.MustParse("16Mi"), - corev1.ResourceCPU: resource.MustParse("10m"), - }, - Requests: corev1.ResourceList{ - corev1.ResourceMemory: resource.MustParse("16Mi"), - corev1.ResourceCPU: resource.MustParse("10m"), - }, - }, // Use a restrictive security context just in case the test cluster has PSAs enabled. SecurityContext: testlib.RestrictiveSecurityContext(), }}})