-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
nsqadmin: use go:embed; match graphite key valid charset #1347
nsqadmin: use go:embed; match graphite key valid charset #1347
Conversation
interesting related PR for original statsd: statsd/statsd#155 |
77f3e75
to
7799f53
Compare
OK, this also addresses #1317 |
heh, now the build pipeline requires the previously temporary |
also, lost "debug mode" from |
Yeah, I think we should. It shouldn't be too much, in fact it should be equivalent to |
OK, think that should do it, PTAL! |
6179931
to
b20f230
Compare
This updates
nsqadmin
to query graphite using the correct key names for ephemeral topics and channels (see #1346)