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

Treewide: Add snmp monitoring for some switches #760

Merged
merged 24 commits into from
Feb 8, 2024
Merged

Treewide: Add snmp monitoring for some switches #760

merged 24 commits into from
Feb 8, 2024

Conversation

FFHener
Copy link
Contributor

@FFHener FFHener commented Feb 5, 2024

This PR adds Monitoring for some switches and several new profiles to archive this. These changes got already deployed at the gateways. Grafana/cgp arent yet at the state of showing all of these values in a beautifull way but this shouldnt stop us from slowly rolling out more data collection from the switches.

Although i didnt touched it Im unhappy about the current airos_6 Situation. When no Station is connected to a AirOS 6 AP the snmp-call for the name of the connected stations obviously cant work resulting in a NoSuchName-Error in our logs. This means that we get at e.g. chris-core every 30 seconds 30 errors. We should address this in the future. #766

TODOs:

  • Add Profile for tplink
  • Add Profile for edgerouter
  • Add + Deploy Monitoring for ak36
  • Add + Deploy Monitoring for l105
  • Add + Deploy Monitoring for ohlauer
  • Add + Deploy Monitoring for saarbruecker
    • Remove saarbruecker-sama monitoring from sama
    • Add profile for Gigabeam LR and add monitoring (also on segen) use airos_8 profile as the missing data doesnt get reported. We get zeros as a reply which is handable.
  • Add + Deploy Monitoring for strom
  • Investigate the NoSuchName Error on airos_6 devices -> Solution collectd: disable logging for locations with AirOS 6 snmp_profiles #766
    • Investigate if we can compress this error
    • If not write a comment
  • Check Scales -> need to be fixed somewhere else. The wrong scale only happens at Grafana. This is because the script feeding prometheus doesn't hand over the scale Information's if i understood rtznmpfl correctly
  • Make shure that snmp is correctly configured on all hosts gateway devices

@FFHener FFHener force-pushed the snmp-switches branch 2 times, most recently from 85b2815 to b00c7d3 Compare February 6, 2024 23:52
@FFHener FFHener marked this pull request as ready for review February 7, 2024 00:15
locations/l105.yml Outdated Show resolved Hide resolved
locations/saarbruecker.yml Outdated Show resolved Hide resolved
locations/segen.yml Outdated Show resolved Hide resolved
Copy link
Member

@Noki Noki left a comment

Choose a reason for hiding this comment

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

looks good to me

@Noki Noki merged commit a96fcd5 into master Feb 8, 2024
5 checks passed
@Noki Noki deleted the snmp-switches branch February 8, 2024 15:29
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