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

Disable snmpd module disk_hw, so no syslog messages about unavailable disks #977

Merged
merged 3 commits into from
Sep 22, 2017
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 3 additions & 8 deletions dockers/docker-snmp-sv2/snmpd.conf.j2
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ agentAddress udp:{{ prefix | ip }}:161
{% endif %}
{% endfor %}
{% else %}
# Listen on all addresses as mgmt ip not specified
# Listen on all addresses as mgmt ip not specified
agentAddress udp:161
{% endif %}

Expand Down Expand Up @@ -61,7 +61,7 @@ sysServices 72
#
# Process Monitoring
#
# TODO: should we enable snmp based monitoring of sswsyncd and other processes?
# todo: should we enable snmp based monitoring of sswsyncd and other processes?
Copy link
Contributor

@jleveque jleveque Sep 22, 2017

Choose a reason for hiding this comment

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

Why change TODO to lowercase? Other TODOs in the file are uppercase.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It's in a revert commit. Since it is a comment format issue, I will not refix it again.


# At least one 'sendmail' process, but no more than 10
#proc sendmail 10 1
Expand All @@ -76,12 +76,7 @@ sysServices 72
# 10MBs required on root disk, 5% free on /var, 10% free on all other disks
disk / 10000
disk /var 5%

# Note: includeAllDisks will conflict with ignoredisk
# TODO: /root/* are introduced by aufs union mount, happening in initramfs stage. Clean them in the mount list.
ignoredisk /root/host
ignoredisk /root/dev
ignoredisk /root/dev/pts
includeAllDisks 10%

# Walk the UCD-SNMP-MIB::dskTable to see the resulting output
# Note that this table will be empty if there are no "disk" entries in the snmpd.conf file
Expand Down
2 changes: 1 addition & 1 deletion dockers/docker-snmp-sv2/supervisord.conf
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ stdout_logfile=syslog
stderr_logfile=syslog

[program:snmpd]
command=/usr/sbin/snmpd -f -LS4d -u Debian-snmp -g Debian-snmp -I -smux,mteTrigger,mteTriggerConf,ifTable,ifXTable,inetCidrRouteTable,ipCidrRouteTable,ip -p /run/snmpd.pid
command=/usr/sbin/snmpd -f -LS4d -u Debian-snmp -g Debian-snmp -I -smux,mteTrigger,mteTriggerConf,ifTable,ifXTable,inetCidrRouteTable,ipCidrRouteTable,ip,disk_hw -p /run/snmpd.pid
priority=3
autostart=false
autorestart=false
Expand Down