On Mon, Aug 10, 2026 at 7:31 PM Mark Michelson <[email protected]> wrote:

> 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
> >
>
>

Thank you Mark,

applied to main.

Regards,
Ales
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to