-
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
Crash importing firehose delivery stream #3833
Comments
@grahamj its crashing due to a missing
In the meantime, I'll submit a PR that correctly returns the error instead of the panic. |
This PR will provide an error message instead of panic on import ARNs missing |
The fix for the |
This has been released in version 1.12.0 of the AWS provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. |
Oops I guess that was my bad providing an invalid ARN. An improved error message will certainly be helpful next time. 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! |
Terraform Version
Terraform v0.11.4
Affected Resource(s)
Terraform Configuration Files
Panic Output
Command
AWS_PROFILE=XXX terraform import aws_kinesis_firehose_delivery_stream.import arn:aws:firehose:us-east1:XXXXXXXXX:XXXXX
Sorry, I don't have time right now to sanitize the crash.log but if you really need it just shout. Hopefully the above should point you in the right direction.
The text was updated successfully, but these errors were encountered: