Skip to content

Commit

Permalink
travis: Fix using DPDK build from master branch cache.
Browse files Browse the repository at this point in the history
If no cache available for current branch, Travis CI uses cache from
the default (master) branch.  This causes build failures on older
OVS branches if cache was cleared because DPDK builds from master
branch are not compatible.

For example, we removed pdump support on master, but branch 2.13
explicitly requests pdump being enabled and fails while using
cached DPDK from the master branch.

Adding the branch name to the version file to avoid using incompatible
DPDK builds.

Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
Signed-off-by: 0-day Robot <robot@bytheb.org>
  • Loading branch information
igsilya authored and ovsrobot committed Mar 9, 2020
1 parent 59e9944 commit 5f5606e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .travis/linux-build.sh
Expand Up @@ -98,7 +98,7 @@ function install_dpdk()
else
if [ -f "${VERSION_FILE}" ]; then
VER=$(cat ${VERSION_FILE})
if [ "${VER}" = "${DPDK_VER}" ]; then
if [ "${VER}" = "${DPDK_VER}-${TRAVIS_BRANCH}" ]; then
EXTRA_OPTS="${EXTRA_OPTS} --with-dpdk=$(pwd)/dpdk-dir/build"
echo "Found cached DPDK ${VER} build in $(pwd)/dpdk-dir"
return
Expand Down Expand Up @@ -128,7 +128,7 @@ function install_dpdk()
EXTRA_OPTS="$EXTRA_OPTS --with-dpdk=$(pwd)/build"
echo "Installed DPDK source in $(pwd)"
popd
echo "${DPDK_VER}" > ${VERSION_FILE}
echo "${DPDK_VER}-${TRAVIS_BRANCH}" > ${VERSION_FILE}
}

function configure_ovs()
Expand Down

0 comments on commit 5f5606e

Please sign in to comment.