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

Reply via email to