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

snmp_profiles: add SwOs and Edgeswitch #739

Merged
merged 5 commits into from
Feb 3, 2024
Merged

snmp_profiles: add SwOs and Edgeswitch #739

merged 5 commits into from
Feb 3, 2024

Conversation

FFHener
Copy link
Contributor

@FFHener FFHener commented Jan 27, 2024

This PR brings monitoring for Mikrotik SwitchOS and Ubiquitys Edgeswitches. In addition it fixes the monitoring of proprietary devices for our gateways.

@FFHener FFHener changed the title Swos: snmp snmp_profiles: add SwOs Jan 28, 2024
@FFHener FFHener changed the title snmp_profiles: add SwOs snmp_profiles: add SwOs and Edgeswitch Jan 29, 2024
@FFHener FFHener force-pushed the swos-snmp branch 2 times, most recently from 1858695 to 7c5c7fd Compare January 29, 2024 13:04
@FFHener FFHener linked an issue Jan 29, 2024 that may be closed by this pull request
@FFHener
Copy link
Contributor Author

FFHener commented Jan 29, 2024

As the grafanapart is independent from this part i moved this part into a new meta issue. We could discuss as well if we should add snmp-monitoring for all switches treewide.

@FFHener FFHener marked this pull request as ready for review January 29, 2024 13:13
@FFHener
Copy link
Contributor Author

FFHener commented Jan 31, 2024

Some SwOS-Switches dont provide temperature data (in the same way?) resulting in this error in logread:
daemon.err collectd[9583]: snmp plugin: OID "iso.3.6.1.4.1.14988.1.1.3.11.0" is undefined (type ASN_NULL)
I personally can live with that right now. When we collected more experience with the monitoring of the different models of switches we could then move over to more individualized snmp-profiles, maybe in combination with #751 (defining switches as hosts and match the corresponding snmp-profile via the model).

@FFHener
Copy link
Contributor Author

FFHener commented Feb 1, 2024

I found another error. At fesev-core collectd reported daemon.err collectd[2546]: uc_update: Value too old: name = fesev-switch/snmp/gauge; value time = 1706777397.464; last cache update = 1706777397.464;. The Switch has the Corerouter as the time server, so this needs to be further investigated.

EDIT: Changing the interval of collecting data for the edgeswitch at etc/collectd/conf.d/bbb-snmp.conf from 30 to 60 seconds solved this problem for now, but i would still like to understand the root cause of this problem.

@FFHener
Copy link
Contributor Author

FFHener commented Feb 3, 2024

The Problem is solved.

TLDR: Names of the value descriptors wherent unique.

I was using the port names as descriptors for the individual names because i thought this would be more helpful in understanding the graphs in grafana (e.g. zwingli-sama helps more than 7). The problem is that port names sometimes get used multiple times e.g. dhcp, intern, ..., but collectd uses these names from my understanding as keys. This problem seems kind of commen as it is mentioned in there docs. After switching the names in TypeInstanceOID to Port Numbers like N or GigabitEthernetN the problem went away. So this PR is ready to be merged now.

@FFHener FFHener force-pushed the swos-snmp branch 2 times, most recently from 0b1dda7 to 5c989f2 Compare February 3, 2024 17:26
@FFHener
Copy link
Contributor Author

FFHener commented Feb 3, 2024

The Error regarding temperature values is fixed as i introduced a new snmp_profile for SwOS Lite.

Got Out of Band approval from @pktpls

@FFHener FFHener merged commit 9a676ab into master Feb 3, 2024
5 checks passed
@FFHener FFHener deleted the swos-snmp branch February 3, 2024 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

snmp: no data gatherd for snmp devices on gateways
1 participant