-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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: Authentication updates for Terraform AWS Provider v3.0.0 #14077
Conversation
Reference: #5018 Reference: #6913 Reference: #7333 Reference: #9236 Reference: #9869 Reference: #9898 Reference: #9962 Reference: #9986 Reference: #10507 Reference: #11429 Reference: #12236 Reference: #12727 Reference: #12815 Reference: #13057 Changes: ``` NOTES * provider: Credential ordering has changed from static, environment, shared credentials, EC2 metadata, default AWS Go SDK (shared configuration, web identity, ECS, EC2 Metadata) to static, environment, shared credentials, default AWS Go SDK (shared configuration, web identity, ECS, EC2 Metadata) * provider: The `AWS_METADATA_TIMEOUT` environment variable no longer has any effect as we now depend on the default AWS Go SDK EC2 Metadata client timeout of one second with two retries ENHANCEMENTS * provider: Always enable shared configuration file support (no longer require `AWS_SDK_LOAD_CONFIG` environment variable) * provider: Add `assume_role` configuration block `duration_seconds`, `policy_arns`, `tags`, and `transitive_tag_keys` arguments BUG FIXES * provider: Ensure configured STS endpoint is used during `AssumeRole` API calls * provider: Prefer AWS shared configuration over EC2 metadata credentials by default * provider: Prefer CodeBuild, ECS, EKS credentials over EC2 metadata credentials by default ``` Output from acceptance testing: ``` --- PASS: TestAccAWSProvider_Region_AwsCommercial (3.89s) --- PASS: TestAccAWSProvider_Region_AwsGovCloudUs (3.90s) --- PASS: TestAccAWSProvider_Region_AwsChina (3.99s) --- PASS: TestAccAWSProvider_IgnoreTags_Keys_None (4.22s) --- PASS: TestAccAWSProvider_IgnoreTags_KeyPrefixes_None (4.29s) --- PASS: TestAccAWSProvider_IgnoreTags_KeyPrefixes_One (4.37s) --- PASS: TestAccAWSProvider_IgnoreTags_KeyPrefixes_Multiple (4.38s) --- PASS: TestAccAWSProvider_IgnoreTags_Keys_One (4.39s) --- PASS: TestAccAWSProvider_IgnoreTags_EmptyConfigurationBlock (4.40s) --- PASS: TestAccAWSProvider_IgnoreTags_Keys_Multiple (4.40s) --- PASS: TestAccAWSProvider_Endpoints_Deprecated (4.42s) --- PASS: TestAccAWSProvider_Endpoints (4.53s) --- PASS: TestAccAWSProvider_AssumeRole_Empty (8.32s) ```
…e guide and remove pre-3.0 notes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀
--- PASS: TestAccAWSProvider_Region_AwsCommercial (11.19s)
--- PASS: TestAccAWSProvider_Region_AwsChina (11.05s)
--- PASS: TestAccAWSProvider_Region_AwsGovCloudUs (11.54s)
--- PASS: TestAccAWSProvider_IgnoreTags_KeyPrefixes_One (11.74s)
--- PASS: TestAccAWSProvider_IgnoreTags_Keys_Multiple (11.83s)
--- PASS: TestAccAWSProvider_IgnoreTags_EmptyConfigurationBlock (11.68s)
--- PASS: TestAccAWSProvider_IgnoreTags_KeyPrefixes_None (12.01s)
--- PASS: TestAccAWSProvider_IgnoreTags_Keys_None (12.00s)
--- PASS: TestAccAWSProvider_IgnoreTags_KeyPrefixes_Multiple (12.13s)
--- PASS: TestAccAWSProvider_IgnoreTags_Keys_One (12.01s)
--- PASS: TestAccAWSProvider_Endpoints_Deprecated (12.42s)
--- PASS: TestAccAWSProvider_Endpoints (12.45s)
--- PASS: TestAccAWSProvider_AssumeRole_Empty (24.31s)
This has been released in version 3.0.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 for triage. Thanks! |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks! |
Community Note
Closes #5018
Closes #6913
Closes #7333
Closes #9236
Closes #9869
Closes #9898
Closes #9962
Closes #9986
Closes #10507
Closes #11429
Closes #12236
Closes #12727
Closes #12815
Closes #13057
Release note for CHANGELOG:
Output from acceptance testing: