-
Notifications
You must be signed in to change notification settings - Fork 706
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
Add dhcp collector #493
Add dhcp collector #493
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @breed808! Some minor comments, but looks good as usual :)
Is it a namespaced class, or something more exotic? We do namespaced queries in the process/iis collector: |
It's been a while since I've looked at the WMI class for the scope statistics. Would it be preferable to investigate the class as a separate collector, to keep the Perflib & WMI components separate? |
I think we'd rather have it in a single collector, but it might make sense to split it in different collect-functions as we've done in some places. |
I think most of the issues have been ironed out by now, I just need to investigate the WMI class. Will update when this has been done. |
I've looked into the WMI class by testing the I can see someone else has had this issue but the general advice appears to be "Use the I'm not sure how this can be queried with WMI 😞 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, let's skip the scope statistics for now and it can be added later if we figure out how. Just one minor doc nit, then we I think we're done.
Great work! 🎉
Unfortunately perflib does not expose scope statistics (DhcpServerv4ScopeStatistics), and WMI queries to this class don't appear to be possible with the stackexchange/wmi module.
Add dhcp collector
Unfortunately perflib does not expose scope statistics
(DhcpServerv4ScopeStatistics), and WMI queries to this class don't
appear to be possible with the stackexchange/wmi module.