Skip to content

Integrate datadog to backend for traces and logs#1299

Merged
beastoin merged 8 commits intomainfrom
integrate-datadog-to-backend-for-traces
Nov 14, 2024
Merged

Integrate datadog to backend for traces and logs#1299
beastoin merged 8 commits intomainfrom
integrate-datadog-to-backend-for-traces

Conversation

@thainguyensunya
Copy link
Copy Markdown
Collaborator

@thainguyensunya thainguyensunya commented Nov 13, 2024

Issue #1217

Key changes:

  • Add ddtrace library
  • Wrap application to datadog serverless-init to collect traces and logs

Example:
Screenshot 2024-11-14 at 11 08 52

#1299 (comment)

@thainguyensunya thainguyensunya changed the title Integrate datadog to backend for traces Integrate datadog to backend for traces and logs Nov 13, 2024
@beastoin
Copy link
Copy Markdown
Collaborator

Great a @thainguyensunya , could you share a image / demo how does it look like with the dev env ?

1/ Could we use Dd as an option for Docker deployment ? You know that we are an opensource, someone don't really need Dd for their backend deployment

@thainguyensunya
Copy link
Copy Markdown
Collaborator Author

Screenshot 2024-11-14 at 10 57 18 AM Screenshot 2024-11-14 at 10 56 37 AM Screenshot 2024-11-14 at 10 58 35 AM Screenshot 2024-11-14 at 10 56 17 AM

@beastoin
Copy link
Copy Markdown
Collaborator

lgtm @thainguyensunya 🥳

@beastoin beastoin merged commit e91a8ef into main Nov 14, 2024
@beastoin beastoin deleted the integrate-datadog-to-backend-for-traces branch November 14, 2024 04:08
@beastoin
Copy link
Copy Markdown
Collaborator

Next, please provide w/ steps to deploy then we push it to production :)

Glucksberg pushed a commit to Glucksberg/omi-local that referenced this pull request Apr 28, 2026
Issue BasedHardware#1217 

**Key changes:**

- Add ddtrace library
- Wrap application to datadog serverless-init to collect traces and logs
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.

2 participants