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

Possibility to add boomerang site (u parameter) to mapper prefix #62

Open
roflmao opened this issue Jun 9, 2015 · 1 comment
Open
Labels

Comments

@roflmao
Copy link

roflmao commented Jun 9, 2015

Is there a way to add a sitename to the prefix from boomerang when sending data to statsd?

@philbooth
Copy link
Contributor

It depends on exactly what information you want to add.

At Nature, we have a requirement to differentiate live/staging/test/dev, which doesn't really lend itself to a generic solution. Instead we use a custom mapper instead that is forked from the statsd one:

https://github.com/nature/boomcatch/blob/master/src/mappers/statsd-npg.js#L80-L94

That may be the correct approach for you.

Alternatively, perhaps we could specify it generically using a new command-line option. But I'm already concerned that the CLI surface area is becoming a monster so we'd have to come up with a very nice way of doing that, possibly in some kind of config file or something.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants