Skip to content

Snmp observ lib #1382

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

Merged
merged 70 commits into from
Feb 17, 2025
Merged

Snmp observ lib #1382

merged 70 commits into from
Feb 17, 2025

Conversation

v-zhuravlev
Copy link
Contributor

@v-zhuravlev v-zhuravlev commented Jan 3, 2025

This is universal SNMP dashboard and alerts supporting multiple network vendors.

There are two tiers of dashboards (using layout you can find for Linux or Windows)

  • Fleet to add network overview
  • Overview to give more details about
  • logs dashboard based on syslog received messages from devices.

This observ lib relies on snmp_exporter modules of version 0.28.0 or above. Please see README.md for information which snmp_exporter modules should be used to poll metric groups below:

Currently supported metrics groups:

  • Interfaces stats like unicast,broadcast,multicasts,errors and octets in/out (all devices)
  • CPU (mikrotik,dell,cisco,juniper,dlink,eltex,ubuquiti airos(old))
  • Memory (mikrotik,dell,cisco,juniper,dlink,eltex,ubuquiti airos(old))
  • sensors (temperature etc) (for Cisco and Juniper only for now)
  • FRU status (for cisco , optional row)
  • Fibre channel (for cisco MDS only, optional row)

Note: more vendors will added once they are supported by snmp_exporter. As well as more metric groups will be added to existing vendors.(like adding support for temperature sensors etc).

Alerts:

  • Interfaces, Fibre channel errors, SNMP exporter, reboot, cpu and memory usage are controlled.

Annotations:

  • Dashboards support annotations when relevant alerts start firing as well as critical alerts are found recevied from syslog.

Datalinks:

  • Fleet table and fleet panels supports data links to drill down to instance of intereset.

Examples:
Fleet overview:
0) image
note: transmitted/received in table is empty is due to static counters from emulated devices. They actually work. :)

  1. Cisco c2950:
    image

  2. Cisco 7600:
    image

  3. Mikrotik
    image

  4. Mikrotik 2
    image

@v-zhuravlev v-zhuravlev marked this pull request as ready for review February 13, 2025 12:59
@v-zhuravlev v-zhuravlev requested a review from a team as a code owner February 13, 2025 12:59
Copy link
Member

@Dasomeone Dasomeone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couple comments to address, but overall this looks VERY nice! Fantastic job Vitaly!

@Dasomeone
Copy link
Member

Thanks for addressing the comments @v-zhuravlev, all looks mostly good though I think two comments got hidden by Github (wish that could be disabled...) so if you could address those I'd be happy to approve :)

@v-zhuravlev
Copy link
Contributor Author

v-zhuravlev commented Feb 14, 2025

Thanks for addressing the comments @v-zhuravlev, all looks mostly good though I think two comments got hidden by Github (wish that could be disabled...) so if you could address those I'd be happy to approve :)

  1. New fleet table, now more compact with 5 columns merged:
image Kept job though, as it is not always the case that grouplabels would be identical.
  1. Updated interfaces table, dropped group/instance labels from there as table is intended to be used with single instance(device) only.
image

Copy link
Member

@Dasomeone Dasomeone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome work Vitaly, thank you for addressing all the feedback! LGTM :)

@v-zhuravlev v-zhuravlev merged commit ef841d5 into master Feb 17, 2025
5 checks passed
@v-zhuravlev v-zhuravlev deleted the snmp-observ-lib branch February 17, 2025 08:02
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.

2 participants