-
Notifications
You must be signed in to change notification settings - Fork 9.2k
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
r/aws_s3_object: Migrate to AWS SDK for Go v2 #33358
Conversation
…ead operations on Amazon S3 Object Tags.
… aren't exposed as nested.
…ite consistency for PUT and DELETE requests of objects.
Community NoteVoting for Prioritization
For Submitters
|
…2`, `checksum_crc32c`, `checksum_sha1` and `checksum_sha256` attributes.
% make testacc TESTARGS='-run=TestAccS3Object_basic' PKG=s3 ==> Checking that code complies with gofmt requirements... TF_ACC=1 go test ./internal/service/s3/... -v -count 1 -parallel 20 -run=TestAccS3Object_basic -timeout 180m === RUN TestAccS3Object_basic === PAUSE TestAccS3Object_basic === CONT TestAccS3Object_basic --- PASS: TestAccS3Object_basic (29.79s) PASS ok github.com/hashicorp/terraform-provider-aws/internal/service/s3 35.537s
% make testacc TESTARGS='-run=TestAccS3Object_checksumAlgorithm' PKG=s3 ==> Checking that code complies with gofmt requirements... TF_ACC=1 go test ./internal/service/s3/... -v -count 1 -parallel 20 -run=TestAccS3Object_checksumAlgorithm -timeout 180m === RUN TestAccS3Object_checksumAlgorithm === PAUSE TestAccS3Object_checksumAlgorithm === CONT TestAccS3Object_checksumAlgorithm --- PASS: TestAccS3Object_checksumAlgorithm (47.84s) PASS ok github.com/hashicorp/terraform-provider-aws/internal/service/s3 53.042s
…_crc32`, `checksum_crc32c`, `checksum_sha1` and `checksum_sha256` attributes.
% make testacc TESTARGS='-run=TestAccS3ObjectCopy_checksumAlgorithm' PKG=s3 ==> Checking that code complies with gofmt requirements... TF_ACC=1 go test ./internal/service/s3/... -v -count 1 -parallel 20 -run=TestAccS3ObjectCopy_checksumAlgorithm -timeout 180m === RUN TestAccS3ObjectCopy_checksumAlgorithm === PAUSE TestAccS3ObjectCopy_checksumAlgorithm === CONT TestAccS3ObjectCopy_checksumAlgorithm --- PASS: TestAccS3ObjectCopy_checksumAlgorithm (48.57s) PASS ok github.com/hashicorp/terraform-provider-aws/internal/service/s3 53.907s
…on error" This reverts commit e2177f1.
% make testacc TESTARGS='-run=TestAccS3Object_' PKG=s3 ACCTEST_PARALLELISM=3 ==> Checking that code complies with gofmt requirements... TF_ACC=1 go test ./internal/service/s3/... -v -count 1 -parallel 3 -run=TestAccS3Object_ -timeout 180m === RUN TestAccS3Object_basic === PAUSE TestAccS3Object_basic === RUN TestAccS3Object_upgradeFromV4 === PAUSE TestAccS3Object_upgradeFromV4 === RUN TestAccS3Object_source === PAUSE TestAccS3Object_source === RUN TestAccS3Object_content === PAUSE TestAccS3Object_content === RUN TestAccS3Object_etagEncryption === PAUSE TestAccS3Object_etagEncryption === RUN TestAccS3Object_contentBase64 === PAUSE TestAccS3Object_contentBase64 === RUN TestAccS3Object_sourceHashTrigger === PAUSE TestAccS3Object_sourceHashTrigger === RUN TestAccS3Object_withContentCharacteristics === PAUSE TestAccS3Object_withContentCharacteristics === RUN TestAccS3Object_nonVersioned === PAUSE TestAccS3Object_nonVersioned === RUN TestAccS3Object_updates === PAUSE TestAccS3Object_updates === RUN TestAccS3Object_updateSameFile === PAUSE TestAccS3Object_updateSameFile === RUN TestAccS3Object_updatesWithVersioning === PAUSE TestAccS3Object_updatesWithVersioning === RUN TestAccS3Object_updatesWithVersioningViaAccessPoint === PAUSE TestAccS3Object_updatesWithVersioningViaAccessPoint === RUN TestAccS3Object_kms === PAUSE TestAccS3Object_kms === RUN TestAccS3Object_sse === PAUSE TestAccS3Object_sse === RUN TestAccS3Object_acl === PAUSE TestAccS3Object_acl === RUN TestAccS3Object_metadata === PAUSE TestAccS3Object_metadata === RUN TestAccS3Object_storageClass === PAUSE TestAccS3Object_storageClass === RUN TestAccS3Object_tags === PAUSE TestAccS3Object_tags === RUN TestAccS3Object_tagsLeadingSingleSlash === PAUSE TestAccS3Object_tagsLeadingSingleSlash === RUN TestAccS3Object_tagsLeadingMultipleSlashes === PAUSE TestAccS3Object_tagsLeadingMultipleSlashes === RUN TestAccS3Object_tagsMultipleSlashes === PAUSE TestAccS3Object_tagsMultipleSlashes === RUN TestAccS3Object_objectLockLegalHoldStartWithNone === PAUSE TestAccS3Object_objectLockLegalHoldStartWithNone === RUN TestAccS3Object_objectLockLegalHoldStartWithOn === PAUSE TestAccS3Object_objectLockLegalHoldStartWithOn === RUN TestAccS3Object_objectLockRetentionStartWithNone === PAUSE TestAccS3Object_objectLockRetentionStartWithNone === RUN TestAccS3Object_objectLockRetentionStartWithSet === PAUSE TestAccS3Object_objectLockRetentionStartWithSet === RUN TestAccS3Object_objectBucketKeyEnabled === PAUSE TestAccS3Object_objectBucketKeyEnabled === RUN TestAccS3Object_bucketBucketKeyEnabled === PAUSE TestAccS3Object_bucketBucketKeyEnabled === RUN TestAccS3Object_defaultBucketSSE === PAUSE TestAccS3Object_defaultBucketSSE === RUN TestAccS3Object_ignoreTags === PAUSE TestAccS3Object_ignoreTags === RUN TestAccS3Object_checksumAlgorithm === PAUSE TestAccS3Object_checksumAlgorithm === RUN TestAccS3Object_keyWithSlashesMigrated === PAUSE TestAccS3Object_keyWithSlashesMigrated === CONT TestAccS3Object_basic === CONT TestAccS3Object_metadata === CONT TestAccS3Object_objectLockRetentionStartWithNone --- PASS: TestAccS3Object_basic (33.76s) === CONT TestAccS3Object_keyWithSlashesMigrated --- PASS: TestAccS3Object_metadata (77.59s) === CONT TestAccS3Object_checksumAlgorithm --- PASS: TestAccS3Object_objectLockRetentionStartWithNone (84.78s) === CONT TestAccS3Object_ignoreTags --- PASS: TestAccS3Object_keyWithSlashesMigrated (88.10s) === CONT TestAccS3Object_defaultBucketSSE --- PASS: TestAccS3Object_checksumAlgorithm (55.77s) === CONT TestAccS3Object_bucketBucketKeyEnabled --- PASS: TestAccS3Object_ignoreTags (60.84s) === CONT TestAccS3Object_objectBucketKeyEnabled --- PASS: TestAccS3Object_defaultBucketSSE (31.46s) === CONT TestAccS3Object_objectLockRetentionStartWithSet --- PASS: TestAccS3Object_bucketBucketKeyEnabled (30.85s) === CONT TestAccS3Object_tagsLeadingMultipleSlashes --- PASS: TestAccS3Object_objectBucketKeyEnabled (29.55s) === CONT TestAccS3Object_objectLockLegalHoldStartWithOn --- PASS: TestAccS3Object_objectLockLegalHoldStartWithOn (57.64s) === CONT TestAccS3Object_objectLockLegalHoldStartWithNone --- PASS: TestAccS3Object_objectLockRetentionStartWithSet (111.16s) === CONT TestAccS3Object_tagsMultipleSlashes --- PASS: TestAccS3Object_tagsLeadingMultipleSlashes (111.68s) === CONT TestAccS3Object_tags --- PASS: TestAccS3Object_objectLockLegalHoldStartWithNone (86.24s) === CONT TestAccS3Object_tagsLeadingSingleSlash --- PASS: TestAccS3Object_tagsMultipleSlashes (114.27s) === CONT TestAccS3Object_updatesWithVersioningViaAccessPoint --- PASS: TestAccS3Object_tags (118.60s) === CONT TestAccS3Object_acl --- PASS: TestAccS3Object_tagsLeadingSingleSlash (115.49s) === CONT TestAccS3Object_sse --- PASS: TestAccS3Object_updatesWithVersioningViaAccessPoint (60.27s) === CONT TestAccS3Object_kms --- PASS: TestAccS3Object_sse (35.29s) === CONT TestAccS3Object_etagEncryption --- PASS: TestAccS3Object_kms (35.48s) === CONT TestAccS3Object_withContentCharacteristics --- PASS: TestAccS3Object_acl (93.05s) === CONT TestAccS3Object_sourceHashTrigger --- PASS: TestAccS3Object_etagEncryption (35.19s) === CONT TestAccS3Object_contentBase64 --- PASS: TestAccS3Object_withContentCharacteristics (30.58s) === CONT TestAccS3Object_source --- PASS: TestAccS3Object_contentBase64 (30.87s) === CONT TestAccS3Object_nonVersioned acctest.go:1604: skipping test; environment variable TF_ACC_ASSUME_ROLE_ARN must be set. Usage: Amazon Resource Name (ARN) of existing IAM Role to assume for testing restricted permissions --- SKIP: TestAccS3Object_nonVersioned (0.00s) === CONT TestAccS3Object_updatesWithVersioning --- PASS: TestAccS3Object_source (35.54s) === CONT TestAccS3Object_updateSameFile --- PASS: TestAccS3Object_sourceHashTrigger (60.89s) === CONT TestAccS3Object_updates --- PASS: TestAccS3Object_updatesWithVersioning (61.58s) === CONT TestAccS3Object_content --- PASS: TestAccS3Object_updateSameFile (59.03s) === CONT TestAccS3Object_storageClass --- PASS: TestAccS3Object_updates (62.77s) === CONT TestAccS3Object_upgradeFromV4 --- PASS: TestAccS3Object_content (33.20s) --- PASS: TestAccS3Object_upgradeFromV4 (83.10s) --- PASS: TestAccS3Object_storageClass (117.16s) PASS ok github.com/hashicorp/terraform-provider-aws/internal/service/s3 722.434s
…EST_PARALLELISM=3 ==> Checking that code complies with gofmt requirements... TF_ACC=1 go test ./internal/service/s3/... -v -count 1 -parallel 3 -run=TestAccS3ObjectDataSource_ -timeout 180m === RUN TestAccS3ObjectDataSource_basic === PAUSE TestAccS3ObjectDataSource_basic === RUN TestAccS3ObjectDataSource_basicViaAccessPoint === PAUSE TestAccS3ObjectDataSource_basicViaAccessPoint === RUN TestAccS3ObjectDataSource_readableBody === PAUSE TestAccS3ObjectDataSource_readableBody === RUN TestAccS3ObjectDataSource_kmsEncrypted === PAUSE TestAccS3ObjectDataSource_kmsEncrypted === RUN TestAccS3ObjectDataSource_bucketKeyEnabled === PAUSE TestAccS3ObjectDataSource_bucketKeyEnabled === RUN TestAccS3ObjectDataSource_allParams === PAUSE TestAccS3ObjectDataSource_allParams === RUN TestAccS3ObjectDataSource_objectLockLegalHoldOff === PAUSE TestAccS3ObjectDataSource_objectLockLegalHoldOff === RUN TestAccS3ObjectDataSource_objectLockLegalHoldOn === PAUSE TestAccS3ObjectDataSource_objectLockLegalHoldOn === RUN TestAccS3ObjectDataSource_leadingSlash === PAUSE TestAccS3ObjectDataSource_leadingSlash === RUN TestAccS3ObjectDataSource_multipleSlashes === PAUSE TestAccS3ObjectDataSource_multipleSlashes === RUN TestAccS3ObjectDataSource_singleSlashAsKey === PAUSE TestAccS3ObjectDataSource_singleSlashAsKey === RUN TestAccS3ObjectDataSource_checksumMode === PAUSE TestAccS3ObjectDataSource_checksumMode === CONT TestAccS3ObjectDataSource_basic === CONT TestAccS3ObjectDataSource_objectLockLegalHoldOff === CONT TestAccS3ObjectDataSource_kmsEncrypted --- PASS: TestAccS3ObjectDataSource_basic (29.02s) === CONT TestAccS3ObjectDataSource_readableBody --- PASS: TestAccS3ObjectDataSource_kmsEncrypted (29.71s) === CONT TestAccS3ObjectDataSource_allParams --- PASS: TestAccS3ObjectDataSource_objectLockLegalHoldOff (32.95s) === CONT TestAccS3ObjectDataSource_bucketKeyEnabled --- PASS: TestAccS3ObjectDataSource_readableBody (28.65s) === CONT TestAccS3ObjectDataSource_basicViaAccessPoint --- PASS: TestAccS3ObjectDataSource_bucketKeyEnabled (29.35s) === CONT TestAccS3ObjectDataSource_multipleSlashes --- PASS: TestAccS3ObjectDataSource_allParams (33.10s) === CONT TestAccS3ObjectDataSource_checksumMode --- PASS: TestAccS3ObjectDataSource_basicViaAccessPoint (29.29s) === CONT TestAccS3ObjectDataSource_singleSlashAsKey --- PASS: TestAccS3ObjectDataSource_checksumMode (28.50s) === CONT TestAccS3ObjectDataSource_leadingSlash --- PASS: TestAccS3ObjectDataSource_singleSlashAsKey (14.84s) === CONT TestAccS3ObjectDataSource_objectLockLegalHoldOn --- PASS: TestAccS3ObjectDataSource_multipleSlashes (49.07s) --- PASS: TestAccS3ObjectDataSource_objectLockLegalHoldOn (31.20s) --- PASS: TestAccS3ObjectDataSource_leadingSlash (47.08s) PASS ok github.com/hashicorp/terraform-provider-aws/internal/service/s3 143.605s
# Conflicts: # go.sum
This functionality has been released in v5.17.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! |
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. |
Description
Migrates the
aws_s3_object
resource and data source to AWS SDK for Go v2.Migrates the
aws_s3_object_copy
resource's Delete functionality to AWS SDK for Go v2.Relations
Closes #28541.
Closes #27370.
Closes #23901.
Relates #33330.
Relates #33304.
Output from Acceptance Testing