From 3192fb9c3da1b859b7d61b9e54ef5e8a24550210 Mon Sep 17 00:00:00 2001 From: Ivan Valdes Date: Fri, 14 Feb 2025 22:12:01 -0800 Subject: [PATCH] Remove unnecessary make command to install tools in e2e tests The make kustomize and controller-gen targets are prerequisites of the targets later called in the e2e tests, so there's no need to manually trigger them. Signed-off-by: Ivan Valdes --- test/e2e/e2e_suite_test.go | 14 +------------- 1 file changed, 1 insertion(+), 13 deletions(-) diff --git a/test/e2e/e2e_suite_test.go b/test/e2e/e2e_suite_test.go index 1a7166e..cd21704 100644 --- a/test/e2e/e2e_suite_test.go +++ b/test/e2e/e2e_suite_test.go @@ -64,21 +64,9 @@ func TestMain(m *testing.M) { // prepare the resources func(ctx context.Context, cfg *envconf.Config) (context.Context, error) { - log.Println("Installing bin tools...") - cmd := exec.Command("make", "kustomize") - if _, err := test_utils.Run(cmd); err != nil { - log.Printf("Failed to install kustomize binary: %s", err) - return ctx, err - } - cmd = exec.Command("make", "controller-gen") - if _, err := test_utils.Run(cmd); err != nil { - log.Printf("Failed to install controller-gen binary: %s", err) - return ctx, err - } - // gen manifest files log.Println("Generate manifests...") - cmd = exec.Command("make", "manifests") + cmd := exec.Command("make", "manifests") if _, err := test_utils.Run(cmd); err != nil { log.Printf("Failed to generate manifests: %s", err) return ctx, err