Skip to content

Commit 93e13f8

Browse files
committed
use var.bucket_name
1 parent d9ad226 commit 93e13f8

File tree

3 files changed

+7
-7
lines changed

3 files changed

+7
-7
lines changed

examples/byo-vpc-eks-sql-redis/main.tf

+2-2
Original file line numberDiff line numberDiff line change
@@ -174,12 +174,12 @@ module "wandb" {
174174

175175
extraEnv = var.other_wandb_env
176176

177-
bucket = var.create_bucket ? null : {
177+
bucket = var.bucket_name != "" ? {
178178
provider = "s3"
179179
name = var.bucket_name
180180
region = data.aws_s3_bucket.file_storage.region
181181
kmsKey = var.bucket_kms_key_arn
182-
}
182+
} : null
183183
defaultBucket = {
184184
provider = "s3"
185185
name = module.file_storage.bucket_name

examples/byo-vpc-sql/main.tf

+3-3
Original file line numberDiff line numberDiff line change
@@ -165,7 +165,7 @@ module "app_eks" {
165165
map_users = var.kubernetes_map_users
166166

167167
bucket_kms_key_arns = local.use_external_bucket ? var.bucket_kms_key_arn : local.kms_key_arn
168-
bucket_arn = var.create_bucket ? module.file_storage.bucket_arn : data.aws_s3_bucket.file_storage.arn
168+
bucket_arn = var.bucket_name == "" ? module.file_storage.bucket_arn : data.aws_s3_bucket.file_storage.arn
169169
bucket_sqs_queue_arn = local.use_internal_queue ? null : data.aws_sqs_queue.file_storage.0.arn
170170

171171
network_id = local.network_id
@@ -279,12 +279,12 @@ module "wandb" {
279279

280280
extraEnv = var.other_wandb_env
281281

282-
bucket = var.create_bucket ? null : {
282+
bucket = var.bucket_name != "" ? {
283283
provider = "s3"
284284
name = var.bucket_name
285285
region = data.aws_s3_bucket.file_storage.region
286286
kmsKey = var.bucket_kms_key_arn
287-
}
287+
} : null
288288

289289
defaultBucket = {
290290
provider = "s3"

main.tf

+2-2
Original file line numberDiff line numberDiff line change
@@ -269,13 +269,13 @@ module "wandb" {
269269
cloudProvider = "aws"
270270
extraEnv = var.other_wandb_env
271271

272-
bucket = var.create_bucket ? null : {
272+
bucket = var.bucket_name != "" ? {
273273
provider = "s3"
274274
name = var.bucket_name
275275
path = var.bucket_path
276276
region = data.aws_s3_bucket.file_storage.region
277277
kmsKey = var.bucket_kms_key_arn
278-
}
278+
} : null
279279
defaultBucket = {
280280
provider = "s3"
281281
name = module.file_storage.bucket_name

0 commit comments

Comments
 (0)