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.

Sign up for GitHub

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

Jump to bottom

There's a significant bug in dnsmasq stats post 0.2. #27

Closed
darron opened this issue Feb 19, 2016 · 3 comments
Closed

There's a significant bug in dnsmasq stats post 0.2. #27

darron opened this issue Feb 19, 2016 · 3 comments
Labels
bug

Comments

@darron
Copy link
Owner

darron commented Feb 19, 2016

Don't have time to track it down right now - but it doesn't appear to give results that made sense.

The statistics were hundreds of times higher than before - which doesn't seem to be right.

Will be digging into it next week.
@darron darron added the bug label Feb 19, 2016
@darron darron changed the title There's a significant bug post 0.2. There's a significant bug in dnsmasq stats post 0.2. Feb 19, 2016
@darron
Copy link
Owner Author

darron commented Feb 19, 2016

It all looks good in local testing - but it made my staging environment crazy:

hmm

@darron
Copy link
Owner Author

darron commented Feb 19, 2016

It's somewhere in here?

v0.2...master

Or else there's a bug in 0.2 that is radically undercounting DNS. I guess we'll see.

@darron
Copy link
Owner Author

darron commented Feb 24, 2016

Duplicated it today.

It's kind of stupid - but I needed a >= sign in two places. Commit incoming.

@darron darron closed this as completed in 721b2cd Feb 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@darron