From e04cb57e6a3808dd59bf2c28792ed513360bc930 Mon Sep 17 00:00:00 2001 From: Andrew Yourtchenko Date: Wed, 3 Jun 2020 19:30:31 +0000 Subject: vpp-checkstyle-verify-{stream}: add the step to API check This patch adds calling the crcchecker.py --check-patchset in the initial checkstyle path. (see change ID I5f13c0976d8a12a58131b3e270f2dc9c00dc7d8c) It adds also: - gerrit-refspec-parameter to be able to specify the patchset manually - credentials-binding for a WebEx Teams bot - I had them added via IT-19800 - necessary plumbing to use these credentials. It is done by a minimal curl setup, which posts into room "VPP API check failures", which I created to be able to monitor the behavior and jump in to discuss and help to sort out the changes that may be problematic. Signed-off-by: Andrew Yourtchenko Change-Id: I4833aa929044413a1ac08ab80668f25c0db8b3ff --- jjb/scripts/vpp/api-checkstyle.sh | 30 ++++++++++++++++++++++++++++++ 1 file changed, 30 insertions(+) create mode 100644 jjb/scripts/vpp/api-checkstyle.sh (limited to 'jjb/scripts/vpp/api-checkstyle.sh') diff --git a/jjb/scripts/vpp/api-checkstyle.sh b/jjb/scripts/vpp/api-checkstyle.sh new file mode 100644 index 000000000..90740337e --- /dev/null +++ b/jjb/scripts/vpp/api-checkstyle.sh @@ -0,0 +1,30 @@ +#!/bin/bash + +VPP_CRC_CHECKER="extras/scripts/crcchecker.py" +VPP_CRC_CHECKER_CMD="$VPP_CRC_CHECKER --check-patchset" + +send_notify() { + # 'roomId' field from the response of: + # curl https://api.ciscospark.com/v1/memberships -H "Authorization: Bearer ${SECRET_WEBEX_TEAMS_ACCESS_TOKEN}" + WEBEX_TEAMS_ROOM_ID='Y2lzY29zcGFyazovL3VzL1JPT00vMzUzZmI3OTAtYTVjNS0xMWVhLWI4ZjYtMDUxN2I4NzFmOWU5' + curl https://api.ciscospark.com/v1/messages -X POST -H "Authorization: Bearer ${SECRET_WEBEX_TEAMS_ACCESS_TOKEN}" -H "Content-Type: application/json" --data '{"roomId":"'${WEBEX_TEAMS_ROOM_ID}'", "markdown": "'"${WEBEX_TEAMS_MESSAGE}"'" }' || true +} + +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" + WEBEX_TEAMS_MESSAGE="API check FAILED for $GERRIT_REFSPEC - see $BUILD_URL" + send_notify + exit $RET_CODE + fi +else + echo "Cannot find $VPP_CRC_CHECKER - skipping API compatibility check" +fi -- cgit 1.2.3-korg