[Libreoffice-commits] dev-tools.git: gerrit/auto-submit

[Guilhem Moulin (via logerrit)]

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

New commits:
commit 84a335d47a0b0e49f88447135d6d8ae60c198344
Author:     Guilhem Moulin <guilhem at libreoffice.org>
AuthorDate: Tue Feb 18 04:05:25 2020 +0100
Commit:     Miklos Vajna <vmiklos at 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 <vmiklos at collabora.com>
    Tested-by: Miklos Vajna <vmiklos at 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']: