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
feature: collect sysDescr and sysObjectID for stats to improve os detection #5510
Conversation
Auto-Deploy finished, Test PR at http://5510.ci.librenms.org or https://5510.ci.librenms.org |
@librenms/reviewers Linux systems contain hostname, which I would like to exclude, but for some devices this is valuable data. Others don't contain hostname at all... Examples:
Is it worth trying to exclude hostnames? Any good ideas on how to do so effectively without removing valuable data? |
Loop through the output from dbFetchRows and do
Also, we need to notify users we are turning this on to give them chance to opt out (docs needs updating). Are we planning to turn it on for just 1 week or leave on all the time? |
Do not submit entries with both fields null
Auto-Deploy finished, Test PR at http://5510.ci.librenms.org or https://5510.ci.librenms.org |
Aren't other characters allowed in hostnames other than a-z0-9. ? |
https://tools.ietf.org/html/rfc952 says yes |
Pending user notification... |
Update docs
Auto-Deploy finished, Test PR at http://5510.ci.librenms.org or https://5510.ci.librenms.org |
The inspection completed: 1 new issues |
DO NOT DELETE THIS TEXT
Please note