r/aws_ebs_volume: Only specify throughput on update for gp3 volumes

[ewbankkit]

Community Note

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

Closes #17644.

Output from acceptance testing:

$ make testacc TEST=./aws/ TESTARGS='-run=TestAccAWSEBSVolume_'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./aws -v -count 1 -parallel 20 -run=TestAccAWSEBSVolume_ -timeout 120m
=== RUN   TestAccAWSEBSVolume_basic
=== PAUSE TestAccAWSEBSVolume_basic
=== RUN   TestAccAWSEBSVolume_updateAttachedEbsVolume
=== PAUSE TestAccAWSEBSVolume_updateAttachedEbsVolume
=== RUN   TestAccAWSEBSVolume_updateSize
=== PAUSE TestAccAWSEBSVolume_updateSize
=== RUN   TestAccAWSEBSVolume_updateType
=== PAUSE TestAccAWSEBSVolume_updateType
=== RUN   TestAccAWSEBSVolume_updateIops_Io1
=== PAUSE TestAccAWSEBSVolume_updateIops_Io1
=== RUN   TestAccAWSEBSVolume_updateIops_Io2
=== PAUSE TestAccAWSEBSVolume_updateIops_Io2
=== RUN   TestAccAWSEBSVolume_kmsKey
=== PAUSE TestAccAWSEBSVolume_kmsKey
=== RUN   TestAccAWSEBSVolume_NoIops
=== PAUSE TestAccAWSEBSVolume_NoIops
=== RUN   TestAccAWSEBSVolume_InvalidIopsForType
=== PAUSE TestAccAWSEBSVolume_InvalidIopsForType
=== RUN   TestAccAWSEBSVolume_InvalidThroughputForType
=== PAUSE TestAccAWSEBSVolume_InvalidThroughputForType
=== RUN   TestAccAWSEBSVolume_withTags
=== PAUSE TestAccAWSEBSVolume_withTags
=== RUN   TestAccAWSEBSVolume_multiAttach
=== PAUSE TestAccAWSEBSVolume_multiAttach
=== RUN   TestAccAWSEBSVolume_outpost
=== PAUSE TestAccAWSEBSVolume_outpost
=== RUN   TestAccAWSEBSVolume_gp3_basic
=== PAUSE TestAccAWSEBSVolume_gp3_basic
=== RUN   TestAccAWSEBSVolume_gp3_iops
=== PAUSE TestAccAWSEBSVolume_gp3_iops
=== RUN   TestAccAWSEBSVolume_gp3_throughput
=== PAUSE TestAccAWSEBSVolume_gp3_throughput
=== RUN   TestAccAWSEBSVolume_gp3_to_gp2
=== PAUSE TestAccAWSEBSVolume_gp3_to_gp2
=== RUN   TestAccAWSEBSVolume_snapshotID
=== PAUSE TestAccAWSEBSVolume_snapshotID
=== RUN   TestAccAWSEBSVolume_snapshotIDAndSize
=== PAUSE TestAccAWSEBSVolume_snapshotIDAndSize
=== RUN   TestAccAWSEBSVolume_disappears
=== PAUSE TestAccAWSEBSVolume_disappears
=== CONT  TestAccAWSEBSVolume_basic
=== CONT  TestAccAWSEBSVolume_disappears
=== CONT  TestAccAWSEBSVolume_snapshotIDAndSize
=== CONT  TestAccAWSEBSVolume_snapshotID
=== CONT  TestAccAWSEBSVolume_gp3_to_gp2
=== CONT  TestAccAWSEBSVolume_gp3_throughput
=== CONT  TestAccAWSEBSVolume_gp3_iops
=== CONT  TestAccAWSEBSVolume_gp3_basic
=== CONT  TestAccAWSEBSVolume_outpost
=== CONT  TestAccAWSEBSVolume_multiAttach
=== CONT  TestAccAWSEBSVolume_withTags
=== CONT  TestAccAWSEBSVolume_InvalidThroughputForType
=== CONT  TestAccAWSEBSVolume_InvalidIopsForType
=== CONT  TestAccAWSEBSVolume_NoIops
=== CONT  TestAccAWSEBSVolume_kmsKey
=== CONT  TestAccAWSEBSVolume_updateIops_Io2
=== CONT  TestAccAWSEBSVolume_updateIops_Io1
=== CONT  TestAccAWSEBSVolume_updateType
=== CONT  TestAccAWSEBSVolume_updateSize
=== CONT  TestAccAWSEBSVolume_updateAttachedEbsVolume
=== CONT  TestAccAWSEBSVolume_outpost
    data_source_aws_outposts_outposts_test.go:66: skipping since no Outposts found
--- SKIP: TestAccAWSEBSVolume_outpost (1.29s)
--- PASS: TestAccAWSEBSVolume_InvalidIopsForType (25.31s)
--- PASS: TestAccAWSEBSVolume_InvalidThroughputForType (25.46s)
--- PASS: TestAccAWSEBSVolume_disappears (64.80s)
--- PASS: TestAccAWSEBSVolume_NoIops (74.88s)
--- PASS: TestAccAWSEBSVolume_basic (75.05s)
--- PASS: TestAccAWSEBSVolume_gp3_basic (81.10s)
--- PASS: TestAccAWSEBSVolume_withTags (81.35s)
--- PASS: TestAccAWSEBSVolume_multiAttach (81.90s)
--- PASS: TestAccAWSEBSVolume_kmsKey (82.81s)
--- PASS: TestAccAWSEBSVolume_snapshotID (89.43s)
--- PASS: TestAccAWSEBSVolume_snapshotIDAndSize (93.87s)
--- PASS: TestAccAWSEBSVolume_updateIops_Io1 (110.84s)
--- PASS: TestAccAWSEBSVolume_gp3_throughput (116.39s)
--- PASS: TestAccAWSEBSVolume_gp3_to_gp2 (116.82s)
--- PASS: TestAccAWSEBSVolume_updateSize (118.95s)
--- PASS: TestAccAWSEBSVolume_updateType (119.42s)
--- PASS: TestAccAWSEBSVolume_gp3_iops (119.51s)
--- PASS: TestAccAWSEBSVolume_updateIops_Io2 (119.88s)
--- PASS: TestAccAWSEBSVolume_updateAttachedEbsVolume (216.09s)
PASS
ok  	github.com/terraform-providers/terraform-provider-aws/aws	216.236s

[YakDriver]

Looks awesome! 🎉

Acceptance tests on GovCloud:

--- PASS: TestAccAWSEBSVolume_basic (38.00s)
--- PASS: TestAccAWSEBSVolume_disappears (31.56s)
--- PASS: TestAccAWSEBSVolume_gp3_basic (38.38s)
--- PASS: TestAccAWSEBSVolume_gp3_iops (66.42s)
--- PASS: TestAccAWSEBSVolume_gp3_throughput (67.50s)
--- PASS: TestAccAWSEBSVolume_gp3_to_gp2 (65.18s)
--- PASS: TestAccAWSEBSVolume_InvalidIopsForType (5.37s)
--- PASS: TestAccAWSEBSVolume_InvalidThroughputForType (5.34s)
--- PASS: TestAccAWSEBSVolume_kmsKey (41.72s)
--- PASS: TestAccAWSEBSVolume_NoIops (36.14s)
--- PASS: TestAccAWSEBSVolume_snapshotID (70.39s)
--- PASS: TestAccAWSEBSVolume_snapshotIDAndSize (70.44s)
--- PASS: TestAccAWSEBSVolume_updateAttachedEbsVolume (197.57s)
--- PASS: TestAccAWSEBSVolume_updateIops_Io1 (67.49s)
--- PASS: TestAccAWSEBSVolume_updateSize (67.67s)
--- PASS: TestAccAWSEBSVolume_updateType (67.57s)
--- PASS: TestAccAWSEBSVolume_withTags (35.60s)
--- SKIP: TestAccAWSEBSVolume_multiAttach (12.07s)
--- SKIP: TestAccAWSEBSVolume_outpost (1.94s)
--- SKIP: TestAccAWSEBSVolume_updateIops_Io2 (11.70s)

Acceptance tests on us-west-2:

--- PASS: TestAccAWSEBSVolume_basic (38.08s)
--- PASS: TestAccAWSEBSVolume_disappears (33.61s)
--- PASS: TestAccAWSEBSVolume_gp3_basic (38.17s)
--- PASS: TestAccAWSEBSVolume_gp3_iops (60.05s)
--- PASS: TestAccAWSEBSVolume_gp3_throughput (60.13s)
--- PASS: TestAccAWSEBSVolume_gp3_to_gp2 (61.17s)
--- PASS: TestAccAWSEBSVolume_InvalidIopsForType (6.15s)
--- PASS: TestAccAWSEBSVolume_InvalidThroughputForType (6.15s)
--- PASS: TestAccAWSEBSVolume_kmsKey (40.95s)
--- PASS: TestAccAWSEBSVolume_multiAttach (38.30s)
--- PASS: TestAccAWSEBSVolume_NoIops (35.40s)
--- PASS: TestAccAWSEBSVolume_snapshotID (64.97s)
--- PASS: TestAccAWSEBSVolume_snapshotIDAndSize (65.29s)
--- PASS: TestAccAWSEBSVolume_updateIops_Io1 (59.01s)
--- PASS: TestAccAWSEBSVolume_updateIops_Io2 (59.33s)
--- PASS: TestAccAWSEBSVolume_updateSize (60.10s)
--- PASS: TestAccAWSEBSVolume_updateType (61.54s)
--- PASS: TestAccAWSEBSVolume_withTags (37.94s)
--- PASS: TestAccAWSEBSVolume_updateAttachedEbsVolume (236.55s)
--- SKIP: TestAccAWSEBSVolume_outpost (1.42s)

[ghost]

This has been released in version 3.29.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!

[ghost]

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!