From 8bca546f583b6751f7d69d12d873245e09f4df47 Mon Sep 17 00:00:00 2001 From: Oleg Drokin Date: Mon, 27 Jul 2015 00:59:26 -0400 Subject: [PATCH] LU-6907 contrib: make gerrit_checkpatch.py less spammy Currently every checkpatch message is broadast to everybody on the patch and then to all watchers. Use the REST API to indicate that on failure the message should only be sent to the patch owner and in case of success the emails should not be sent at all. Change-Id: I65b2017c1cc9558eed3707d81f936acac4af37f5 Signed-off-by: Oleg Drokin Reviewed-on: http://review.whamcloud.com/15729 Reviewed-by: John L. Hammond Tested-by: Jenkins Reviewed-by: Andreas Dilger --- contrib/scripts/gerrit_checkpatch.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/contrib/scripts/gerrit_checkpatch.py b/contrib/scripts/gerrit_checkpatch.py index 9649978..5058deb 100755 --- a/contrib/scripts/gerrit_checkpatch.py +++ b/contrib/scripts/gerrit_checkpatch.py @@ -176,14 +176,16 @@ def review_input_and_score(path_line_comments, warning_count): 'labels': { 'Code-Review': code_review_score }, - 'comments': review_comments + 'comments': review_comments, + 'notify': 'OWNER', }, score else: return { 'message': 'Looks good to me.', 'labels': { 'Code-Review': code_review_score - } + }, + 'notify': 'NONE', }, score -- 1.8.3.1