-
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
importing an S3 bucket fails on "Error importing AWS S3 bucket policy" #9508
Comments
I've just encountered the same issue on terraform v0.12.18, aws provider v92.43.0, but was able to import successfully after replacing variables in provider configuration by static values (basically hardcoded account id just for import). I suspect it might be related to how terraform is resolving variables during import, something like hashicorp/terraform#21662 |
Ran into this.
|
Reference: #394 Reference: #9001 Reference: #9508 Reference: #12805 Output from acceptance testing: ``` --- PASS: TestAccAWSS3Bucket_acceleration (70.53s) --- PASS: TestAccAWSS3Bucket_AclToGrant (64.37s) --- PASS: TestAccAWSS3Bucket_basic (37.90s) --- PASS: TestAccAWSS3Bucket_Bucket_EmptyString (39.08s) --- PASS: TestAccAWSS3Bucket_Cors_Delete (32.28s) --- PASS: TestAccAWSS3Bucket_Cors_EmptyOrigin (39.25s) --- PASS: TestAccAWSS3Bucket_Cors_Update (68.80s) --- PASS: TestAccAWSS3Bucket_disableDefaultEncryption_whenDefaultEncryptionIsEnabled (67.23s) --- PASS: TestAccAWSS3Bucket_enableDefaultEncryption_whenAES256IsUsed (37.19s) --- PASS: TestAccAWSS3Bucket_enableDefaultEncryption_whenTypical (44.32s) --- PASS: TestAccAWSS3Bucket_forceDestroy (37.21s) --- PASS: TestAccAWSS3Bucket_forceDestroyWithEmptyPrefixes (38.50s) --- PASS: TestAccAWSS3Bucket_forceDestroyWithObjectLockEnabled (37.77s) --- PASS: TestAccAWSS3Bucket_generatedName (38.80s) --- PASS: TestAccAWSS3Bucket_GrantToAcl (60.31s) --- PASS: TestAccAWSS3Bucket_LifecycleBasic (89.67s) --- PASS: TestAccAWSS3Bucket_LifecycleExpireMarkerOnly (67.52s) --- PASS: TestAccAWSS3Bucket_LifecycleRule_Expiration_EmptyConfigurationBlock (30.08s) --- PASS: TestAccAWSS3Bucket_Logging (56.73s) --- PASS: TestAccAWSS3Bucket_namePrefix (40.92s) --- PASS: TestAccAWSS3Bucket_objectLock (68.34s) --- PASS: TestAccAWSS3Bucket_Policy (97.07s) --- PASS: TestAccAWSS3Bucket_region (34.45s) --- PASS: TestAccAWSS3Bucket_Replication (159.22s) --- PASS: TestAccAWSS3Bucket_ReplicationConfiguration_Rule_Destination_AccessControlTranslation (94.18s) --- PASS: TestAccAWSS3Bucket_ReplicationConfiguration_Rule_Destination_AddAccessControlTranslation (95.79s) --- PASS: TestAccAWSS3Bucket_ReplicationExpectVersioningValidationError (28.62s) --- PASS: TestAccAWSS3Bucket_ReplicationSchemaV2 (167.50s) --- PASS: TestAccAWSS3Bucket_ReplicationWithoutPrefix (55.52s) --- PASS: TestAccAWSS3Bucket_ReplicationWithoutStorageClass (58.02s) --- PASS: TestAccAWSS3Bucket_RequestPayer (67.28s) --- PASS: TestAccAWSS3Bucket_shouldFailNotFound (19.65s) --- PASS: TestAccAWSS3Bucket_tagsWithNoSystemTags (119.32s) --- PASS: TestAccAWSS3Bucket_tagsWithSystemTags (171.42s) --- PASS: TestAccAWSS3Bucket_UpdateAcl (65.51s) --- PASS: TestAccAWSS3Bucket_UpdateGrant (92.38s) --- PASS: TestAccAWSS3Bucket_Versioning (95.55s) --- PASS: TestAccAWSS3Bucket_Website_Simple (95.12s) --- PASS: TestAccAWSS3Bucket_WebsiteRedirect (91.21s) --- PASS: TestAccAWSS3Bucket_WebsiteRoutingRules (65.48s) ```
…14121) Reference: #394 Reference: #9001 Reference: #9508 Reference: #12805 Output from acceptance testing: ``` --- PASS: TestAccAWSS3Bucket_acceleration (70.53s) --- PASS: TestAccAWSS3Bucket_AclToGrant (64.37s) --- PASS: TestAccAWSS3Bucket_basic (37.90s) --- PASS: TestAccAWSS3Bucket_Bucket_EmptyString (39.08s) --- PASS: TestAccAWSS3Bucket_Cors_Delete (32.28s) --- PASS: TestAccAWSS3Bucket_Cors_EmptyOrigin (39.25s) --- PASS: TestAccAWSS3Bucket_Cors_Update (68.80s) --- PASS: TestAccAWSS3Bucket_disableDefaultEncryption_whenDefaultEncryptionIsEnabled (67.23s) --- PASS: TestAccAWSS3Bucket_enableDefaultEncryption_whenAES256IsUsed (37.19s) --- PASS: TestAccAWSS3Bucket_enableDefaultEncryption_whenTypical (44.32s) --- PASS: TestAccAWSS3Bucket_forceDestroy (37.21s) --- PASS: TestAccAWSS3Bucket_forceDestroyWithEmptyPrefixes (38.50s) --- PASS: TestAccAWSS3Bucket_forceDestroyWithObjectLockEnabled (37.77s) --- PASS: TestAccAWSS3Bucket_generatedName (38.80s) --- PASS: TestAccAWSS3Bucket_GrantToAcl (60.31s) --- PASS: TestAccAWSS3Bucket_LifecycleBasic (89.67s) --- PASS: TestAccAWSS3Bucket_LifecycleExpireMarkerOnly (67.52s) --- PASS: TestAccAWSS3Bucket_LifecycleRule_Expiration_EmptyConfigurationBlock (30.08s) --- PASS: TestAccAWSS3Bucket_Logging (56.73s) --- PASS: TestAccAWSS3Bucket_namePrefix (40.92s) --- PASS: TestAccAWSS3Bucket_objectLock (68.34s) --- PASS: TestAccAWSS3Bucket_Policy (97.07s) --- PASS: TestAccAWSS3Bucket_region (34.45s) --- PASS: TestAccAWSS3Bucket_Replication (159.22s) --- PASS: TestAccAWSS3Bucket_ReplicationConfiguration_Rule_Destination_AccessControlTranslation (94.18s) --- PASS: TestAccAWSS3Bucket_ReplicationConfiguration_Rule_Destination_AddAccessControlTranslation (95.79s) --- PASS: TestAccAWSS3Bucket_ReplicationExpectVersioningValidationError (28.62s) --- PASS: TestAccAWSS3Bucket_ReplicationSchemaV2 (167.50s) --- PASS: TestAccAWSS3Bucket_ReplicationWithoutPrefix (55.52s) --- PASS: TestAccAWSS3Bucket_ReplicationWithoutStorageClass (58.02s) --- PASS: TestAccAWSS3Bucket_RequestPayer (67.28s) --- PASS: TestAccAWSS3Bucket_shouldFailNotFound (19.65s) --- PASS: TestAccAWSS3Bucket_tagsWithNoSystemTags (119.32s) --- PASS: TestAccAWSS3Bucket_tagsWithSystemTags (171.42s) --- PASS: TestAccAWSS3Bucket_UpdateAcl (65.51s) --- PASS: TestAccAWSS3Bucket_UpdateGrant (92.38s) --- PASS: TestAccAWSS3Bucket_Versioning (95.55s) --- PASS: TestAccAWSS3Bucket_Website_Simple (95.12s) --- PASS: TestAccAWSS3Bucket_WebsiteRedirect (91.21s) --- PASS: TestAccAWSS3Bucket_WebsiteRoutingRules (65.48s) ```
The removal of the automatic |
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
Terraform Version
v0.12.5
Affected Resource(s)
Terraform Configuration Files
I have not defined a policy in the configuration, and there is no policy attached to the bucket.
Note that this bucket is in a subaccount than the one we typically use. To handle this, we map our provider account id to terraform workspace, like so:
This has been working for us for a variety of configurations, this is the first time we've encountered this issue.
Debug Output
Expected Behavior
The S3 bucket should have been imported into the state.
Actual Behavior
The bucket failed to import, producing the error message mentioned in the debug output.
I suspect what's happening is that the aws provider is correctly assuming role to import the bucket, but doesn't assume role to get the policy, resulting in the 'AccessDenied'.
Steps to Reproduce
assume_role
.terraform import
Important Factoids
We're using a provider with
assume_role
to import the resource from a subaccount.The text was updated successfully, but these errors were encountered: