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

nsqd: ability to skip ephemeral topics/channels in statsd output #1346

Merged
merged 1 commit into from May 21, 2021

Conversation

jehiah
Copy link
Member

@jehiah jehiah commented May 20, 2021

When using nsqd with statsd/graphite ephemeral topics and channels can dramatically increase the number of datasets on the graphite side. This creates an option to filter out those from being written to graphite)

I thought this was existing behaviour but discovered this issue due to a new usage pattern that used a dynamic and frequently changing ephemeral channel name. In my case the # was being stripped out of key names and so I had missinterpreted missing graphs in nsqadmin for ephemeral channels as no data in graphite, when it was actually a missmatch between the query and resulting dataset key.

To provide a way to filter out ephemeral topics and channels this adds a new --statsd-exclude-ephemeral argument which defaults to false (existing behavior).

@jehiah jehiah added the feature label May 20, 2021
@jehiah jehiah self-assigned this May 20, 2021
@jehiah jehiah requested review from mreiferson and ploxiln May 20, 2021 17:32
@ploxiln
Copy link
Member

ploxiln commented May 20, 2021

the # was being stripped out of key names and so I had missinterpreted missing graphs in nsqadmin

Is this a bug in nsqd/nsqadmin, should we also try to fix this? (separately would be fine)

@jehiah
Copy link
Member Author

jehiah commented May 21, 2021

the # was being stripped out of key names and so I had missinterpreted missing graphs in nsqadmin

Is this a bug in nsqd/nsqadmin, should we also try to fix this? (separately would be fine)

It looks like this is happening in bitly/statsdaemon (i'm not positive what other statsd implementations do) so it does seem reasonable to match that behavior in nsqadmin by default (i'll do that separately).

https://github.com/bitly/statsdaemon/blob/fc46d9cfe29b674a0c8abc723afaa9370430cdcd/statsdaemon.go#L64-L88

@jehiah jehiah merged commit a493996 into nsqio:master May 21, 2021
@mreiferson mreiferson deleted the statsd_exclude_ephemeral_1346 branch May 23, 2021 16:10
@mreiferson mreiferson changed the title nsqd: configurable to skip ephemeral topics/channels in statsd output nsqd: ability to skip ephemeral topics/channels in statsd output Aug 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants