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.

Sign up for GitHub

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

Jump to bottom

Datadog config optimization and fixes #201

Merged
merged 2 commits into from
May 18, 2024
Merged

Datadog config optimization and fixes #201

merged 2 commits into from
May 18, 2024

Conversation

devinmatte
Copy link
Member

@devinmatte devinmatte commented May 17, 2024 •
edited
Loading

Motivation

We were using too much CPU just on monitoring. Using less features to lower overhead/costs

Changes

  • Reduce datadog config usage
  • Add dd-trace to python code
  • Fix port clash

Testing Instructions

@github-actions github-actions bot added the ci/cd Affects or Updates CI/CD label May 17, 2024
@devinmatte devinmatte changed the title We don't need that much datadog, it was too much Datadog config optimization and fixes May 17, 2024
@devinmatte devinmatte merged commit 8fecca3 into transitmatters:main May 18, 2024
3 checks passed
@devinmatte devinmatte deleted the datadog-less branch May 18, 2024 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nathan-weinberg nathan-weinberg nathan-weinberg approved these changes

@idreyn idreyn Awaiting requested review from idreyn idreyn is a code owner

@mathcolo mathcolo Awaiting requested review from mathcolo mathcolo is a code owner

Assignees
No one assigned
Labels
ci/cd Affects or Updates CI/CD
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@devinmatte @nathan-weinberg