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

[exporter/datadog] Add new metrics helper functions based on native APIs #16858

Merged
merged 2 commits into from
Dec 16, 2022

Conversation

songy23
Copy link
Member

@songy23 songy23 commented Dec 12, 2022

Description:

Add internal metrics helper functions that consume or transform metrics into native Datadog API format (github.com/DataDog/datadog-api-client-go) rather than Zorkian format.

Most changes are a copy-and-paste of current functions with modifications to github.com/DataDog/datadog-api-client-go data types. This PR only changes internal functions. No public functions are changed yet.

Link to tracking Issue:

#16776

Testing:

Documentation:

@github-actions github-actions bot added the exporter/datadog Datadog components label Dec 12, 2022
@songy23
Copy link
Member Author

songy23 commented Dec 12, 2022

Needs to wait for #16614 to be merged then rebase this PR.

@runforesight
Copy link

runforesight bot commented Dec 12, 2022

Foresight Summary

    
Major Impacts
Foresight hasn't detected any major impact on your workflows and tests.

View More Details

 build-and-test workflow has finished in 18 minutes 6 seconds (42 minutes 24 seconds less than main branch avg.) and finished at 14th Dec, 2022.


Job Failed Steps Tests
integration-tests N/A  N/A See Details

🔎 See details on Foresight

*You can configure Foresight comments in your organization settings page.

@songy23 songy23 force-pushed the ddog-metrics-helpers-2 branch from bc3c003 to 70fe7b1 Compare December 13, 2022 15:13
@songy23 songy23 changed the title [WIP] [exporter/datadog] Add new metrics helper functions based on native APIs [exporter/datadog] Add new metrics helper functions based on native APIs Dec 13, 2022
@songy23 songy23 marked this pull request as ready for review December 13, 2022 15:54
@songy23 songy23 requested a review from a team December 13, 2022 15:54
@songy23 songy23 requested a review from mx-psi as a code owner December 13, 2022 15:54
@songy23
Copy link
Member Author

songy23 commented Dec 13, 2022

Please add label Skip Changelog.

@mx-psi mx-psi added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Dec 13, 2022
exporter/datadogexporter/internal/metrics/utils.go Outdated Show resolved Hide resolved
@mx-psi mx-psi merged commit 13e096b into open-telemetry:main Dec 16, 2022
@songy23
Copy link
Member Author

songy23 commented Dec 16, 2022

Thanks for the review!

@songy23 songy23 deleted the ddog-metrics-helpers-2 branch December 16, 2022 16:45
@plantfansam plantfansam mentioned this pull request Jul 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exporter/datadog Datadog components Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants