Skip to content
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

terraform crashes when trying to import firehose stream #3006

Closed
hashibot opened this issue Jan 16, 2018 · 3 comments
Closed

terraform crashes when trying to import firehose stream #3006

hashibot opened this issue Jan 16, 2018 · 3 comments
Assignees
Labels
bug Addresses a defect in current functionality. crash Results from or addresses a Terraform crash or kernel panic. service/firehose Issues and PRs that pertain to the firehose service.
Milestone

Comments

@hashibot
Copy link

This issue was originally opened by @gnowxilef as hashicorp/terraform#17100. It was migrated here as a result of the provider split. The original body of the issue is below.


Terraform Version

0.11.2

Crash Output

https://gist.github.com/gnowxilef/2604ff400b49bab4ca5746e572cbc24c

Expected Behavior

import of firehose stream should be successful

Actual Behavior

terraform crashed and burned :(

Steps to Reproduce

  1. AWS_REGION=eu-central-1 AWS_PROFILE=aec1w terraform import aws_kinesis_firehose_delivery_stream.main arn:aws:firehose:eu-central-1:382359729765:deliverystream/syslog2_op2
@hashibot hashibot added bug Addresses a defect in current functionality. crash Results from or addresses a Terraform crash or kernel panic. labels Jan 16, 2018
@bflad bflad added the service/firehose Issues and PRs that pertain to the firehose service. label Jan 16, 2018
@bflad bflad added this to the v1.7.1 milestone Jan 16, 2018
@radeksimko radeksimko self-assigned this Jan 16, 2018
@radeksimko
Copy link
Member

@maxkramerbcgdv
Copy link

I had the same issue, reverting back to 1.6.0 fixed this for me.

drewsonne pushed a commit to drewsonne/terraform-provider-aws that referenced this issue Mar 3, 2018
hashicorp#2970)

* Fix kinesis firehose stream breaking for s3 destinations
- import doesn't work for s3 destination

* Run tests for extended_s3 config

* Fix invalid test config

* Avoid crash on empty redshift's S3BackupDescription

Fixes hashicorp#3006

* Fix flattenProcessingConfiguration

Flattener was previously broken, hence drifts were not detected.
Adding an error check should prevent breaking it in the future.

* Filter out default params in processing configuration

Firehose API automatically sets some default parameters
which we need to filter out to prevent diffs during routine
use of the resource.

* Suppress diff for redshift_configuration.0.password

* Replace string with constant
@ghost
Copy link

ghost commented Apr 8, 2020

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!

@ghost ghost locked and limited conversation to collaborators Apr 8, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Addresses a defect in current functionality. crash Results from or addresses a Terraform crash or kernel panic. service/firehose Issues and PRs that pertain to the firehose service.
Projects
None yet
Development

No branches or pull requests

4 participants