DataDog / datadog-lambda-go

The Datadog AWS Lambda package for Go
Apache License 2.0
59 stars 40 forks source link

feat: enable universal instrumentation by default #151

Closed duncanista closed 11 months ago

duncanista commented 11 months ago

What does this PR do?

Enabled universal instrumentation and tracing by default.

Honors environment variable if set.

Screenshot 2023-12-07 at 4 33 14 PM

Motivation

Multiple tickets of customers not understanding why they need to have another env var to get inferred spans ready.

Testing Guidelines

Additional Notes

Types of changes

Checklist

codecov-commenter commented 11 months ago

Codecov Report

Attention: 22 lines in your changes are missing coverage. Please review.

Comparison is base (e8e55b8) 83.37% compared to head (0965f08) 82.12%. Report is 6 commits behind head on main.

:exclamation: Current head 0965f08 differs from pull request most recent head e964357. Consider uploading reports for the commit e964357 to get more accurate results

Files Patch % Lines
ddlambda.go 36.66% 18 Missing and 1 partial :warning:
internal/trace/listener.go 50.00% 3 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #151 +/- ## ========================================== - Coverage 83.37% 82.12% -1.25% ========================================== Files 13 13 Lines 848 884 +36 ========================================== + Hits 707 726 +19 - Misses 112 129 +17 Partials 29 29 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.