Skip to content

Commit 1b694d1

Browse files
committed
OCPBUGS-33925: tfvars: change asset's name
Now that some platforms are using CAPI by default, an error during the tfvars asset generation can be confusing by making it look like we're still using terraform: ``` 05-19 17:36:32.935 level=fatal msg=failed to fetch Terraform Variables: failed to fetch dependency of "Terraform Variables": failed to generate asset "Platform Provisioning Check": baseDomain: Invalid value: "qe.devcluster.openshift.com": the zone already has record sets for the domain of the cluster: [api.gpei-0519a.qe.devcluster.openshift.com. (A)] ``` By changing the asset's name and leaving the rest unchanged, we can eliminate such confusion and avoid breaking other platforms still using terraform, until we're ready to stop generating tfvars json files altogether.
1 parent 3a11908 commit 1b694d1

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

pkg/asset/cluster/tfvars/tfvars.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ const (
8282
// https://www.terraform.io/docs/configuration/variables.html#variable-files
8383
TfPlatformVarsFileName = "terraform.platform.auto.tfvars.json"
8484

85-
tfvarsAssetName = "Terraform Variables"
85+
tfvarsAssetName = "Cluster Infrastructure Variables"
8686
)
8787

8888
// TerraformVariables depends on InstallConfig, Manifests,

0 commit comments

Comments
 (0)