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

setup for testing of SNMP trap messages #156

Merged
merged 11 commits into from
Nov 14, 2017
Merged

Conversation

mbukatov
Copy link
Contributor

@mbukatov mbukatov commented Nov 9, 2017

This pull request creates ansible playbook to enable snmp alerting and configures snmptrapd on the client machine to receive them.

Based on current Installation Guide (Tendrl v1.5.4)

Known problems with current SNMP setup as described in the Install Guide:

Copy snmp.conf.yaml.sample file without any changes from tendrl-notifier
repository, commit 3ded35b07ea9ba278379c14d293966172134ad59, release 1.5.4.
Package net-snmp-devel installed to have net-snmp-config tool available
on the client machine.
@mbukatov mbukatov self-assigned this Nov 9, 2017
@mbukatov mbukatov changed the title wip: setup for testing of SNMP trap messages setup for testing of SNMP trap messages Nov 13, 2017
- name: restart snmptrapd
service:
name=snmptrapd
state=restarted
Copy link
Contributor

@dahorak dahorak Nov 14, 2017

Choose a reason for hiding this comment

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

Is this handler used somewhere? (I just wan't to know if I miss something.)

Copy link
Contributor Author

@mbukatov mbukatov Nov 14, 2017

Choose a reason for hiding this comment

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

Good point. Fixed in cf78a7f

Copy link
Contributor

@dahorak dahorak left a comment

Choose a reason for hiding this comment

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

LGTM

@mbukatov mbukatov merged commit 81fcb3e into usmqe:master Nov 14, 2017
@mbukatov mbukatov deleted the snmp branch November 14, 2017 08:49
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