On 11/27/23 13:38, Eelco Chaudron wrote:
> Signed-off-by: Eelco Chaudron <echau...@redhat.com>
> ---
> .ci/linux-build.sh | 9 +++++----
> .github/workflows/build-and-test.yml | 3 +++
> 2 files changed, 8 insertions(+), 4 deletions(-)
>
> diff --git a/.ci/linux-build.sh b/.ci/linux-build.sh
> index aa2ecc505..e9e1e24b5 100755
> --- a/.ci/linux-build.sh
> +++ b/.ci/linux-build.sh
> @@ -6,6 +6,7 @@ set -x
> CFLAGS_FOR_OVS="-g -O2"
> SPARSE_FLAGS=""
> EXTRA_OPTS="--enable-Werror"
> +JOBS=${JOBS:-"-j4"}
Is this used anywhere? Seems a little orthogonal to the
purpose of this set.
>
> function install_dpdk()
> {
> @@ -46,7 +47,7 @@ function build_ovs()
> configure_ovs $OPTS
> make selinux-policy
>
> - make -j4
> + make $JOBS
> }
>
> if [ "$DEB_PACKAGE" ]; then
> @@ -122,8 +123,8 @@ if [ "$TESTSUITE" = 'test' ]; then
> configure_ovs
>
> export DISTCHECK_CONFIGURE_FLAGS="$OPTS"
> - make distcheck -j4 CFLAGS="${CFLAGS_FOR_OVS}" \
> - TESTSUITEFLAGS=-j4 RECHECK=yes
> + make distcheck $JOBS CFLAGS="${CFLAGS_FOR_OVS}" \
> + TESTSUITEFLAGS=$JOBS RECHECK=yes
> else
> build_ovs
> for testsuite in $TESTSUITE; do
> @@ -134,7 +135,7 @@ else
> export DPDK_EAL_OPTIONS="--lcores 0@1,1@1,2@1"
> run_as_root="sudo -E PATH=$PATH"
> fi
> - $run_as_root make $testsuite TESTSUITEFLAGS=-j4 RECHECK=yes
> + $run_as_root make $testsuite TESTSUITEFLAGS=$JOBS RECHECK=yes
> done
> fi
>
> diff --git a/.github/workflows/build-and-test.yml
> b/.github/workflows/build-and-test.yml
> index 09654205e..5d441157c 100644
> --- a/.github/workflows/build-and-test.yml
> +++ b/.github/workflows/build-and-test.yml
> @@ -164,6 +164,9 @@ jobs:
> m32: m32
> opts: --disable-ssl
>
> + - compiler: gcc
> + testsuite: check-ovsdb-cluster
FWIW, there is no need to run this testsuite as root.
But I'm not sure it worth changing the scripts in order
to avoid that.
Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
