-
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
r/aws_kinesis_stream: Prevent long retries when quotas would be exceeded #40499
Conversation
…s not exceed the account limit.
…reamName cannot be provided together` errors when the data stream has more than 1000 shards.
…-demand stream would not exceed the AWS account's [data stream quota](https://docs.aws.amazon.com/streams/latest/dev/service-sizes-and-limits.html), preventing the provider from retrying for 1 hour in the case that the quota is exceeded. This change requires the `kinesis:DescribeLimits` IAM permission.
Community NoteVoting for Prioritization
For Submitters
|
% make testacc TESTARGS='-run=TestAccKinesisStream_updateStreamMode' PKG=kinesis ACCTEST_PARALLELISM=1 make: Verifying source code with gofmt... ==> Checking that code complies with gofmt requirements... TF_ACC=1 go1.23.3 test ./internal/service/kinesis/... -v -count 1 -parallel 1 -run=TestAccKinesisStream_updateStreamMode -timeout 360m 2024/12/10 08:48:23 Initializing Terraform AWS Provider... === RUN TestAccKinesisStream_updateStreamModeOnDemandToProvisionedStartWithNone === PAUSE TestAccKinesisStream_updateStreamModeOnDemandToProvisionedStartWithNone === RUN TestAccKinesisStream_updateStreamModeOnDemandToProvisionedStartWithProvisioned === PAUSE TestAccKinesisStream_updateStreamModeOnDemandToProvisionedStartWithProvisioned === RUN TestAccKinesisStream_updateStreamModeProvisionedToOnDemand === PAUSE TestAccKinesisStream_updateStreamModeProvisionedToOnDemand === CONT TestAccKinesisStream_updateStreamModeOnDemandToProvisionedStartWithNone --- PASS: TestAccKinesisStream_updateStreamModeOnDemandToProvisionedStartWithNone (152.21s) === CONT TestAccKinesisStream_updateStreamModeProvisionedToOnDemand --- PASS: TestAccKinesisStream_updateStreamModeProvisionedToOnDemand (62.36s) === CONT TestAccKinesisStream_updateStreamModeOnDemandToProvisionedStartWithProvisioned --- PASS: TestAccKinesisStream_updateStreamModeOnDemandToProvisionedStartWithProvisioned (62.41s) PASS ok github.com/hashicorp/terraform-provider-aws/internal/service/kinesis 282.127s
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 🎉
% make testacc PKG=kinesis TESTS="TestAccKinesisStream_|TestAccKinesisStreamDataSource_"
make: Verifying source code with gofmt...
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go1.23.3 test ./internal/service/kinesis/... -v -count 1 -parallel 20 -run='TestAccKinesisStream_|TestAccKinesisStreamDataSource_' -timeout 360m
2024/12/10 10:25:24 Initializing Terraform AWS Provider...
=== NAME TestAccKinesisStreamDataSource_pagedShards
stream_data_source_test.go:97: skipping tests; shard count (1100) > shard limit quota (500)
--- SKIP: TestAccKinesisStreamDataSource_pagedShards (0.79s)
--- PASS: TestAccKinesisStream_encryptionWithoutKMSKeyThrowsError (44.26s)
--- PASS: TestAccKinesisStream_disappears (54.17s)
--- PASS: TestAccKinesisStream_basic (62.08s)
--- PASS: TestAccKinesisStream_failOnBadStreamCountAndStreamModeCombination (68.59s)
--- PASS: TestAccKinesisStream_basicOnDemand (70.43s)
--- PASS: TestAccKinesisStreamDataSource_encryption (71.43s)
--- PASS: TestAccKinesisStream_enforceConsumerDeletion (71.98s)
--- PASS: TestAccKinesisStream_tags (76.74s)
--- PASS: TestAccKinesisStream_updateStreamModeOnDemandToProvisionedStartWithProvisioned (77.90s)
--- PASS: TestAccKinesisStream_updateStreamModeProvisionedToOnDemand (88.38s)
--- PASS: TestAccKinesisStream_updateKMSKeyID (98.87s)
--- PASS: TestAccKinesisStream_retentionPeriod (103.59s)
--- PASS: TestAccKinesisStreamDataSource_basic (104.98s)
--- PASS: TestAccKinesisStream_shardLevelMetrics (108.09s)
--- PASS: TestAccKinesisStream_encryption (121.26s)
--- PASS: TestAccKinesisStream_updateStreamModeOnDemandToProvisionedStartWithNone (163.60s)
--- PASS: TestAccKinesisStream_createMultipleConcurrentStreams (232.57s)
--- PASS: TestAccKinesisStream_shardCount (768.31s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/kinesis 773.572s
This functionality has been released in v5.81.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! |
Description
shard_count
would not exceed the AWS account's shard quota when the data stream capacity mode isPROVISIONED
.Relations
Closes #40494.
Closes #38094.
Output from Acceptance Testing