Freeze break request: add fedmsg bot in #fedora-admin

[Kevin Fenzi]

Greetings

Since we switched over to pagure for fedora-infrastructure issues we
don't have in irc channel notifications of new tickets/tickets being
closed, etc. 

Here's a stab at adding a fedmsg irc notification for #fedora-admin. 

It would be nice to also have our notifications _NOT_ go to
#fedora-apps, but I am not sure how best to exclude them. 

Additional patches for that welcome. 

In the mean time, +1s?

kevin
--
From 204718ff3ed6bd60f7b4297d3145ce06a5af55ed Mon Sep 17 00:00:00 2001
From: Kevin Fenzi <kevin(a)scrye.com&gt;
Date: Wed, 28 Sep 2016 20:44:02 +0000
Subject: [PATCH] add fedmsg bot for #fedora-admin for pagure issues changes

---
 roles/fedmsg/irc/templates/ircbot.py | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)

diff --git a/roles/fedmsg/irc/templates/ircbot.py
b/roles/fedmsg/irc/templates/ircbot.py index 2bb4c03..3d66e91 100644
--- a/roles/fedmsg/irc/templates/ircbot.py
+++ b/roles/fedmsg/irc/templates/ircbot.py
@@ -28,6 +28,29 @@ config = dict(
             ),
         ),
 
+        # For fedora-admin
+        dict(
+            network='chat.freenode.net',
+            port=6667,
+            make_pretty=True,
+            make_terse=True,
+
+            {% if env == 'staging' %}
+            nickname='fedmsg-admin-s',
+            {% else %}
+            nickname='fedmsg-admin',
+            {% endif %}
+            channel='fedora-admin',
+            filters=dict(
+                topic=[
+                    '^((?!(pagure)).)*$',
+                ],
+                body=[
+                    "^((?!(fedora-infrastructure|u'name':
u'pagure')).)*$",
+                ],
+            ),
+        ),
+
         # For fedora-apps
         dict(
             network='chat.freenode.net',
-- 
1.8.3.1

[Kevin Fenzi]

On top of that last patch, Patrick pointed out I didn't need part of
the body regex, so adding to that: 

diff --git a/roles/fedmsg/irc/templates/ircbot.py b/roles/fedmsg/irc/templates/ircbot.py
index 3d66e91..4723938 100644
--- a/roles/fedmsg/irc/templates/ircbot.py
+++ b/roles/fedmsg/irc/templates/ircbot.py
@@ -46,7 +46,7 @@ config = dict(
                     '^((?!(pagure)).)*$',
                 ],
                 body=[
-                    "^((?!(fedora-infrastructure|u'name':
u'pagure')).)*$",
+                    "^((?!(fedora-infrastructure)).)*$",
                 ],
             ),
         ),

[Patrick Uiterwijk]

+1

[Stephen John Smoogen]

+1 the two look good.

On 28 September 2016 at 17:58, Patrick  マルタインアンドレアス  Uiterwijk
<puiterwijk(a)redhat.com&gt; wrote:
>  +1
>  _______________________________________________
>  infrastructure mailing list -- infrastructure(a)lists.fedoraproject.org
>  To unsubscribe send an email to infrastructure-leave(a)lists.fedoraproject.org 


-- 
Stephen J Smoogen.