Re: [ovs-dev] [PATCH ovn 06/10] github: Add some arm build lanes.

Tue, 17 Feb 2026 15:53:58 -0800 

On 2/16/26 1:49 AM, Ales Musil wrote:

*External email: Use caution opening links or attachments*
On Fri, Feb 13, 2026 at 10:39 PM Ihar Hrachyshka via dev <[email protected]> wrote: 

    This patch does not run every build job for ARM, but a few.

    Signed-off-by: Ihar Hrachyshka <[email protected]>
    ---


Hi Ihar,

this isn't a full review just a note, we run ARM jobs in CirrusCI.
Also we are struggling heavily with runner capacity and adding
extra jobs are probably not desirable right now. I suppose we could
try to update CirrusCI to include also system-tests.



Thank you for the heads-up Ales!
Would you consider keeping a separate github job for debian packaging (even if just for x86_64)? The current debian package already builds and runs integration tests (unless nocheck is set as build arg), so a single job could cover it end-to-end for the platform.
I found the current in-tree packaging is not exercised anywhere in CI (?) and may become abandoned. (It has some current issues that I'm looking to resolve, which brought me to the idea of adding some CI coverage in the first place.)
Let me know what you prefer and I will drop patches from the series that are not fitting. 



     .github/workflows/test.yml | 72
    ++++++++++++++++++++++----------------
     1 file changed, 42 insertions(+), 30 deletions(-)

    diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
    index 887bb8899..8500d029c 100644
    --- a/.github/workflows/test.yml
    +++ b/.github/workflows/test.yml
    @@ -24,9 +24,12 @@ jobs:
         # +-------+-------------------+-------------------+
         env:
           DEPENDENCIES: podman
    -    name: Prepare container
    +    name: Prepare container ${{ matrix.runner }}
         if: github.repository_owner == 'ovn-org' || github.event_name
    != 'schedule'
    -    runs-on: ubuntu-24.04
    +    runs-on: ${{ matrix.runner }}
    +    strategy:
    +      matrix:
    +        runner: [ubuntu-24.04, ubuntu-24.04-arm]

         steps:
           - uses: actions/checkout@v4
    @@ -77,7 +80,7 @@ jobs:
             uses: actions/cache/save@v4
             with:
               path: /tmp/image.tar
    -          key: ${{ github.sha }}/${{ github.event_name }}
    +          key: ${{ github.sha }}/${{ github.event_name }}/${{
    matrix.runner }}

       build-linux:
         needs: [prepare-container]
    @@ -93,37 +96,46 @@ jobs:
           UNSTABLE:    ${{ matrix.cfg.unstable }}

         name: linux ${{ join(matrix.cfg.*, ' ') }}
    -    runs-on: ubuntu-24.04
    +    runs-on: ${{ matrix.cfg.runner }}

         strategy:
           fail-fast: false
           matrix:
             cfg:
    -        - { compiler: gcc, opts: --disable-ssl }
    -        - { compiler: clang, opts: --disable-ssl }
    -        - { compiler: gcc, testsuite: dist-test }
    -        - { compiler: gcc, testsuite: test, test_range: "-300" }
    -        - { compiler: gcc, testsuite: test, test_range: "301-600" }
    -        - { compiler: gcc, testsuite: test, test_range: "601-",
    unstable: unstable }
    -        - { compiler: clang, testsuite: test, sanitizers:
    sanitizers, test_range: "-300" }
    -        - { compiler: clang, testsuite: test, sanitizers:
    sanitizers, test_range: "301-600" }
    -        - { compiler: clang, testsuite: test, sanitizers:
    sanitizers, test_range: "601-", unstable: unstable }
    -        - { compiler: gcc, testsuite: test, libs: -ljemalloc,
    test_range: "-300" }
    -        - { compiler: gcc, testsuite: test, libs: -ljemalloc,
    test_range: "301-600" }
    -        - { compiler: gcc, testsuite: test, libs: -ljemalloc,
    test_range: "601-", unstable: unstable }
    -        - { compiler: gcc, testsuite: system-test-dpdk, dpdk:
    dpdk, test_range: "-100" }
    -        - { compiler: gcc, testsuite: system-test-dpdk, dpdk:
    dpdk, test_range: "101-200" }
    -        - { compiler: gcc, testsuite: system-test-dpdk, dpdk:
    dpdk, test_range: "201-", unstable: unstable }
    -        - { compiler: gcc, testsuite: system-test-userspace,
    test_range: "-100" }
    -        - { compiler: gcc, testsuite: system-test-userspace,
    test_range: "101-200" }
    -        - { compiler: gcc, testsuite: system-test-userspace,
    test_range: "201-", unstable: unstable }
    -        - { compiler: gcc, testsuite: system-test, test_range:
    "-100" }
    -        - { compiler: gcc, testsuite: system-test, test_range:
    "101-200" }
    -        - { compiler: gcc, testsuite: system-test, test_range:
    "201-", unstable: unstable }
    -        - { compiler: clang, testsuite: system-test, sanitizers:
    sanitizers, test_range: "-100" }
    -        - { compiler: clang, testsuite: system-test, sanitizers:
    sanitizers, test_range: "101-200" }
    -        - { compiler: clang, testsuite: system-test, sanitizers:
    sanitizers, test_range: "201-", unstable: unstable }
    -        - { arch: x86, compiler: gcc, opts: --disable-ssl }
    +        # x86_64 builds
    +        - { runner: ubuntu-24.04, compiler: gcc, opts:
    --disable-ssl }
    +        - { runner: ubuntu-24.04, compiler: clang, opts:
    --disable-ssl }
    +        - { runner: ubuntu-24.04, compiler: gcc, testsuite:
    dist-test }
    +        - { runner: ubuntu-24.04, compiler: gcc, testsuite: test,
    test_range: "-300" }
    +        - { runner: ubuntu-24.04, compiler: gcc, testsuite: test,
    test_range: "301-600" }
    +        - { runner: ubuntu-24.04, compiler: gcc, testsuite: test,
    test_range: "601-", unstable: unstable }
    +        - { runner: ubuntu-24.04, compiler: clang, testsuite:
    test, sanitizers: sanitizers, test_range: "-300" }
    +        - { runner: ubuntu-24.04, compiler: clang, testsuite:
    test, sanitizers: sanitizers, test_range: "301-600" }
    +        - { runner: ubuntu-24.04, compiler: clang, testsuite:
    test, sanitizers: sanitizers, test_range: "601-", unstable: unstable }
    +        - { runner: ubuntu-24.04, compiler: gcc, testsuite: test,
    libs: -ljemalloc, test_range: "-300" }
    +        - { runner: ubuntu-24.04, compiler: gcc, testsuite: test,
    libs: -ljemalloc, test_range: "301-600" }
    +        - { runner: ubuntu-24.04, compiler: gcc, testsuite: test,
    libs: -ljemalloc, test_range: "601-", unstable: unstable }
    +        - { runner: ubuntu-24.04, compiler: gcc, testsuite:
    system-test-dpdk, dpdk: dpdk, test_range: "-100" }
    +        - { runner: ubuntu-24.04, compiler: gcc, testsuite:
    system-test-dpdk, dpdk: dpdk, test_range: "101-200" }
    +        - { runner: ubuntu-24.04, compiler: gcc, testsuite:
    system-test-dpdk, dpdk: dpdk, test_range: "201-", unstable: unstable }
    +        - { runner: ubuntu-24.04, compiler: gcc, testsuite:
    system-test-userspace, test_range: "-100" }
    +        - { runner: ubuntu-24.04, compiler: gcc, testsuite:
    system-test-userspace, test_range: "101-200" }
    +        - { runner: ubuntu-24.04, compiler: gcc, testsuite:
    system-test-userspace, test_range: "201-", unstable: unstable }
    +        - { runner: ubuntu-24.04, compiler: gcc, testsuite:
    system-test, test_range: "-100" }
    +        - { runner: ubuntu-24.04, compiler: gcc, testsuite:
    system-test, test_range: "101-200" }
    +        - { runner: ubuntu-24.04, compiler: gcc, testsuite:
    system-test, test_range: "201-", unstable: unstable }
    +        - { runner: ubuntu-24.04, compiler: clang, testsuite:
    system-test, sanitizers: sanitizers, test_range: "-100" }
    +        - { runner: ubuntu-24.04, compiler: clang, testsuite:
    system-test, sanitizers: sanitizers, test_range: "101-200" }
    +        - { runner: ubuntu-24.04, compiler: clang, testsuite:
    system-test, sanitizers: sanitizers, test_range: "201-", unstable:
    unstable }
    +        - { runner: ubuntu-24.04, arch: x86, compiler: gcc, opts:
    --disable-ssl }
    +        # ARM builds
    +        - { runner: ubuntu-24.04-arm, compiler: gcc, opts:
    --disable-ssl }
    +        - { runner: ubuntu-24.04-arm, compiler: gcc, testsuite:
    test, test_range: "-300" }
    +        - { runner: ubuntu-24.04-arm, compiler: gcc, testsuite:
    test, test_range: "301-600" }
    +        - { runner: ubuntu-24.04-arm, compiler: gcc, testsuite:
    test, test_range: "601-", unstable: unstable }
    +        - { runner: ubuntu-24.04-arm, compiler: gcc, testsuite:
    system-test, test_range: "-100" }
    +        - { runner: ubuntu-24.04-arm, compiler: gcc, testsuite:
    system-test, test_range: "101-200" }
    +        - { runner: ubuntu-24.04-arm, compiler: gcc, testsuite:
    system-test, test_range: "201-", unstable: unstable }

         steps:
         - name: system-level-dependencies
    @@ -176,7 +188,7 @@ jobs:
           uses: actions/cache@v4
           with:
             path: /tmp/image.tar
    -        key: ${{ github.sha }}/${{ github.event_name }}
    +        key: ${{ github.sha }}/${{ github.event_name }}/${{
    matrix.cfg.runner }}

         - name: load image
           run: |
-- 2.52.0 

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


Regards,
Ales


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

Reply via email to