Skip to content

Adding procedure for SNMP traps#141

Merged
leifmadsen merged 10 commits into
masterfrom
csibbitt-1190-SNMPtraps
Nov 12, 2020
Merged

Adding procedure for SNMP traps#141
leifmadsen merged 10 commits into
masterfrom
csibbitt-1190-SNMPtraps

Conversation

@csibbitt

@csibbitt csibbitt commented Nov 5, 2020

Copy link
Copy Markdown
Contributor

No description provided.

@csibbitt

csibbitt commented Nov 5, 2020

Copy link
Copy Markdown
Contributor Author

@leifmadsen leifmadsen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Awesome! Looks good to me.

@mrunge

mrunge commented Nov 6, 2020

Copy link
Copy Markdown
Member

Great, thank you!

Comment thread doc-Service-Telemetry-Framework/modules/proc_configuring-snmp-traps.adoc Outdated
[id="configuring-snmp-traps_{context}"]
= Configuring SNMP Traps

{Project} ({ProjectShort}) can deliver alerts via SNMP Traps.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@csibbitt This may or not be relevant but I'll ask anyway. Why would I want to use SNMP Traps?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You can understand SNMP Traps as alert or notification.

@leifmadsen leifmadsen Nov 9, 2020

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Enabling SNMPtraps functionality allows for defined PrometheusRules that would result in a fired alert via Alertmanager to be consumed by the prometheus-snmp-webhook pod via webhook receiver, for ultimate delivery to an SNMPtrap receiver.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Why would I want to use SNMP Traps?

To integrate STF with an existing infrastructure monitoring platform that receives notifications via SNMP traps.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks Chris

Comment thread doc-Service-Telemetry-Framework/modules/proc_configuring-snmp-traps.adoc Outdated
Comment thread doc-Service-Telemetry-Framework/modules/proc_configuring-snmp-traps.adoc Outdated
Comment thread doc-Service-Telemetry-Framework/modules/proc_configuring-snmp-traps.adoc Outdated
Comment thread doc-Service-Telemetry-Framework/modules/proc_configuring-snmp-traps.adoc Outdated
JoanneOFlynn2018 and others added 5 commits November 9, 2020 09:38
…traps.adoc

Co-authored-by: JoanneOFlynn2018 <45287002+JoanneOFlynn2018@users.noreply.github.com>
…traps.adoc

Co-authored-by: JoanneOFlynn2018 <45287002+JoanneOFlynn2018@users.noreply.github.com>
…traps.adoc

Co-authored-by: JoanneOFlynn2018 <45287002+JoanneOFlynn2018@users.noreply.github.com>
Comment thread doc-Service-Telemetry-Framework/modules/proc_configuring-snmp-traps.adoc Outdated
Comment thread doc-Service-Telemetry-Framework/modules/proc_configuring-snmp-traps.adoc Outdated
Comment thread doc-Service-Telemetry-Framework/modules/proc_configuring-snmp-traps.adoc Outdated

@JoanneOFlynn2018 JoanneOFlynn2018 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hi Chris,
Made a change to the intro.
Also, wondering if SNMP Traps should live in the Alerts section? Don't want to slow down the process any more than I have already. We can change this at a later date if needs be!

@leifmadsen

Copy link
Copy Markdown
Member

Hi Chris,
Made a change to the intro.
Also, wondering if SNMP Traps should live in the Alerts section? Don't want to slow down the process any more than I have already. We can change this at a later date if needs be!

That makes a lot of sense actually, especially since the configuration is under the alerting parameter. I think I'll go ahead and merge this for now, and we can adjust where it lives after.

@leifmadsen leifmadsen merged commit e8d5328 into master Nov 12, 2020
@leifmadsen leifmadsen deleted the csibbitt-1190-SNMPtraps branch November 12, 2020 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants