search

Betterment / test_track

Server app for the TestTrack multi-platform split-testing and feature-gating system
MIT License
119 stars 33 forks source link

Adds ddtrace gem and configures DataDog #93

Closed agirlnamedsophia closed 5 years ago

agirlnamedsophia commented 5 years ago

Summary

This adds support for a DataDog integration. It configures the default process names as well so that they will come through as test_track, test_track-http, test_track-active-record, and test_track-delayed_job.

Datadog will only be configured if the environment variable DATADOG_ENABLED is set to true.

/domain @Betterment/test_track_core /platform @jmileham @samandmoore

nanda-prbot commented 5 years ago

@agirlnamedsophia needs to request domain and platform reviewers, or explicitly opt out, e.g.:

/no-platform

jmileham commented 5 years ago

<< domain lgtm from a testtrack standpoint. This seems like a good turnkey config if people want it, and the gem heft can't be that bad regardless.

nanda-prbot commented 5 years ago

Needs somebody from @jmileham and @samandmoore to claim platform review

Use the shovel operator to claim, e.g.:

@myname << domain && platform

jmileham commented 5 years ago

(leaving platform for somebody DD-aware)

samandmoore commented 5 years ago

<<platformlgtm

nanda-prbot commented 5 years ago

Approved! :guitar: :zap: :dizzy: