Skip to content

Commit fa4182b

Browse files
feat: exposed the input management_endpoint_type_for_bucket for bucket DAs and defaulted the value to direct (#938)
1 parent 2accb1e commit fa4182b

File tree

7 files changed

+50
-6
lines changed

7 files changed

+50
-6
lines changed

ibm_catalog.json

Lines changed: 26 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -421,7 +421,6 @@
421421
},
422422
{
423423
"key": "management_endpoint_type_for_bucket",
424-
"hidden": true,
425424
"options": [
426425
{
427426
"displayname": "direct",
@@ -759,6 +758,19 @@
759758
{
760759
"key": "bucket_hard_quota"
761760
},
761+
{
762+
"key": "management_endpoint_type_for_bucket",
763+
"options": [
764+
{
765+
"displayname": "direct",
766+
"value": "direct"
767+
},
768+
{
769+
"displayname": "private",
770+
"value": "private"
771+
}
772+
]
773+
},
762774
{
763775
"key": "monitoring_crn"
764776
},
@@ -1147,7 +1159,6 @@
11471159
},
11481160
{
11491161
"key": "management_endpoint_type_for_bucket",
1150-
"hidden": true,
11511162
"options": [
11521163
{
11531164
"displayname": "direct",
@@ -1544,6 +1555,19 @@
15441555
{
15451556
"key": "bucket_hard_quota"
15461557
},
1558+
{
1559+
"key": "management_endpoint_type_for_bucket",
1560+
"options": [
1561+
{
1562+
"displayname": "direct",
1563+
"value": "direct"
1564+
},
1565+
{
1566+
"displayname": "private",
1567+
"value": "private"
1568+
}
1569+
]
1570+
},
15471571
{
15481572
"key": "monitoring_crn"
15491573
},

solutions/cross-regional-bucket/fully-configurable/variables.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,7 @@ variable "bucket_name" {
126126
variable "management_endpoint_type_for_bucket" {
127127
description = "The type of endpoint for the IBM terraform provider to manage the bucket. Possible values: `public`, `private`, `direct`."
128128
type = string
129-
default = "private"
129+
default = "direct"
130130
validation {
131131
condition = contains(["public", "private", "direct"], var.management_endpoint_type_for_bucket)
132132
error_message = "The value of management_endpoint_type_for_bucket must be one of: `public`, `private`, `direct`."

solutions/cross-regional-bucket/security-enforced/main.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ module "cross_regional_bucket" {
1414
existing_cos_instance_crn = var.existing_cos_instance_crn
1515
bucket_access_tags = var.bucket_access_tags
1616
bucket_name = var.bucket_name
17-
management_endpoint_type_for_bucket = "private"
17+
management_endpoint_type_for_bucket = var.management_endpoint_type_for_bucket
1818
cross_region_location = var.cross_region_location
1919
bucket_storage_class = var.bucket_storage_class
2020
force_delete = var.force_delete

solutions/cross-regional-bucket/security-enforced/variables.tf

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,16 @@ variable "bucket_name" {
9696
description = "The name to give the newly provisioned Object Storage bucket."
9797
}
9898

99+
variable "management_endpoint_type_for_bucket" {
100+
description = "The type of endpoint for the IBM terraform provider to manage the bucket. Possible values: `private`, `direct`."
101+
type = string
102+
default = "direct"
103+
validation {
104+
condition = contains(["private", "direct"], var.management_endpoint_type_for_bucket)
105+
error_message = "The value of management_endpoint_type_for_bucket must be one of: `private`, `direct`."
106+
}
107+
}
108+
99109
variable "cross_region_location" {
100110
description = "Specify the cross-region bucket location. Possible values: `us`, `eu`, `ap`."
101111
type = string

solutions/regional-bucket/fully-configurable/variables.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ variable "bucket_name" {
135135
variable "management_endpoint_type_for_bucket" {
136136
description = "The type of endpoint for the IBM terraform provider to manage the bucket. Possible values: `public`, `private`, `direct`."
137137
type = string
138-
default = "private"
138+
default = "direct"
139139
validation {
140140
condition = contains(["public", "private", "direct"], var.management_endpoint_type_for_bucket)
141141
error_message = "The value of management_endpoint_type_for_bucket must be one of: `public`, `private`, `direct`."

solutions/regional-bucket/security-enforced/main.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ module "regional_bucket" {
1414
existing_cos_instance_crn = var.existing_cos_instance_crn
1515
bucket_access_tags = var.bucket_access_tags
1616
bucket_name = var.bucket_name
17-
management_endpoint_type_for_bucket = "private"
17+
management_endpoint_type_for_bucket = var.management_endpoint_type_for_bucket
1818
region = var.region
1919
bucket_storage_class = var.bucket_storage_class
2020
force_delete = var.force_delete

solutions/regional-bucket/security-enforced/variables.tf

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -105,6 +105,16 @@ variable "bucket_name" {
105105
description = "The name to give the newly provisioned Object Storage bucket."
106106
}
107107

108+
variable "management_endpoint_type_for_bucket" {
109+
description = "The type of endpoint for the IBM terraform provider to manage the bucket. Possible values: `private`, `direct`."
110+
type = string
111+
default = "direct"
112+
validation {
113+
condition = contains(["private", "direct"], var.management_endpoint_type_for_bucket)
114+
error_message = "The value of management_endpoint_type_for_bucket must be one of: `private`, `direct`."
115+
}
116+
}
117+
108118
variable "bucket_storage_class" {
109119
type = string
110120
description = "The storage class of the newly provisioned Object Storage bucket. Possible values: `standard`, `vault`, `cold`, `smart` `onerate_active`."

0 commit comments

Comments
 (0)