Skip to content

terraform module for the lambda_function #1

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

Merged
merged 38 commits into from
Apr 27, 2018

Conversation

mrapptambour
Copy link

@mrapptambour mrapptambour commented Jan 17, 2018

module "dd_appender" {
  source     = "github.com/Placester/dd-aws-lambda-functions//Log"
  DD_API_KEY = {}
  role       = ""
  function_name = "my_service_dd_appender"
  metadata = {
    service = "my_service"
  }
}

@@ -30,6 +30,11 @@
"ddsourcecategory": "aws",
}

try:
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aeikenberry thinking of having Terraform map variable impact the default metadata on each of these logs...is this the right python code to support that?

Michael Rapp and others added 27 commits January 17, 2018 12:07
As we're sometimes seeing connections resets, or SSLErrors
as otherwise we see errors in the logs, and this is confusing
If we successfully retry, then it is not an error
[Logs] Improve retry logic to catch exceptions in the socket
now nicely showing 'cloudwatch' as source in Datadog Logs service for Lambda subscription stream
should be awslogS not singular
Add source auto detection for mysql and mariadb in RDS
DanielLanger and others added 9 commits March 13, 2018 10:33
Infrastructure tags for lambda function are lowercase, so we need to do the same here to avoid any case mismatch.
Lowercase on the lambda function name to match infrastructure tags
[Logs] Update endpoint for logs-submitter lambda function
Properties:
Description: Pushes RDS Enhanced metrics to Datadog.
Environment:
Variables:
kmsEncryptedKeys: YOUR_KEY
kmsEncryptedKeys: 'YOUR_KEY'
Copy link

@aeikenberry aeikenberry Apr 27, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mrapptambour

is this right?

@aeikenberry aeikenberry merged commit 5a8187b into master Apr 27, 2018
@aeikenberry aeikenberry deleted the feature/terraform-appender-module branch April 27, 2018 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants