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.

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

Support new aggregator APIs for the event platform #9165

Merged
merged 1 commit into from
Apr 16, 2021

Conversation

djova
Copy link
Contributor

@djova djova commented Apr 14, 2021

What does this PR do?

Follow-up to DataDog/datadog-agent#7735: add the aggregator submit_event_platform_event API as well as two new APIs that build on this to submit dbm-samples and dbm-metrics.

Also add a new default_json_event_encoding to be used for when datadog_checks/base/utils/db/statement_samples.py is removed in #9166

Motivation

Enable checks to use the new aggregator API to submit event platform events.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • PR title must be written as a CHANGELOG entry (see why)
  • Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
  • PR must have changelog/ and integration/ labels attached

@djova djova requested a review from ofek April 14, 2021 19:12
@djova djova marked this pull request as ready for review April 15, 2021 19:40
@djova djova requested review from a team as code owners April 15, 2021 19:40
Follow-up to DataDog/datadog-agent#7735, add the aggregator `submit_event_platform_event` API as well as two new APIs that build on this to submit `dbm-samples` and `dbm-metrics`.
@djova djova force-pushed the djova/add-new-aggregator-event-platform-api branch from 1bdbc39 to 6b51884 Compare April 15, 2021 19:50
@ofek ofek changed the title add new aggregator event platform APIs Support new aggregator APIs for the event platform Apr 15, 2021
@ofek ofek merged commit 10a56dc into master Apr 16, 2021
@ofek ofek deleted the djova/add-new-aggregator-event-platform-api branch April 16, 2021 14:24
github-actions bot pushed a commit that referenced this pull request Apr 16, 2021
Follow-up to DataDog/datadog-agent#7735, add the aggregator `submit_event_platform_event` API as well as two new APIs that build on this to submit `dbm-samples` and `dbm-metrics`. 10a56dc
djova added a commit that referenced this pull request Apr 16, 2021
Follow-up to #9165: Update python & mysql checks to submit DBM events via the new aggregator API.

Also improves the tests to better handle threads.

Motivation: Submit events using the more robust agent go code to with proper batching, buffering, retries, error handling, and tracking of internal statistics.
djova added a commit that referenced this pull request Apr 16, 2021
Follow-up to #9165: Update python & mysql checks to submit DBM events via the new aggregator API.

Also improves the tests to better handle threads.

Motivation: Submit events using the more robust agent go code to with proper batching, buffering, retries, error handling, and tracking of internal statistics.
djova added a commit that referenced this pull request Apr 16, 2021
Follow-up to #9165: Update python & mysql checks to submit DBM events via the new aggregator API.

Also improves the tests to better handle threads.

Motivation: Submit events using the more robust agent go code to with proper batching, buffering, retries, error handling, and tracking of internal statistics.
ofek pushed a commit that referenced this pull request Apr 16, 2021
* update python & mysql checks

Follow-up to #9165: Update python & mysql checks to submit DBM events via the new aggregator API.

Also improves the tests to better handle threads.

Motivation: Submit events using the more robust agent go code to with proper batching, buffering, retries, error handling, and tracking of internal statistics.

* bump datadog-checks-base min version"
djova added a commit that referenced this pull request Apr 19, 2021
Send metrics via the new aggregator API added in #9165
djova added a commit that referenced this pull request Apr 21, 2021
Send metrics via the new aggregator API added in #9165
djova added a commit that referenced this pull request Apr 22, 2021
Send metrics via the new aggregator API added in #9165
djova added a commit that referenced this pull request Apr 23, 2021
Send metrics via the new aggregator API added in #9165
djova added a commit that referenced this pull request Apr 23, 2021
Send metrics via the new aggregator API added in #9165
djova added a commit that referenced this pull request Apr 26, 2021
Send DBM query metrics via the new aggregator API added in #9165.
djova added a commit that referenced this pull request Apr 26, 2021
Send DBM query metrics via the new aggregator API added in #9165.
djova added a commit that referenced this pull request Apr 26, 2021
Send DBM query metrics via the new aggregator API added in #9165.
djova added a commit that referenced this pull request Apr 26, 2021
Send DBM query metrics via the new aggregator API added in #9165.
djova added a commit that referenced this pull request Apr 30, 2021
Send DBM query metrics via the new aggregator API added in #9165.
djova added a commit that referenced this pull request Apr 30, 2021
Send DBM query metrics via the new aggregator API added in #9165.
djova added a commit that referenced this pull request May 7, 2021
Send DBM query metrics via the new aggregator API added in #9165.
ofek pushed a commit that referenced this pull request May 7, 2021
* mysql: send metrics to new intake as events

Send DBM query metrics via the new aggregator API added in #9165.

* lazy resolve hostname

* fix merge
ofek pushed a commit that referenced this pull request May 7, 2021
* postgres: send metrics to new intake as events

Send DBM query metrics via the new aggregator API added in #9165.

* lazy resolve hostname
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants