Thanks Ales! Acked-by: Mark Michelson <[email protected]>
On Mon, Aug 10, 2026 at 12:47 PM Ales Musil via dev <[email protected]> wrote: > > Do not overwrite the jobs directly in the file, use > the env variable instead. > > Fixes: 87f3c2364a5b ("utilities: Add simple container automation") > Signed-off-by: Ales Musil <[email protected]> > --- > v2: Rebase on top of current main. > --- > .ci/ci.sh | 14 ++------------ > 1 file changed, 2 insertions(+), 12 deletions(-) > > diff --git a/.ci/ci.sh b/.ci/ci.sh > index 76c364868..0344c8862 100755 > --- a/.ci/ci.sh > +++ b/.ci/ci.sh > @@ -92,12 +92,6 @@ function copy_sources_to_workdir() { > " > } > > -function overwrite_jobs() { > - container_exec " > - sed -i s/-j[0-9]/-j$jobs/ $CONTAINER_WORKDIR/.ci/linux-build.sh > - " > -} > - > function run_tests() { > container_exec " > cd $CONTAINER_WORKDIR \ > @@ -105,7 +99,7 @@ function run_tests() { > ARCH=$ARCH CC=$CC LIBS=$LIBS OPTS=$OPTS TESTSUITE=$TESTSUITE \ > TEST_RANGE=$TEST_RANGE SANITIZERS=$SANITIZERS DPDK=$DPDK \ > RECHECK=$RECHECK UNSTABLE=$UNSTABLE TIMEOUT=$TIMEOUT \ > - BASE_VERSION=$BASE_VERSION ./.ci/linux-build.sh > + BASE_VERSION=$BASE_VERSION JOBS=$JOBS ./.ci/linux-build.sh > " > } > > @@ -130,7 +124,7 @@ while true; do > ;; > --jobs) > shift > - jobs="$1" > + JOBS="-j$1" > ;; > --ovn-path) > shift > @@ -183,10 +177,6 @@ trap remove_container EXIT > > copy_sources_to_workdir > > -if [ -n "$jobs" ]; then > - overwrite_jobs > -fi > - > if [ -n "$shell" ];then > container_shell > exit 0 > -- > 2.55.0 > > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
