From a9dcab7a8ef57f3c41feabc2301013df40e5d79f Mon Sep 17 00:00:00 2001 From: Dave Wallace Date: Mon, 14 Aug 2023 22:58:31 -0400 Subject: vpp 23.06 post release clean up. - Remove vpp stable/2210 branch support Change-Id: Icef62c0e073a787645fe1663d7ca7e0da0ba440b Signed-off-by: Dave Wallace --- docker/scripts/lib_csit.sh | 44 +++++++++++++++++++++++++++++--------------- docker/scripts/lib_vpp.sh | 6 +++--- jjb/vpp/docs.yaml | 6 ------ jjb/vpp/vpp.yaml | 22 ---------------------- 4 files changed, 32 insertions(+), 46 deletions(-) diff --git a/docker/scripts/lib_csit.sh b/docker/scripts/lib_csit.sh index 9ac2a1961..36ab0b27d 100644 --- a/docker/scripts/lib_csit.sh +++ b/docker/scripts/lib_csit.sh @@ -50,12 +50,14 @@ csit_checkout_branch_for_vpp() { && source "$csit_bash_function_dir/branch.sh" CSIT_DIR="$csit_dir" checkout_csit_for_vpp "$vpp_branch" + # shellcheck disable=SC2034 csit_branch="$(git branch | grep -e '^*' | mawk '{print $2}')" } csit_install_packages() { local branch="$1" - local branchname="$(echo $branch | sed -e 's,/,_,')" + local branchname + branchname="$(echo $branch | sed -e 's,/,_,')" local csit_dir="$DOCKER_CSIT_DIR" local csit_ansible_dir="$csit_dir/fdio.infra.ansible" if [ ! -d "$csit_ansible_dir" ] ; then @@ -71,7 +73,8 @@ csit_install_packages() { [ "$OS_ARCH" = "aarch64" ] && exclude_roles="$exclude_roles -e iperf" # Not in double quotes to let bash remove newline characters - local yaml_files="$(grep -r packages_by $csit_ansible_dir | cut -d: -f1 | sort -u | grep -v $exclude_roles)" + local yaml_files + yaml_files="$(grep -r packages_by $csit_ansible_dir | cut -d: -f1 | sort -u | grep -v $exclude_roles)" packages="$(dbld_csit_find_ansible_packages.py --$OS_ID --$OS_ARCH $yaml_files)" packages="${packages/bionic /}" packages="${packages/focal /}" @@ -96,7 +99,6 @@ csit_install_packages() { csit_pip_cache() { local branch="$1" - local VENV_OPTS="" # ensure PS1 is defined (used by virtualenv activate script) PS1=${PS1:-"#"} CSIT_DIR="$DOCKER_CSIT_DIR" @@ -104,8 +106,11 @@ csit_pip_cache() { if [ -f "$CSIT_DIR/VPP_REPO_URL" ] \ && [ -f "$CSIT_DIR/requirements.txt" ]; then + local branchname + # use bash variable substitution to replace '/' with '_' to convert from + # vpp to csit branch name nomenclature + branchname="${branch////_}" local csit_bash_function_dir="$CSIT_DIR/resources/libraries/bash/function" - local branchname="$(echo $branch | sed -e 's,/,_,')" local bld_log="$DOCKER_BUILD_LOG_DIR" bld_log="${bld_log}/$FDIOTOOLS_IMAGENAME-$branchname-csit_pip_cache-bld.log" local pip_cmd="python3 -m pip --disable-pip-version-check" @@ -116,27 +121,36 @@ csit_pip_cache() { git clean -qfdx rm -rf "$PYTHONPATH/env" - # TODO: Update CSIT release branches to avoid build breakage - # Fixes https://github.com/pypa/pip/issues/8260 - $pip_cmd install pip==21.0.1 - # rls2009_lts-* branches missing cherry-pick of - # https://gerrit.fd.io/r/c/csit/+/31338 - sed -i 's/scipy==1.1.0/scipy==1.5.4/' "$PYTHONPATH/requirements.txt" - # Virtualenv version is pinned in common.sh in newer csit branches. # (note: xargs removes leading/trailing spaces) local common_sh="$csit_bash_function_dir/common.sh" install_virtualenv="$(grep 'virtualenv' $common_sh | grep pip | grep install | cut -d'|' -f1 | xargs)" $install_virtualenv - virtualenv --no-download --python=$(which python3) "$CSIT_DIR/env" + virtualenv --no-download --python="$(which python3)" "$CSIT_DIR/env" source "$CSIT_DIR/env/bin/activate" if [ "$OS_ARCH" = "aarch64" ] ; then - local numpy_ver="$(grep numpy $PYTHONPATH/requirements.txt)" + local numpy_ver + numpy_ver="$(grep numpy $PYTHONPATH/requirements.txt)" [ -n "$numpy_ver" ] && $pip_cmd install $numpy_ver 2>&1 | \ tee -a $bld_log fi + # TODO: Update CSIT release branches to PyYAML==6.0.1 or greater + # then remove this workaround: https://github.com/yaml/pyyaml/issues/736 + if grep -q 'PyYAML==5.4.1' "$PYTHONPATH/requirements.txt" ; then + local constraintfile + constraintfile="/tmp/constraint.txt" + + # create a constraint file that limits the Cython version to one that should work + echo 'Cython < 3.0' > "$constraintfile" + + # seed pip's local wheel cache with a PyYAML wheel + PIP_CONSTRAINT="$constraintfile" $pip_cmd wheel PyYAML==5.4.1 + + rm -f "$constraintfile" + fi + # Install csit python requirements $pip_cmd install -r "$CSIT_DIR/requirements.txt" 2>&1 | \ tee -a "$bld_log" @@ -144,9 +158,9 @@ csit_pip_cache() { $pip_cmd install -r "$CSIT_DIR/tox-requirements.txt" 2>&1 | \ tee -a "$bld_log" # Run tox which installs pylint requirments - pushd $CSIT_DIR >& /dev/null + pushd "$CSIT_DIR" >& /dev/null || exit 1 tox || true - popd >& /dev/null + popd >& /dev/null || exit 1 # Clean up virtualenv directories deactivate diff --git a/docker/scripts/lib_vpp.sh b/docker/scripts/lib_vpp.sh index db3c98a82..6a66f1595 100644 --- a/docker/scripts/lib_vpp.sh +++ b/docker/scripts/lib_vpp.sh @@ -96,7 +96,7 @@ docker_build_setup_vpp() { # to create an enumerated set of jobs jobs that match the # definitions here. declare -A VPP_BRANCHES -VPP_BRANCHES["debian-11"]="stable/2210 stable/2302 stable/2306 master" -VPP_BRANCHES["ubuntu-20.04"]="stable/2210 stable/2302 stable/2306 master" -VPP_BRANCHES["ubuntu-22.04"]="stable/2210 stable/2302 stable/2306 master" +VPP_BRANCHES["debian-11"]="stable/2302 stable/2306 master" +VPP_BRANCHES["ubuntu-20.04"]="stable/2302 stable/2306 master" +VPP_BRANCHES["ubuntu-22.04"]="stable/2302 stable/2306 master" export VPP_BRANCHES diff --git a/jjb/vpp/docs.yaml b/jjb/vpp/docs.yaml index bc025c6da..ea0d23a68 100644 --- a/jjb/vpp/docs.yaml +++ b/jjb/vpp/docs.yaml @@ -16,9 +16,6 @@ - 'master': branch: 'master' branch-refspec: '' - - '2210': - branch: 'stable/2210' - branch-refspec: '' - '2302': branch: 'stable/2302' branch-refspec: '' @@ -32,9 +29,6 @@ # os deprecated in 23.02 - stream: '2302' os: 'ubuntu2004' - # os deprecated in 22.10 - - stream: '2210' - os: 'ubuntu2004' # os deprecated in master - stream: 'master' os: 'ubuntu2004' diff --git a/jjb/vpp/vpp.yaml b/jjb/vpp/vpp.yaml index c0291284b..3bacf3b40 100644 --- a/jjb/vpp/vpp.yaml +++ b/jjb/vpp/vpp.yaml @@ -40,9 +40,6 @@ - 'master': branch: 'master' branch-refspec: '' - - '2210': - branch: 'stable/2210' - branch-refspec: '' - '2302': branch: 'stable/2302' branch-refspec: '' @@ -57,9 +54,6 @@ # os deprecated in 23.02 - stream: '2302' os: 'ubuntu2004' - # os deprecated in 22.10 - - stream: '2210' - os: 'ubuntu2004' # os deprecated in master - stream: 'master' os: 'ubuntu2004' @@ -88,11 +82,6 @@ branch-refspec: '' make-test-os: 'ubuntu-20.04 ubuntu-22.04' make-test-multiworker-os: 'debian-11' - - '2210': - branch: 'stable/2210' - branch-refspec: '' - make-test-os: 'ubuntu-20.04' - make-test-multiworker-os: 'debian-11' - '2302': branch: 'stable/2302' branch-refspec: '' @@ -125,9 +114,6 @@ - 'master': branch: 'master' branch-refspec: '' - - '2210': - branch: 'stable/2210' - branch-refspec: '' - '2302': branch: 'stable/2302' branch-refspec: '' @@ -141,9 +127,6 @@ # os deprecated in 23.02 - stream: '2302' os: 'ubuntu2004' - # os deprecated in 22.10 - - stream: '2210' - os: 'ubuntu2004' # os deprecated in master - stream: 'master' os: 'ubuntu2004' @@ -245,11 +228,6 @@ branch-refspec: '' make-test-os: 'ubuntu-20.04 ubuntu-22.04' make-test-multiworker-os: '' - - '2210': - branch: 'stable/2210' - branch-refspec: '' - make-test-os: 'ubuntu-20.04' - make-test-multiworker-os: '' - '2302': branch: 'stable/2302' branch-refspec: '' -- cgit 1.2.3-korg