diff options
| author |   Andrew Yourtchenko <ayourtch@gmail.com> | 2021-06-02 14:18:37 +0000 |
|---|---|---|
| committer | Andrew Yourtchenko <ayourtch@gmail.com> | 2021-06-02 14:22:05 +0000 |
| commit | 5a4e5f4f73a37f13713ed0014ed47ddde4f00113 (patch) | |
| tree | 245c3ab6112e03c35c47731c3d2ff606b9ff5e6c | |
| parent | 381367d3b33aedb5bea293aaa9be25580ebad80b (diff) | |
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 <ayourtch@gmail.com>
| -rw-r--r-- | jjb/scripts/vpp/api-checkstyle.sh | 4 |
1 files changed, 0 insertions, 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" |