From 1ce594b7ee0a92bad5793e63c9a31e0a7c7e0903 Mon Sep 17 00:00:00 2001 From: Sean Sullivan Date: Fri, 6 Aug 2021 22:31:55 -0700 Subject: [PATCH] Fixes flaky GKE kubectl test --- test/e2e/kubectl/kubectl.go | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/test/e2e/kubectl/kubectl.go b/test/e2e/kubectl/kubectl.go index c5fc2f5d84b..4afa875e783 100644 --- a/test/e2e/kubectl/kubectl.go +++ b/test/e2e/kubectl/kubectl.go @@ -644,6 +644,11 @@ var _ = SIGDescribe("Kubectl client", func() { }) ginkgo.It("should handle in-cluster config", func() { + // This test does not work for dynamically linked kubectl binaries; only statically linked ones. The + // problem happens when the kubectl binary is copied to a pod in the cluster. For dynamically linked + // binaries, the necessary libraries are not also copied. For this reason, the test can not be + // guaranteed to work with GKE, which sometimes run tests using a dynamically linked kubectl. + e2eskipper.SkipIfProviderIs("gke") // TODO: Find a way to download and copy the appropriate kubectl binary, or maybe a multi-arch kubectl image // for now this only works on amd64 e2eskipper.SkipUnlessNodeOSArchIs("amd64")