From a6d3c7a3c5856c66bfe9fd8297c43b9390a9cb2a Mon Sep 17 00:00:00 2001 From: dtfranz Date: Wed, 22 May 2024 10:45:14 -0700 Subject: [PATCH] Remove BundleDeployments gathering from e2e's gatherArtifacts When tests fail, gatherArtifacts is run to collect cluster state. This PR removes gathering of BundleDeployments since we don't use them anymore and it's generating additional misleading error messages. Signed-off-by: dtfranz --- test/e2e/cluster_extension_install_test.go | 18 ------------------ 1 file changed, 18 deletions(-) diff --git a/test/e2e/cluster_extension_install_test.go b/test/e2e/cluster_extension_install_test.go index 96bae39348..a3efe884e0 100644 --- a/test/e2e/cluster_extension_install_test.go +++ b/test/e2e/cluster_extension_install_test.go @@ -322,7 +322,6 @@ func TestClusterExtensionInstallSuccessorVersion(t *testing.T) { // - clusterextensions // - pods logs // - deployments -// - bundledeployments // - catalogsources func getArtifactsOutput(t *testing.T) { basePath := env.GetString("ARTIFACT_PATH", "") @@ -382,23 +381,6 @@ func getArtifactsOutput(t *testing.T) { } } - // Get all BundleDeployments in the namespace and save them to the artifact path. - bundleDeployments := rukpakv1alpha2.BundleDeploymentList{} - if err := c.List(context.Background(), &bundleDeployments, client.InNamespace("")); err != nil { - fmt.Printf("Failed to list bundleDeployments: %v", err) - } - for _, bundleDeployment := range bundleDeployments.Items { - // Save bundleDeployment to artifact path - bundleDeploymentYaml, err := yaml.Marshal(bundleDeployment) - if err != nil { - fmt.Printf("Failed to marshal bundleDeployment: %v", err) - continue - } - if err := os.WriteFile(filepath.Join(artifactPath, bundleDeployment.Name+"-bundleDeployment.yaml"), bundleDeploymentYaml, 0600); err != nil { - fmt.Printf("Failed to write bundleDeployment to file: %v", err) - } - } - for _, namespace := range namespaces.Items { // let's ignore kube-* namespaces. if strings.Contains(namespace.Name, "kube-") {