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

Add Kinesis streams as Lambda event sources #3897

Closed
nickwareing opened this issue Nov 12, 2015 · 9 comments
Closed

Add Kinesis streams as Lambda event sources #3897

nickwareing opened this issue Nov 12, 2015 · 9 comments

Comments

@nickwareing
Copy link

Feature request to be able to do this in Terraform:

aws lambda

Instructions for doing this via the AWS CLI are in step 3.2 of the docs here: http://docs.aws.amazon.com/lambda/latest/dg/wt-kinesis-configure-kinesis.html

@mrwilby
Copy link

mrwilby commented Nov 12, 2015

+1

@nickwareing
Copy link
Author

My current workaroud is to use local-exec. eg.

resource "aws_lambda_function" "my-lambda" {
    .....
    provisioner "local-exec" {
      command = "aws lambda create-event-source-mapping --function-name ${self.function_name} --event-source  ${aws_kinesis_stream.kinesis-mystream.arn} --batch-size 500 --starting-position TRIM_HORIZON"
    }
}

@vancluever
Copy link
Contributor

+1 and thanks for the workaround @nick-nz!

@vancluever
Copy link
Contributor

I have submitted the above referenced issue (#4089) as the event mapping workaround here seems to not work if IAM access exists between the role and the Lambda function. It bails saying that the mapping is a duplicate, and upon inspection on the console, the mapping is there, albeit disabled with an error.

@vancluever
Copy link
Contributor

Correction to the above, it might be a stale mapping due to the need for a workaround. Check out the issue for more details.

@vancluever
Copy link
Contributor

Submitted PR to add aws_lambda_event_source_mapping (referenced back to here)

@jen20
Copy link
Contributor

jen20 commented Dec 1, 2015

#4093 has now been merged, so I'll go ahead and close this issue. It will be in the 0.6.8 release of Terraform. Thanks for bringing this up @nick-nz, and implementing it @vancluever!

@jen20 jen20 closed this as completed Dec 1, 2015
@nickwareing
Copy link
Author

Awesome. Thanks @vancluever!

omeid pushed a commit to omeid/terraform that referenced this issue Mar 30, 2018
…scaling_group-az-vpc-diffs

docs/resource/aws_autoscaling_group: Clarify usage of availability_zones
@ghost
Copy link

ghost commented Apr 29, 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 have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 29, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants