gerrit/auto-submit/gerrit-autosubmit |    5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

New commits:
commit 84a335d47a0b0e49f88447135d6d8ae60c198344
Author:     Guilhem Moulin <guil...@libreoffice.org>
AuthorDate: Tue Feb 18 04:05:25 2020 +0100
Commit:     Miklos Vajna <vmik...@collabora.com>
CommitDate: Fri Feb 21 09:09:14 2020 +0100

    gerrit-autosubmit: subscribe to "comment-added" events only
    
    No need to stream everything happening on the sever if other event types
    are being filtered client-side anyway :-)
    
    Change-Id: I98e6d510eec976143c731df59dd949098cde2ccb
    Reviewed-on: https://gerrit.libreoffice.org/c/dev-tools/+/88915
    Reviewed-by: Miklos Vajna <vmik...@collabora.com>
    Tested-by: Miklos Vajna <vmik...@collabora.com>

diff --git a/gerrit/auto-submit/gerrit-autosubmit 
b/gerrit/auto-submit/gerrit-autosubmit
index fec0bd2..802fbf8 100755
--- a/gerrit/auto-submit/gerrit-autosubmit
+++ b/gerrit/auto-submit/gerrit-autosubmit
@@ -27,7 +27,8 @@ if port != None and port != "":
     sshcommand.extend(["-p", port])
 
 while True:
-    cmd = sshcommand + ["-n", "-o", "ServerAliveInterval=5", server, "gerrit", 
"stream-events"]
+    cmd = sshcommand + ["-n", "-o", "ServerAliveInterval=5", server, "gerrit",
+            "stream-events", "-s", "comment-added"]
     sock = subprocess.Popen(cmd, stdout=subprocess.PIPE, bufsize=1)
 
     try:
@@ -35,7 +36,7 @@ while True:
             event = json.loads(line)
 
             if event['type'] != "comment-added":
-                continue
+                continue # should never happen
             if 'username' not in event['change']['owner'].keys():
                 continue
             if event['change']['owner']['username'] != os.environ['USER']:
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to