LGTM +1
On Sat, Oct 20, 2018 at 2:54 PM Patrick マルタインアンドレアス Uiterwijk
<puiterwijk(a)redhat.com> wrote:
>
> Hi all,
>
> Can I get +1s for the patchset to allow clime to ack service and host issues in
nagios, and to make the initial COPR notifs go to him?
>
> Patrick
>
>
> From 1994c662f8eabcec78f24a8d7b2edc155d25bac2 Mon Sep 17 00:00:00 2001
> From: Patrick Uiterwijk <patrick(a)puiterwijk.org>
> Date: Sat, 20 Oct 2018 11:19:57 +0200
> Subject: [PATCH 1/3] Allow clime to ack services
>
> Signed-off-by: Patrick Uiterwijk <patrick(a)puiterwijk.org>
> ---
> roles/nagios_server/templates/nagios/configs/cgi.cfg.j2 | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/roles/nagios_server/templates/nagios/configs/cgi.cfg.j2
b/roles/nagios_server/templates/nagios/configs/cgi.cfg.j2
> index 5929a6c78..b7992f168 100644
> --- a/roles/nagios_server/templates/nagios/configs/cgi.cfg.j2
> +++ b/roles/nagios_server/templates/nagios/configs/cgi.cfg.j2
> @@ -173,9 +173,9 @@ authorized_for_all_hosts=*
>
> #authorized_for_all_service_commands=nagiosadmin
> #authorized_for_all_host_commands=nagiosadmin
>
-authorized_for_all_service_commands=averi,codeblock,kevin,nb,pfrields,puiterwijk,smooge,tibbs,pbrobinson,spot,pingou,tflink,mizdebsk,msimacek,stickster,bstinson,cverna
>
+authorized_for_all_service_commands=averi,codeblock,kevin,nb,pfrields,puiterwijk,smooge,tibbs,pbrobinson,spot,pingou,tflink,mizdebsk,msimacek,stickster,bstinson,cverna,clime
>
>
-authorized_for_all_host_commands=averi,codeblock,kevin,nb,pfrields,puiterwijk,smooge,tibbs,pbrobinson,spot,pingou,tflink,mizdebsk,msimacek,stickster,bstinson,cverna
>
+authorized_for_all_host_commands=averi,codeblock,kevin,nb,pfrields,puiterwijk,smooge,tibbs,pbrobinson,spot,pingou,tflink,mizdebsk,msimacek,stickster,bstinson,cverna,clime
>
> # STATUSMAP BACKGROUND IMAGE
> # This option allows you to specify an image to be used as a
> --
> 2.17.1
>
>
> From 192e13613f1e68ad7dce2783fce1f18f73c95a6a Mon Sep 17 00:00:00 2001
> From: Patrick Uiterwijk <patrick(a)puiterwijk.org>
> Date: Sat, 20 Oct 2018 11:22:01 +0200
> Subject: [PATCH 2/3] Add contact info for clime
>
> Signed-off-by: Patrick Uiterwijk <patrick(a)puiterwijk.org>
> ---
> .../nagios_server/files/nagios/contactgroups/copr.cfg | 6 ++++++
> roles/nagios_server/files/nagios/contacts/clime.cfg | 11 +++++++++++
> 2 files changed, 17 insertions(+)
> create mode 100644 roles/nagios_server/files/nagios/contactgroups/copr.cfg
> create mode 100644 roles/nagios_server/files/nagios/contacts/clime.cfg
>
> diff --git a/roles/nagios_server/files/nagios/contactgroups/copr.cfg
b/roles/nagios_server/files/nagios/contactgroups/copr.cfg
> new file mode 100644
> index 000000000..c404e7f3e
> --- /dev/null
> +++ b/roles/nagios_server/files/nagios/contactgroups/copr.cfg
> @@ -0,0 +1,6 @@
> +define contactgroup {
> + contactgroup_name copr
> + alias COPR Notifications
> + members clime
> +
> +}
> diff --git a/roles/nagios_server/files/nagios/contacts/clime.cfg
b/roles/nagios_server/files/nagios/contacts/clime.cfg
> new file mode 100644
> index 000000000..e33c4d007
> --- /dev/null
> +++ b/roles/nagios_server/files/nagios/contacts/clime.cfg
> @@ -0,0 +1,11 @@
> +define contact{
> + contact_name clime
> + alias clime
> + service_notification_period 24x7
> + host_notification_period 24x7
> + service_notification_options w,u,c,r
> + host_notification_options d,u,r
> + service_notification_commands notify-by-epager
> + host_notification_commands host-notify-by-epager
> + pager clime(a)redhat.com
> +}
> --
> 2.17.1
>
>
> From 2ef75ed7be038996940b5145a1824f459cfa6d79 Mon Sep 17 00:00:00 2001
> From: Patrick Uiterwijk <patrick(a)puiterwijk.org>
> Date: Sat, 20 Oct 2018 11:37:31 +0200
> Subject: [PATCH 3/3] Set COPR backend to send notifs to copr
>
> Signed-off-by: Patrick Uiterwijk <patrick(a)puiterwijk.org>
> ---
> roles/nagios_server/files/nagios/services/copr.cfg | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/roles/nagios_server/files/nagios/services/copr.cfg
b/roles/nagios_server/files/nagios/services/copr.cfg
> index add302185..2f1f09824 100644
> --- a/roles/nagios_server/files/nagios/services/copr.cfg
> +++ b/roles/nagios_server/files/nagios/services/copr.cfg
> @@ -3,4 +3,5 @@ define service {
> service_description Check Copr backend consecutive build failures
> check_command check_by_nrpe!check_copr_backend_failed
> use defaulttemplate
> + contact_groups copr
> }
> --
> 2.17.1
> _______________________________________________
> infrastructure mailing list -- infrastructure(a)lists.fedoraproject.org
> To unsubscribe send an email to infrastructure-leave(a)lists.fedoraproject.org
> Fedora Code of Conduct:
https://getfedora.org/code-of-conduct.html
> List Guidelines:
https://fedoraproject.org/wiki/Mailing_list_guidelines
> List Archives:
https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedora...