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

Add Datadog support #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add Datadog support #5

wants to merge 1 commit into from

Conversation

rs
Copy link
Owner

@rs rs commented Mar 4, 2018

@platinummonkey: I ported your Datadog source into the new code structure. The result does not work yet but and perhaps you could help me figuring why:

  • Your implementation used an empty application key, which results into an "invalid application key" error.
  • When both API and app keys are valid, the API returns empty series, I'm not sure why.

@rs rs changed the title Add Datadob support Add Datadog support Mar 4, 2018
@platinummonkey
Copy link

Hey! Sorry, hadn't read my notifications in a while, the dd api client does do some validation on the input. As long as the API/APP Key have the correct lengths, it won't error out, but will also not return data as expected. I just hadn't gotten around to handling that case yet.

  • If you're getting an empty series I suspect that the data may not be readily available on Dd itself? there is some delay between first metric+tag creation and the ability to query for it, though it's a minute or two. Did you have data in your demo account?

@rs
Copy link
Owner Author

rs commented Apr 27, 2018

Yes I have data in the account I'm testing on. The account contains production data.

@platinummonkey
Copy link

@rs can you post a sample query and drop me a ping on the public datadoghq slack channel? Should be able to help figure this out 👍

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