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_firehose_delivery_stream: Set 'kinesis_source_configuration' in Read function #13536
r/aws_kinesis_firehose_delivery_stream: Set 'kinesis_source_configuration' in Read function #13536
Conversation
…tion' in Read function.
…rce_configuration' and add '_disappears' test.
Changed Redshift cluster to run in-VPC. $ make testacc TEST=./aws/ TESTARGS='-run=TestAccAWSKinesisFirehoseDeliveryStream_RedshiftConfigUpdates'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./aws/ -v -count 1 -parallel 20 -run=TestAccAWSKinesisFirehoseDeliveryStream_RedshiftConfigUpdates -timeout 120m
=== RUN TestAccAWSKinesisFirehoseDeliveryStream_RedshiftConfigUpdates
=== PAUSE TestAccAWSKinesisFirehoseDeliveryStream_RedshiftConfigUpdates
=== CONT TestAccAWSKinesisFirehoseDeliveryStream_RedshiftConfigUpdates
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_RedshiftConfigUpdates (503.48s)
PASS
ok github.com/terraform-providers/terraform-provider-aws/aws 503.517s |
Could anyone take a look of this PR ? |
@ewbankkit who could help with this approve to merge? |
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.
Thanks for this fix @ewbankkit ! Validations, setting of attribute, and tests looks great 👍
Output of acceptance tests:
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_s3KinesisStreamSource (85.94s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_ExtendedS3_DataFormatConversionConfiguration_OpenXJsonSerDe_Empty (97.74s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_s3WithCloudwatchLogging (97.89s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_ExtendedS3_DataFormatConversionConfiguration_OrcSerDe_Empty (98.15s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_s3basic (98.85s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_ExtendedS3_ErrorOutputPrefix (102.76s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_disappears (107.86s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_ExtendedS3_DataFormatConversionConfiguration_ParquetSerDe_Empty (108.90s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_ExtendedS3KmsKeyArn (118.96s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_ExtendedS3_ExternalUpdate (125.20s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_ExtendedS3_ProcessingConfiguration_Empty (127.74s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_basic (130.87s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_ExtendedS3basic (135.62s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_ExtendedS3_DataFormatConversionConfiguration_Enabled (136.33s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_ExtendedS3_DataFormatConversionConfiguration_Serializer_Update (138.41s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_ExtendedS3_DataFormatConversionConfiguration_HiveJsonSerDe_Empty (141.10s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_s3basicWithTags (144.11s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_ExtendedS3_DataFormatConversionConfiguration_Deserializer_Update (145.91s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_ExtendedS3_KinesisStreamSource (84.20s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_s3ConfigUpdates (190.31s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_missingProcessingConfiguration (99.62s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_s3basicWithSSE (202.28s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_ExtendedS3Updates (130.34s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_SplunkConfigUpdates (123.12s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_RedshiftConfigUpdates (475.88s)
--- PASS: TestAccAWSKinesisFirehoseDeliveryStream_ElasticsearchConfigUpdates (854.05s)
This has been released in version 2.65.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 #13530.
Also:
_disappears
acceptance testRelease note for CHANGELOG:
Output from acceptance testing:
The failure is not related to this change - My default VPC has no attached IGW.
I'll open a tech-debt issue to add an acceptance test
PreCheck
for that - #13537.