From 5a4e5f4f73a37f13713ed0014ed47ddde4f00113 Mon Sep 17 00:00:00 2001 From: Andrew Yourtchenko Date: Wed, 2 Jun 2021 14:18:37 +0000 Subject: api-checkstyle: do not send the notification about the successful API checks the api-checkstyle was initially done with notification for both successful and failed api-checkstyle runs, just as a matter of precaution. It's time to keep the notification only for the fail case, thus fulfilling the point of notifications. Change-Id: I41a12e7a30bc852c9280fc08a6eae426438f7d92 Signed-off-by: Andrew Yourtchenko --- jjb/scripts/vpp/api-checkstyle.sh | 4 ---- 1 file changed, 4 deletions(-) diff --git a/jjb/scripts/vpp/api-checkstyle.sh b/jjb/scripts/vpp/api-checkstyle.sh index c79884fa..a1c2f84a 100644 --- a/jjb/scripts/vpp/api-checkstyle.sh +++ b/jjb/scripts/vpp/api-checkstyle.sh @@ -32,10 +32,6 @@ if [ -f $VPP_CRC_CHECKER ]; then echo "Running $VPP_CRC_CHECKER_CMD" if $VPP_CRC_CHECKER_CMD; then echo "API check successful" - - # for now - notify the same room during the monitoring period about the successes as well - WEBEX_TEAMS_MESSAGE="API check successful for $GERRIT_REFSPEC - see $BUILD_URL" - send_notify else RET_CODE=$? echo "API check failed: ret code $RET_CODE; please read https://wiki.fd.io/view/VPP/ApiChangeProcess and discuss with ayourtch@gmail.com if unsure how to proceed" -- cgit 1.2.3-korg