Re: FBR: Make clime be on the pager for COPR

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



LGTM +1


On Sat, Oct 20, 2018 at 2:54 PM Patrick マルタインアンドレアス Uiterwijk
<puiterwijk@xxxxxxxxxx> 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@xxxxxxxxxxxxxx>
> Date: Sat, 20 Oct 2018 11:19:57 +0200
> Subject: [PATCH 1/3] Allow clime to ack services
>
> Signed-off-by: Patrick Uiterwijk <patrick@xxxxxxxxxxxxxx>
> ---
>  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@xxxxxxxxxxxxxx>
> Date: Sat, 20 Oct 2018 11:22:01 +0200
> Subject: [PATCH 2/3] Add contact info for clime
>
> Signed-off-by: Patrick Uiterwijk <patrick@xxxxxxxxxxxxxx>
> ---
>  .../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@xxxxxxxxxx
> +}
> --
> 2.17.1
>
>
> From 2ef75ed7be038996940b5145a1824f459cfa6d79 Mon Sep 17 00:00:00 2001
> From: Patrick Uiterwijk <patrick@xxxxxxxxxxxxxx>
> 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@xxxxxxxxxxxxxx>
> ---
>  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@xxxxxxxxxxxxxxxxxxxxxxx
> To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx
> 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@xxxxxxxxxxxxxxxxxxxxxxx
_______________________________________________
infrastructure mailing list -- infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx
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@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Development]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]

  Powered by Linux