Skip to content

Commit

Permalink
Remove workaround for utility available in TF AWS provider v5.x (#5257,…
Browse files Browse the repository at this point in the history
… PR #6867)
  • Loading branch information
dsotirho-ucsc committed Feb 1, 2025
2 parents 895c09d + fa92c74 commit 9b521dd
Showing 1 changed file with 1 addition and 9 deletions.
10 changes: 1 addition & 9 deletions terraform/browser/browser.tf.json.template.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@
JSON,
cached_property,
config,
false,
iif,
)
from azul.collections import (
Expand Down Expand Up @@ -462,14 +461,7 @@ def bucket_origin_id(bucket):


def bucket_regional_domain_name(bucket):
if false():
return '${aws_s3_bucket.%s.bucket_regional_domain_name}' % bucket # noqa
else:
assert config.region == 'us-east-1'
# FIXME: Remove workaround for
# https://github.com/hashicorp/terraform-provider-aws/issues/15102
# https://github.com/DataBiosphere/azul/issues/5257
return aws.qualified_bucket_name(bucket) + '.s3.us-east-1.amazonaws.com'
return '${aws_s3_bucket.%s.bucket_regional_domain_name}' % bucket


def cloudfront_function(script: Path):
Expand Down

0 comments on commit 9b521dd

Please sign in to comment.