-
Notifications
You must be signed in to change notification settings - Fork 11
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 lambda-warming scheduled task to Terraform config #714
Comments
On PR #719 there was a discussion (about restricting the "invoke Lambda" permission) in which I said
It seems likely that that same variable could be used to set the target of this rule and avoid adding another circular dependency, so that would make it worth doing. |
Closed via false positive from Github's keyword matching. PR #729 adds a Terraform variable, per the comment above, but didn't succeed in adding the scheduled task. The attempt lives on in a branch, but unless the deployment setup changes significantly (so that the Lambda function is either created by Terraform or preexists the Terraform build) or AWS changes the permissions system to allow IAM-based permissions for CloudWatch to invoke Lambda functions, it won't work. |
As noted on PR #712, the Lambda-warming strategy for Tilegarden requires a scheduled Cloudwatch event. That PR adds a little section to the deployment README about doing it by hand, but it would be much preferable to have Terraform do it. The main complication is that it adds to the circular dependency between the Terraform config and the stuff created by Claudia. In this case, it's the ARN of the Lambda function that would be needed but not exist the first time Terraform is run.
The text was updated successfully, but these errors were encountered: