From c256685900056413518f1d8239624d101f184b07 Mon Sep 17 00:00:00 2001 From: Agustin Bettati Date: Mon, 17 Feb 2025 18:15:34 +0100 Subject: [PATCH 1/2] chore: Temporary fix on tenant cluster test passing zone name explicitly --- .../advancedcluster/resource_advanced_cluster_test.go | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/internal/service/advancedcluster/resource_advanced_cluster_test.go b/internal/service/advancedcluster/resource_advanced_cluster_test.go index be0e681720..18e041a629 100644 --- a/internal/service/advancedcluster/resource_advanced_cluster_test.go +++ b/internal/service/advancedcluster/resource_advanced_cluster_test.go @@ -136,11 +136,12 @@ func TestAccAdvancedCluster_basicTenant(t *testing.T) { CheckDestroy: acc.CheckDestroyCluster, Steps: []resource.TestStep{ { - Config: configTenant(t, true, projectID, clusterName, ""), + // zone name is hardcoded directly as a temporary fix, depends on CLOUDP-300819 or CLOUDP-301101 + Config: configTenant(t, true, projectID, clusterName, "Zone 1"), Check: checkTenant(true, projectID, clusterName), }, { - Config: configTenant(t, true, projectID, clusterNameUpdated, ""), + Config: configTenant(t, true, projectID, clusterNameUpdated, "Zone 1"), Check: checkTenant(true, projectID, clusterNameUpdated), }, acc.TestStepImportCluster(resourceName), From e67cff884e854dbcc7aa51ea2e01baf00e5bf954 Mon Sep 17 00:00:00 2001 From: Agustin Bettati Date: Mon, 17 Feb 2025 18:51:15 +0100 Subject: [PATCH 2/2] add additional comment in second step --- .../service/advancedcluster/resource_advanced_cluster_test.go | 1 + 1 file changed, 1 insertion(+) diff --git a/internal/service/advancedcluster/resource_advanced_cluster_test.go b/internal/service/advancedcluster/resource_advanced_cluster_test.go index 18e041a629..18c07de674 100644 --- a/internal/service/advancedcluster/resource_advanced_cluster_test.go +++ b/internal/service/advancedcluster/resource_advanced_cluster_test.go @@ -141,6 +141,7 @@ func TestAccAdvancedCluster_basicTenant(t *testing.T) { Check: checkTenant(true, projectID, clusterName), }, { + // zone name is hardcoded directly as a temporary fix, depends on CLOUDP-300819 or CLOUDP-301101 Config: configTenant(t, true, projectID, clusterNameUpdated, "Zone 1"), Check: checkTenant(true, projectID, clusterNameUpdated), },