alloy icon indicating copy to clipboard operation
alloy copied to clipboard

feat(otelcol/exporter/datadog) Add Datadog exporter wrapper

Open polyrain opened this issue 9 months ago • 4 comments

PR Description

This PR adds a wrapper around the Datadog Exporter from the opentelemetry-collector-contrib repo. It enables use of the Datadog exporter, which allows Alloy to send telemetry (metrics and traces at this point) to Datadog.

This is useful in migrating a Datadog setup towards a more OSS architecture. By allowing Alloy to talk to Datadog, Alloy can serve a new wedge of users using this proprietary platform and embrace OTel more.

Documentation and more tests need to be added, as well as more scrutiny; I intend on continuing work on this branch to address these over the next few days. This MR has dependency on #711 , hence the large diff 😅

Which issue(s) this PR fixes

Fixes #714

Notes to the Reviewer

PR Checklist

  • [x] CHANGELOG.md updated
  • [x] Documentation added
  • [x] Tests updated
  • [x] Config converters updated

polyrain avatar May 02 '24 07:05 polyrain

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar May 02 '24 07:05 CLAassistant

This PR has not had any activity in the past 30 days, so the needs-attention label has been added to it. If you do not have enough time to follow up on this PR or you think it's no longer relevant, consider closing it. The needs-attention label signals to maintainers that something has fallen through the cracks. No action is needed by you; your PR will be kept open and you do not have to respond to this comment. The label will be removed the next time this job runs if there is new activity. Thank you for your contributions!

github-actions[bot] avatar Jun 03 '24 00:06 github-actions[bot]

@wildum I've tested this locally and it works. I'll get onto some documentation tomorrow, but can you look over the code as it is now so I can get some feedback? I need to update the tests to acknowledge the new defaults for the config still as well 😅

polyrain avatar Jul 20 '24 07:07 polyrain

I've addressed the PR comments and rebased on main, testing locally seems to be working :+1: @wildum can I get another look? I'll add the config converter now

polyrain avatar Aug 16 '24 04:08 polyrain

I've triggered a rebase to pull in the newly added shared community notifcation. The shortcode for the notification is included in a review suggestion. The rendered output looks like this:

image

clayton-cornell avatar Aug 26 '24 17:08 clayton-cornell