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

monitoring: SNMP OID per every Prometheus alert rule #27978

Merged
merged 1 commit into from May 29, 2019

Conversation

votdev
Copy link
Member

@votdev votdev commented May 6, 2019

Use the Ceph enterprise OID 50495 (https://www.iana.org/assignments/enterprise-numbers/enterprise-numbers) and create OIDs for every Prometheus alert rule according to the schema at https://github.com/SUSE/prometheus-webhook-snmp/blob/master/README.md.

Example OID:
1.3.6.1.4.1.50495.15.1.2.2.1

All alert rule OIDs are located below the object identifier 15 (15 for p which is the first character of prometheus). Check out the MIB at https://github.com/SUSE/prometheus-webhook-snmp/blob/master/PROMETHEUS-ALERT-CEPH-MIB.txt for more details.

Signed-off-by: Volker Theile vtheile@suse.com

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug

@votdev votdev requested review from jan--f and liewegas May 6, 2019 15:20
Copy link
Contributor

@jan--f jan--f left a comment

Choose a reason for hiding this comment

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

👍

@tchaikov
Copy link
Contributor

tchaikov commented May 7, 2019

could you prefix the title of your commit message with the subcomponent your are changing ? see https://github.com/ceph/ceph/blob/master/SubmittingPatches.rst#3-describe-your-changes . and would be great if you could shorten the title a little bit.

@votdev votdev changed the title Add a SNMP OID per every Prometheus Alermanager alert rule. monitoring: Add SNMP OID per every Prometheus alert rule May 7, 2019
@votdev votdev changed the title monitoring: Add SNMP OID per every Prometheus alert rule monitoring: SNMP OID per every Prometheus alert rule May 7, 2019
@votdev votdev force-pushed the add_snmp_oids_to_prometheus_alerts branch from cc0f796 to 1bb5666 Compare May 7, 2019 08:57
@jan--f jan--f requested a review from b-ranto May 8, 2019 11:19
@tchaikov
Copy link
Contributor

@votdev please note the title of the commit message and the title of a PR is not the same thing. could you update your commit and repush it as well? thanks!

@votdev votdev force-pushed the add_snmp_oids_to_prometheus_alerts branch 2 times, most recently from 4f9ba18 to 56029b3 Compare May 21, 2019 07:09
@votdev
Copy link
Member Author

votdev commented May 21, 2019

jenkins retest this please

Copy link
Contributor

@LenzGr LenzGr left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks.

@votdev
Copy link
Member Author

votdev commented May 27, 2019

jenkins retest this please

Use the Ceph enterprise OID 50495 (https://www.iana.org/assignments/enterprise-numbers/enterprise-numbers) and create OIDs for every Prometheus alert rule according to the schema at https://github.com/SUSE/prometheus-webhook-snmp/blob/master/README.md.

Example OID:
1.3.6.1.4.1.50495.15.1.2.2.1

All alert rule OIDs are located below the object identifier 15 (15 for p which is the first character of prometheus). Check out the MIB at https://github.com/SUSE/prometheus-webhook-snmp/blob/master/PROMETHEUS-ALERT-CEPH-MIB.txt for more details.

Signed-off-by: Volker Theile <vtheile@suse.com>
@votdev votdev force-pushed the add_snmp_oids_to_prometheus_alerts branch from 56029b3 to 8e6838c Compare May 28, 2019 08:00
@LenzGr LenzGr merged commit a9ffaa2 into ceph:master May 29, 2019
@votdev votdev deleted the add_snmp_oids_to_prometheus_alerts branch May 29, 2019 14:03
@smithfarm
Copy link
Contributor

@LenzGr @p-se @jan--f @votdev @b-ranto Is this something that should be backported to nautilus?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants