Pass consumer_id as a tag in Datadog plugin #2054
Labels
good first issue
Issues that beginners/volunteers can easily help with.
Comments
|
@shashiranjan84 curious, is this handled or addressed in #2394 per chance? |
|
@p0pr0ck5 no, Current changes only allow static tags |
thibaultcha
added
the
good first issue
|
This likely won't be something we will be able to work on in the short term. Community patches and contributions welcome! |
|
Any update on this capability? |
mathematician
added a commit
to mathematician/kong
that referenced
this issue
Apr 6, 2018
- Every request_count metric will include a consumer_id tag if available - Update datadog request_count schema to include consumer_identifier - Update datadog request_count migrations to include consumer_identifier - Update tests to include tag if consumer is used Related issue: Kong#2054
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
A follow up to #998. Inside the Datadog plugin, would it be possible to add an option to place the consumer_id inside a dogstatsd tag rather than the metric name? The advantage is that this makes it easier in Datadog to do things like plot the top 10 API users or worst performing APIs. So an option meaning
kong.<api_name>.request.countwould end up having a dogstatsd tag with the consumer_id.This could easily be extended to all the other metrics, for example adding this tag to
kong.<api_name>.<http_status_code>.countwould mean it could also be broken down by response code. We could even go as far as putting the response code in a tag as well sokong.<api_name>.response.countwould come through tagged with the response code, this would provide similar capability but the pivot would be on response code.The text was updated successfully, but these errors were encountered: