feat: exposed the ability to set the provider_visibility in the DA …

…[Learn more](https://registry.terraform.io/providers/IBM-Cloud/IBM/latest/docs#visibility-1) (#586)
terraform-ibm-modules · Nov 26, 2024 · 53580ae · 53580ae
1 parent bb897b5
commit 53580ae
Show file tree

Hide file tree

Showing 5 changed files with 30 additions and 0 deletions.
diff --git a/cra-config.yaml b/cra-config.yaml
@@ -6,3 +6,4 @@ CRA_TARGETS:
     PROFILE_ID: "48279384-3d29-4089-8259-8ed354774b4a"         # SCC profile ID (currently set to 'CIS IBM Cloud Foundations Benchmark v1.1.0' '1.1.0' profile).
     CRA_ENVIRONMENT_VARIABLES:  # An optional map of environment variables for CRA, where the key is the variable name and value is the value. Useful for providing TF_VARs.
       TF_VAR_resource_group_name: "test"
+      TF_VAR_provider_visibility: "public"
diff --git a/ibm_catalog.json b/ibm_catalog.json
@@ -65,6 +65,23 @@
             {
               "key": "ibmcloud_api_key"
             },
+            {
+              "key": "provider_visibility",
+              "options": [
+                {
+                  "displayname": "private",
+                  "value": "private"
+                },
+                {
+                  "displayname": "public",
+                  "value": "public"
+                },
+                {
+                  "displayname": "public-and-private",
+                  "value": "public-and-private"
+                }
+              ]
+            },
             {
               "key": "use_existing_resource_group"
             },

diff --git a/solutions/standard/provider.tf b/solutions/standard/provider.tf
@@ -1,4 +1,5 @@
 provider "ibm" {
   ibmcloud_api_key = var.ibmcloud_api_key
   region           = var.region
+  visibility       = var.provider_visibility
 }
diff --git a/solutions/standard/variables.tf b/solutions/standard/variables.tf
@@ -8,6 +8,16 @@ variable "ibmcloud_api_key" {
   sensitive   = true
 }
 
+variable "provider_visibility" {
+  description = "Set the visibility value for the IBM terraform provider. Supported values are `public`, `private`, `public-and-private`. [Learn more](https://registry.terraform.io/providers/IBM-Cloud/ibm/latest/docs/guides/custom-service-endpoints)."
+  type        = string
+  default     = "private"
+
+  validation {
+    condition     = contains(["public", "private", "public-and-private"], var.provider_visibility)
+    error_message = "Invalid visibility option. Allowed values are 'public', 'private', or 'public-and-private'."
+  }
+}
 variable "use_existing_resource_group" {
   type        = bool
   description = "Whether to use an existing resource group."

diff --git a/tests/pr_test.go b/tests/pr_test.go
@@ -78,6 +78,7 @@ func TestRunUpgradeDASolution(t *testing.T) {
 	terraformVars := map[string]interface{}{
 		"resource_group_name":       options.Prefix,
 		"kms_endpoint_type":         "public",
+		"provider_visibility":       "public",
 		"existing_kms_instance_crn": permanentResources["hpcs_south_crn"],
 		"keys":                      []map[string]interface{}{{"key_ring_name": "my-key-ring", "keys": []map[string]interface{}{{"key_name": "some-key-name-1"}, {"key_name": "some-key-name-2"}}}},
 		"resource_tags":             []string{"kms-da-upg"},