Skip to content

Commit 5cf83f9

Browse files
committed
Fix panic by only constructing crClientForHC only when the hcKubeconfig is defined
1 parent 7025ca5 commit 5cf83f9

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

tests/e2e/e2e_suite_test.go

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -149,19 +149,20 @@ func TestOADPE2E(t *testing.T) {
149149

150150
gomega.RegisterFailHandler(ginkgo.Fail)
151151

152+
kubernetesClientForSuiteRun, err = kubernetes.NewForConfig(kubeConfig)
153+
gomega.Expect(err).NotTo(gomega.HaveOccurred())
154+
152155
// Set up kubeConfigForHC if kubeconfig_hc flag is provided
153156
if hcKubeconfig != "" {
154157
kubeConfigForHC, err = clientcmd.BuildConfigFromFlags("", hcKubeconfig)
155158
gomega.Expect(err).NotTo(gomega.HaveOccurred())
159+
156160
kubeConfigForHC.QPS = kubeConfig.QPS
157161
kubeConfigForHC.Burst = kubeConfig.Burst
158-
}
159162

160-
kubernetesClientForSuiteRun, err = kubernetes.NewForConfig(kubeConfig)
161-
gomega.Expect(err).NotTo(gomega.HaveOccurred())
162-
163-
crClientForHC, err = client.New(kubeConfigForHC, client.Options{Scheme: lib.Scheme})
164-
gomega.Expect(err).NotTo(gomega.HaveOccurred())
163+
crClientForHC, err = client.New(kubeConfigForHC, client.Options{Scheme: lib.Scheme})
164+
gomega.Expect(err).NotTo(gomega.HaveOccurred())
165+
}
165166

166167
runTimeClientForSuiteRun, err = client.New(kubeConfig, client.Options{Scheme: lib.Scheme})
167168
gomega.Expect(err).NotTo(gomega.HaveOccurred())

0 commit comments

Comments
 (0)