summary |
shortlog |
log |
commit | commitdiff |
review |
tree
raw |
patch |
inline | side by side (from parent 1:
586dda5)
This patch does the following:
1) Introduces a new PARAMATER called is-csit-vpp-job
2) Sets that PARAMETER to 'True' for csit-vpp-jobs
3) Changes the include-raw-vpp-build.sh so that:
a) It uses make verify only if this is not a csit job *and*
if the patch has as a parent a point from which make verify
is known to work
b) It uses the old build.sh if its building CSIT jobs or the patch
is from before make verify.
The result is, patches should verify or not on their own merit, not
on the happenstance of coming before or after make verify, and
we don't do the extra make verify work for jobs were CSIT just needs
packages built.
Note: This patch is a fixed version of https://gerrit.fd.io/r/#/c/4083/
and has been tested using the 'DRYRUN' env variable.
Change-Id: Ie6f1e6ce2809e21ea025bccfaec703cd7519e20e
Signed-off-by: Ed Warnicke <hagbard@gmail.com>
echo "sha1sum of this script: ${0}"
sha1sum $0
echo "sha1sum of this script: ${0}"
sha1sum $0
-# Check to make sure the patch doesn't need to be rebased
-# Since there was a discontinuity, patches with a
-# parent before 30d41ff need to be rebased
-
-(git log --oneline | grep 30d41ff > /dev/null 2>&1);if [ $? != 0 ]; then REBASE_NEEDED="1";fi
-(git log --oneline | grep fb0815d > /dev/null 2>&1);if [ $? == 0 ]; then VPP_REPO="1";fi
-echo "REBASE_NEEDED: ${REBASE_NEEDED}"
-echo "VPP_REPO: ${VPP_REPO}"
+echo "CC=${CC}"
+echo "IS_CSIT_VPP_JOB=${IS_CSIT_VPP_JOB}"
+# If and only if we are doing verify *after* make verify was made to work
+# and we are not a CSIT job just building packages, then use make verify,
+# else use the old build-root/vagrant/build.sh
+if (git log --oneline | grep 37682e1 > /dev/null 2>&1) && \
+ [ "x${IS_CSIT_VPP_JOB}" != "xTrue" ]
+then
+ echo "Building using \"make verify\""
+ [ "x${DRYRUN}" == "xTrue" ] || make UNATTENDED=yes verify
+else
+ echo "Building using \"build-root/vagrant/build.sh\""
+ [ "x${DRYRUN}" == "xTrue" ] || build-root/vagrant/build.sh
+fi
if [ "x${VPP_REPO}" == "x1" ]; then
if [ "x${REBASE_NEEDED}" == "x1" ]; then
if [ "x${VPP_REPO}" == "x1" ]; then
if [ "x${REBASE_NEEDED}" == "x1" ]; then
-echo "CC=${CC}"
-echo "IS_CSIT_VPP_JOB=${IS_CSIT_VPP_JOB}"
-make UNATTENDED=yes verify
echo "*******************************************************************"
echo "* VPP BUILD SUCCESSFULLY COMPLETED"
echo "*******************************************************************"
echo "* VPP BUILD SUCCESSFULLY COMPLETED"