profile
viewpoint

prudhvi/coinbase 1

Coinbase Ruby OAuth2 client

prudhvi/meetup 1

Meetup ruby client

prudhvi/neography 1

A thin Ruby wrapper to the Neo4j Rest API

prudhvi/omniauth-coinbase 1

CoinBase strategy for OmniAuth

prudhvi/attr_accessor_with_init 0

Default free constructor with attr_accessor

prudhvi/bazel-deps 0

Generate bazel dependencies for maven artifacts

prudhvi/benchmarking 0

Benchmarking Scala Micro-Frameworks

prudhvi/bitbucket-codeship-status 0

Small app that will automatically update newly created pull requests in Bitbucket with the branch's Codeship build status.

prudhvi/bitcoin-ruby 0

bitcoin utils and protocol in ruby.

prudhvi/cab_app 0

API for location based cab app

pull request commentstripe/veneur

Disable clientTelemetry for internal statsd client used by Veneur

cc @aditya-stripe | @ChimeraCoder

prudhvi

comment created time in 3 months

push eventprudhvi/veneur

Yoav Srebrnik

commit sha bfe29dd2586276efaf4f575314959f198a0b7572

Add root timestamp and HTTP headers for X-Ray (#790)

view details

Prudhvi Dhulipalla

commit sha 0eecc64b050d83bf05565f18b9f0807db2e5040c

Merge branch 'master' into disable_statsd_telemetry

view details

push time in 3 months

PR opened stripe/veneur

Disable clientTelemetry for internal statsd client used by Veneur

Summary

Disabled dogstatsd client telemetry on the internal statsd client used by Veneur.

Motivation

Fixes https://github.com/stripe/veneur/issues/794 To prevent extra MTS from being emitted about internal statsd client used by Veneur.

Test plan

Tested the branch in our staging environment and also relying on the existing specs .

+4 -3

0 comment

4 changed files

pr created time in 3 months

push eventprudhvi/veneur

Prudhvi Chaitanya Dhulipalla

commit sha 2cb4c51a238f9c625ccd9a55ee0d5b28dba51de2

Disable clientTelemetry for internal statsd client used by Veneur

view details

push time in 3 months

issue commentstripe/veneur

Disable dogstatsd client telemetry

cc @ChimeraCoder, @antifuchs a friendly reminder, Any thoughts on this?

prudhvi

comment created time in 3 months

issue commentstripe/veneur

Disable dogstatsd client telemetry

cc @aditya-stripe

prudhvi

comment created time in 3 months

issue openedstripe/veneur

Disable dogstatsd client telemetry

Hi Team With the upgrade of dogstatsd client used by veneur we noticed a lot of meta-metrics (client Telemetry) being emitted which is a lot of MTS . Wondering if you are willing to accept a change to 1)Disable telemetry or 2)Enable/Disable client telemetry via a config

FYI my WIP https://github.com/stripe/veneur/compare/master...prudhvi:disable_statsd_telemetry?expand=1 Metric look like below

veneur.datadog.dogstatsd.client.events
veneur.datadog.dogstatsd.client.metric_dropped_on_receive
veneur.datadog.dogstatsd.client.service_checks
veneur.datadog.dogstatsd.client.packets_sent
veneur.datadog.dogstatsd.client.packets_dropped
veneur.datadog.dogstatsd.client.packets_dropped_queue
veneur.datadog.dogstatsd.client.bytes_dropped
veneur.datadog.dogstatsd.client.bytes_dropped_queue
veneur.datadog.dogstatsd.client.bytes_dropped_writer
veneur.datadog.dogstatsd.client.bytes_sent
veneur.datadog.dogstatsd.client.packets_dropped_writer
veneur.datadog.dogstatsd.client.metrics	

They might keep adding more. based on https://github.com/DataDog/datadog-go#monitoring-this-client

I understand we could use prefix drops config to not send these to sink, but wondering if you are open to disabling the emission in first place. These weren't present in earlier version of statsd client and it contributed to lot of MTS as this gets emitted for every app veneur sidecar in our infra.

created time in 3 months

push eventprudhvi/veneur

Prudhvi Chaitanya Dhulipalla

commit sha 634152b60602e58b72866a531da7136eb9afc6ec

Disable clientTelemetry for statsd client

view details

push time in 3 months

create barnchprudhvi/veneur

branch : disable_statsd_telemetry

created branch time in 3 months

more