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

collectd: disable logging for locations with AirOS 6 snmp_profiles #766

Closed
FFHener opened this issue Feb 7, 2024 · 3 comments · Fixed by #769
Closed

collectd: disable logging for locations with AirOS 6 snmp_profiles #766

FFHener opened this issue Feb 7, 2024 · 3 comments · Fixed by #769
Labels
enhancement New feature or request

Comments

@FFHener
Copy link
Contributor

FFHener commented Feb 7, 2024

When a AirOS 6 AP has no Stations connected a snmp-call asking for their names results in a NoSuchName error for each parameter. That means a lot of log spamming. As we only need collectd logging when we debug it i would suggest adding a script at rc.local deactivating logging after boot.

@FFHener FFHener added the enhancement New feature or request label Feb 7, 2024
@Noki
Copy link
Member

Noki commented Feb 7, 2024

As we only need collectd logging when we debug it

I disagree. We want to have historical data for our sites so we can identify performance degradations. We have had antennas whose alignment has deteriorated over time, e.g. due to wind, snow load or unstable mounting, and we have had antennas that have deteriorated due to water ingress. We can only detect all of this with continuous monitoring.

@FFHener
Copy link
Contributor Author

FFHener commented Feb 7, 2024

Thats a misunderstabding. Collectd should still log the snmp-data, but not the errors to logd locally on the device

@Noki
Copy link
Member

Noki commented Feb 7, 2024

Got it! Yes, that's what we want.

FFHener added a commit that referenced this issue Feb 9, 2024
When collectd wants to get the Names of Stations via snmp connnected
to a device running AirOS 6 that have none connected it reports a
"NoSuchName" error to logd resulting in a lots of logspamming. At
chris-core it was 60 per minute.

This Script will change the option "procd_set_param stderr  1" to "0"
in the file /etc/init.d/collectd to deactivate logging of all errors.
The monitoring will still work. I prefer not having logs over logspamming
as the logfiles will get useless otherwise. It is still possible to
change this option back to 1 until next boot to get a better insight
when debugging collectd-issues.

This fixes #766
FFHener added a commit that referenced this issue Feb 10, 2024
When collectd wants to get the Names of Stations via snmp connnected
to a device running AirOS 6 that have none connected it reports a
"NoSuchName" error to logd resulting in a lots of logspamming. At
chris-core it was 60 per minute.

This Script will change the option "procd_set_param stderr  1" to "0"
in the file /etc/init.d/collectd to deactivate logging of all errors.
The monitoring will still work. I prefer not having logs over logspamming
as the logfiles will get useless otherwise. It is still possible to
change this option back to 1 until next boot to get a better insight
when debugging collectd-issues.

This fixes #766
FFHener added a commit that referenced this issue Feb 10, 2024
When collectd wants to get the Names of Stations via snmp connnected
to a device running AirOS 6 that have none connected it reports a
"NoSuchName" error to logd resulting in a lots of logspamming. At
chris-core it was 60 per minute.

This Script will change the option "procd_set_param stderr  1" to "0"
in the file /etc/init.d/collectd to deactivate logging of all errors.
The monitoring will still work. I prefer not having logs over logspamming
as the logfiles will get useless otherwise. It is still possible to
change this option back to 1 until next boot to get a better insight
when debugging collectd-issues.

This fixes #766
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants