Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

provider: Adds support for enabling regional endpoint for S3 #33024

Merged
merged 7 commits into from
Aug 18, 2023

Conversation

gdavison
Copy link
Contributor

@gdavison gdavison commented Aug 15, 2023

Description

Adds configuration parameter s3_us_east_1_regional_endpoint and enables support for environment variable AWS_S3_US_EAST_1_REGIONAL_ENDPOINT and shared config file parameter s3_us_east_1_regional_endpoint.

Relates #10923

Depends on hashicorp/aws-sdk-go-base#610

@github-actions
Copy link

Community Note

Voting for Prioritization

  • Please vote on this pull request by adding a 👍 reaction to the original post to help the community and maintainers prioritize this pull request.
  • Please see our prioritization guide for information on how we prioritize.
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request.

For Submitters

  • Review the contribution guide relating to the type of change you are making to ensure all of the necessary steps have been taken.
  • For new resources and data sources, use skaff to generate scaffolding with comments detailing common expectations.
  • Whether or not the branch has been rebased will not impact prioritization, but doing so is always a welcome surprise.

@github-actions github-actions bot added size/M Managed by automation to categorize the size of a PR. documentation Introduces or discusses updates to documentation. service/s3 Issues and PRs that pertain to the s3 service. provider Pertains to the provider itself, rather than any interaction with AWS. client-connections Pertains to the AWS Client and service connections. labels Aug 15, 2023
@terraform-aws-provider terraform-aws-provider bot added the prioritized Part of the maintainer teams immediate focus. To be addressed within the current quarter. label Aug 15, 2023
@gdavison gdavison force-pushed the f-s3-us-east-1-regional branch from 6f61e70 to 24cbe3a Compare August 15, 2023 19:34
@gdavison gdavison marked this pull request as draft August 15, 2023 20:01
@gdavison gdavison marked this pull request as ready for review August 17, 2023 17:35
.changelog/33024.txt Outdated Show resolved Hide resolved
Co-authored-by: Kit Ewbank <Kit_Ewbank@hotmail.com>
Copy link
Contributor

@ewbankkit ewbankkit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀.

% make sane
==> Sane Check (48 tests of Top 30 resources)
==> Like 'sanity' except full output and stops soon after 1st error
==> NOTE: NOT an exhaustive set of tests! Finds big problems only.
go: downloading github.com/hashicorp/aws-sdk-go-base/v2/awsv1shim/v2 v2.0.0-beta.34.0.20230814235649-2c1764785bed
=== RUN   TestAccIAMInstanceProfile_basic
=== PAUSE TestAccIAMInstanceProfile_basic
=== RUN   TestAccIAMInstanceProfile_tags
=== PAUSE TestAccIAMInstanceProfile_tags
=== RUN   TestAccIAMPolicyDocumentDataSource_basic
=== PAUSE TestAccIAMPolicyDocumentDataSource_basic
=== RUN   TestAccIAMPolicyDocumentDataSource_sourceConflicting
=== PAUSE TestAccIAMPolicyDocumentDataSource_sourceConflicting
=== RUN   TestAccIAMPolicy_basic
=== PAUSE TestAccIAMPolicy_basic
=== RUN   TestAccIAMPolicy_tags
=== PAUSE TestAccIAMPolicy_tags
=== RUN   TestAccIAMPolicy_policy
=== PAUSE TestAccIAMPolicy_policy
=== RUN   TestAccIAMRolePolicyAttachment_basic
=== PAUSE TestAccIAMRolePolicyAttachment_basic
=== RUN   TestAccIAMRolePolicyAttachment_disappears
=== PAUSE TestAccIAMRolePolicyAttachment_disappears
=== RUN   TestAccIAMRolePolicyAttachment_Disappears_role
=== PAUSE TestAccIAMRolePolicyAttachment_Disappears_role
=== RUN   TestAccIAMRolePolicy_basic
=== PAUSE TestAccIAMRolePolicy_basic
=== RUN   TestAccIAMRolePolicy_unknownsInPolicy
=== PAUSE TestAccIAMRolePolicy_unknownsInPolicy
=== RUN   TestAccIAMRole_basic
=== PAUSE TestAccIAMRole_basic
=== RUN   TestAccIAMRole_namePrefix
=== PAUSE TestAccIAMRole_namePrefix
=== RUN   TestAccIAMRole_disappears
=== PAUSE TestAccIAMRole_disappears
=== RUN   TestAccIAMRole_InlinePolicy_basic
=== PAUSE TestAccIAMRole_InlinePolicy_basic
=== CONT  TestAccIAMInstanceProfile_basic
=== CONT  TestAccIAMRolePolicyAttachment_disappears
=== CONT  TestAccIAMRolePolicyAttachment_basic
=== CONT  TestAccIAMRole_InlinePolicy_basic
=== CONT  TestAccIAMRole_disappears
=== CONT  TestAccIAMRolePolicyAttachment_Disappears_role
=== CONT  TestAccIAMPolicyDocumentDataSource_sourceConflicting
=== CONT  TestAccIAMPolicy_basic
=== CONT  TestAccIAMPolicy_policy
=== CONT  TestAccIAMPolicy_tags
=== CONT  TestAccIAMRolePolicy_basic
=== CONT  TestAccIAMRolePolicy_unknownsInPolicy
=== CONT  TestAccIAMRole_namePrefix
=== CONT  TestAccIAMPolicyDocumentDataSource_basic
=== CONT  TestAccIAMInstanceProfile_tags
=== CONT  TestAccIAMRole_basic
--- PASS: TestAccIAMPolicyDocumentDataSource_basic (118.75s)
--- PASS: TestAccIAMPolicyDocumentDataSource_sourceConflicting (119.42s)
--- PASS: TestAccIAMRolePolicyAttachment_Disappears_role (134.18s)
--- PASS: TestAccIAMRole_disappears (135.55s)
--- PASS: TestAccIAMRolePolicyAttachment_disappears (143.46s)
--- PASS: TestAccIAMRolePolicy_unknownsInPolicy (144.40s)
--- PASS: TestAccIAMInstanceProfile_basic (160.85s)
--- PASS: TestAccIAMRole_namePrefix (161.30s)
--- PASS: TestAccIAMRole_basic (161.33s)
--- PASS: TestAccIAMPolicy_basic (163.36s)
--- PASS: TestAccIAMRolePolicyAttachment_basic (211.32s)
--- PASS: TestAccIAMRolePolicy_basic (212.12s)
--- PASS: TestAccIAMPolicy_policy (213.05s)
--- PASS: TestAccIAMPolicy_tags (235.70s)
--- PASS: TestAccIAMInstanceProfile_tags (236.43s)
--- PASS: TestAccIAMRole_InlinePolicy_basic (236.48s)
PASS
ok  	github.com/hashicorp/terraform-provider-aws/internal/service/iam	242.070s
=== RUN   TestAccLogsGroup_basic
=== PAUSE TestAccLogsGroup_basic
=== RUN   TestAccLogsGroup_multiple
=== PAUSE TestAccLogsGroup_multiple
=== CONT  TestAccLogsGroup_multiple
=== CONT  TestAccLogsGroup_basic
--- PASS: TestAccLogsGroup_multiple (66.45s)
--- PASS: TestAccLogsGroup_basic (80.04s)
PASS
ok  	github.com/hashicorp/terraform-provider-aws/internal/service/logs	89.597s
=== RUN   TestAccVPCDataSource_basic
=== PAUSE TestAccVPCDataSource_basic
=== RUN   TestAccVPCRouteTableAssociation_Subnet_basic
=== PAUSE TestAccVPCRouteTableAssociation_Subnet_basic
=== RUN   TestAccVPCRouteTable_basic
=== PAUSE TestAccVPCRouteTable_basic
=== RUN   TestAccVPCSecurityGroupRule_race
=== PAUSE TestAccVPCSecurityGroupRule_race
=== RUN   TestAccVPCSecurityGroupRule_protocolChange
=== PAUSE TestAccVPCSecurityGroupRule_protocolChange
=== RUN   TestAccVPCSecurityGroup_basic
=== PAUSE TestAccVPCSecurityGroup_basic
=== RUN   TestAccVPCSecurityGroup_ipRangesWithSameRules
=== PAUSE TestAccVPCSecurityGroup_ipRangesWithSameRules
=== RUN   TestAccVPCSecurityGroup_vpcAllEgress
=== PAUSE TestAccVPCSecurityGroup_vpcAllEgress
=== RUN   TestAccVPCSubnet_basic
=== PAUSE TestAccVPCSubnet_basic
=== RUN   TestAccVPC_tenancy
=== PAUSE TestAccVPC_tenancy
=== CONT  TestAccVPCDataSource_basic
=== CONT  TestAccVPC_tenancy
=== CONT  TestAccVPCSubnet_basic
=== CONT  TestAccVPCSecurityGroup_ipRangesWithSameRules
=== CONT  TestAccVPCSecurityGroupRule_race
=== CONT  TestAccVPCSecurityGroup_vpcAllEgress
=== CONT  TestAccVPCSecurityGroupRule_protocolChange
=== CONT  TestAccVPCRouteTable_basic
=== CONT  TestAccVPCRouteTableAssociation_Subnet_basic
=== CONT  TestAccVPCSecurityGroup_basic
--- PASS: TestAccVPCSecurityGroup_vpcAllEgress (161.99s)
--- PASS: TestAccVPCDataSource_basic (168.26s)
--- PASS: TestAccVPCRouteTable_basic (172.11s)
--- PASS: TestAccVPCSecurityGroup_basic (174.48s)
--- PASS: TestAccVPCSubnet_basic (175.77s)
--- PASS: TestAccVPCRouteTableAssociation_Subnet_basic (175.78s)
--- PASS: TestAccVPCSecurityGroup_ipRangesWithSameRules (175.91s)
--- PASS: TestAccVPC_tenancy (231.53s)
--- PASS: TestAccVPCSecurityGroupRule_protocolChange (233.72s)
--- PASS: TestAccVPCSecurityGroupRule_race (254.19s)
PASS
ok  	github.com/hashicorp/terraform-provider-aws/internal/service/ec2	267.003s
=== RUN   TestAccECSService_basic
=== PAUSE TestAccECSService_basic
=== RUN   TestAccECSService_basicImport
=== PAUSE TestAccECSService_basicImport
=== RUN   TestAccECSTaskDefinition_basic
=== PAUSE TestAccECSTaskDefinition_basic
=== CONT  TestAccECSService_basic
=== CONT  TestAccECSTaskDefinition_basic
=== CONT  TestAccECSService_basicImport
--- PASS: TestAccECSTaskDefinition_basic (170.09s)
--- PASS: TestAccECSService_basic (205.80s)
--- PASS: TestAccECSService_basicImport (278.95s)
PASS
ok  	github.com/hashicorp/terraform-provider-aws/internal/service/ecs	299.450s
=== RUN   TestAccELBV2TargetGroup_basic
=== PAUSE TestAccELBV2TargetGroup_basic
=== CONT  TestAccELBV2TargetGroup_basic
--- PASS: TestAccELBV2TargetGroup_basic (94.04s)
PASS
ok  	github.com/hashicorp/terraform-provider-aws/internal/service/elbv2	101.484s
=== RUN   TestAccKMSKey_basic
=== PAUSE TestAccKMSKey_basic
=== CONT  TestAccKMSKey_basic
--- PASS: TestAccKMSKey_basic (157.48s)
PASS
ok  	github.com/hashicorp/terraform-provider-aws/internal/service/kms	188.855s
=== RUN   TestAccLambdaFunction_basic
=== PAUSE TestAccLambdaFunction_basic
=== RUN   TestAccLambdaPermission_basic
=== PAUSE TestAccLambdaPermission_basic
=== CONT  TestAccLambdaFunction_basic
=== CONT  TestAccLambdaPermission_basic
--- PASS: TestAccLambdaPermission_basic (132.00s)
--- PASS: TestAccLambdaFunction_basic (141.48s)
PASS
ok  	github.com/hashicorp/terraform-provider-aws/internal/service/lambda	150.709s
=== RUN   TestAccMetaPartitionDataSource_basic
=== PAUSE TestAccMetaPartitionDataSource_basic
=== RUN   TestAccMetaRegionDataSource_basic
=== PAUSE TestAccMetaRegionDataSource_basic
=== RUN   TestAccMetaRegionDataSource_endpoint
=== PAUSE TestAccMetaRegionDataSource_endpoint
=== RUN   TestAccMetaRegionDataSource_endpointAndName
=== PAUSE TestAccMetaRegionDataSource_endpointAndName
=== CONT  TestAccMetaPartitionDataSource_basic
=== CONT  TestAccMetaRegionDataSource_endpoint
=== CONT  TestAccMetaRegionDataSource_endpointAndName
=== CONT  TestAccMetaRegionDataSource_basic
--- PASS: TestAccMetaPartitionDataSource_basic (91.21s)
--- PASS: TestAccMetaRegionDataSource_basic (92.03s)
--- PASS: TestAccMetaRegionDataSource_endpoint (93.28s)
--- PASS: TestAccMetaRegionDataSource_endpointAndName (94.20s)
PASS
ok  	github.com/hashicorp/terraform-provider-aws/internal/service/meta	100.067s
=== RUN   TestAccRoute53Record_basic
=== PAUSE TestAccRoute53Record_basic
=== RUN   TestAccRoute53Record_Latency_basic
=== PAUSE TestAccRoute53Record_Latency_basic
=== RUN   TestAccRoute53ZoneDataSource_name
=== PAUSE TestAccRoute53ZoneDataSource_name
=== CONT  TestAccRoute53Record_basic
=== CONT  TestAccRoute53ZoneDataSource_name
=== CONT  TestAccRoute53Record_Latency_basic
--- PASS: TestAccRoute53ZoneDataSource_name (129.79s)
--- PASS: TestAccRoute53Record_basic (215.59s)
--- PASS: TestAccRoute53Record_Latency_basic (224.43s)
PASS
ok  	github.com/hashicorp/terraform-provider-aws/internal/service/route53	238.887s
=== RUN   TestAccS3BucketACL_updateACL
=== PAUSE TestAccS3BucketACL_updateACL
=== RUN   TestAccS3BucketPolicy_basic
=== PAUSE TestAccS3BucketPolicy_basic
=== RUN   TestAccS3BucketPublicAccessBlock_basic
=== PAUSE TestAccS3BucketPublicAccessBlock_basic
=== RUN   TestAccS3Bucket_Basic_basic
=== PAUSE TestAccS3Bucket_Basic_basic
=== RUN   TestAccS3Bucket_Security_corsUpdate
=== PAUSE TestAccS3Bucket_Security_corsUpdate
=== CONT  TestAccS3BucketACL_updateACL
=== CONT  TestAccS3Bucket_Security_corsUpdate
=== CONT  TestAccS3Bucket_Basic_basic
=== CONT  TestAccS3BucketPublicAccessBlock_basic
=== CONT  TestAccS3BucketPolicy_basic
--- PASS: TestAccS3BucketPolicy_basic (129.34s)
--- PASS: TestAccS3BucketPublicAccessBlock_basic (130.79s)
--- PASS: TestAccS3Bucket_Basic_basic (131.19s)
--- PASS: TestAccS3Bucket_Security_corsUpdate (164.07s)
--- PASS: TestAccS3BucketACL_updateACL (164.62s)
PASS
ok  	github.com/hashicorp/terraform-provider-aws/internal/service/s3	187.585s
=== RUN   TestAccSecretsManagerSecret_basic
=== PAUSE TestAccSecretsManagerSecret_basic
=== RUN   TestAccSecretsManagerSecret_basicReplica
=== PAUSE TestAccSecretsManagerSecret_basicReplica
=== CONT  TestAccSecretsManagerSecret_basic
=== CONT  TestAccSecretsManagerSecret_basicReplica
--- PASS: TestAccSecretsManagerSecret_basic (107.93s)
--- PASS: TestAccSecretsManagerSecret_basicReplica (130.82s)
PASS
ok  	github.com/hashicorp/terraform-provider-aws/internal/service/secretsmanager	166.908s
=== RUN   TestAccSTSCallerIdentityDataSource_basic
=== PAUSE TestAccSTSCallerIdentityDataSource_basic
=== CONT  TestAccSTSCallerIdentityDataSource_basic
--- PASS: TestAccSTSCallerIdentityDataSource_basic (72.11s)
PASS
ok  	github.com/hashicorp/terraform-provider-aws/internal/service/sts	121.472s

@gdavison gdavison merged commit 454633f into main Aug 18, 2023
@gdavison gdavison deleted the f-s3-us-east-1-regional branch August 18, 2023 00:24
@github-actions github-actions bot added this to the v5.13.0 milestone Aug 18, 2023
github-actions bot pushed a commit that referenced this pull request Aug 18, 2023
@github-actions
Copy link

This functionality has been released in v5.13.0 of the Terraform AWS Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you!

@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
client-connections Pertains to the AWS Client and service connections. documentation Introduces or discusses updates to documentation. prioritized Part of the maintainer teams immediate focus. To be addressed within the current quarter. provider Pertains to the provider itself, rather than any interaction with AWS. service/s3 Issues and PRs that pertain to the s3 service. size/M Managed by automation to categorize the size of a PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants