On 3/21/25 3:33 PM, Ales Musil wrote:
> On Fri, Mar 21, 2025 at 3:32 PM Dumitru Ceara via dev <
> ovs-dev@openvswitch.org> wrote:
> 
>> The test was still testing upgrades from 22.03 to HEAD, which jumps over
>> the latest LTS, 24.03.  That's an unsupported upgrade path.  Change the
>> test to check that upgrading from 24.03 to HEAD works.
>>
>> Signed-off-by: Dumitru Ceara <dce...@redhat.com>
>> ---
>>  .github/workflows/ovn-fake-multinode-tests.yml | 8 ++++----
>>  1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/.github/workflows/ovn-fake-multinode-tests.yml
>> b/.github/workflows/ovn-fake-multinode-tests.yml
>> index 7d97d0272f..7c33d0739b 100644
>> --- a/.github/workflows/ovn-fake-multinode-tests.yml
>> +++ b/.github/workflows/ovn-fake-multinode-tests.yml
>> @@ -19,7 +19,7 @@ jobs:
>>        matrix:
>>          cfg:
>>          - { repo: "${{ github.repository }}", branch: "${{
>> github.ref_name }}" }
>> -        - { repo: ovn-org/ovn, branch: "branch-22.03" }
>> +        - { repo: ovn-org/ovn, branch: "branch-24.03" }
>>      env:
>>        RUNC_CMD: podman
>>        OS_IMAGE: "fedora:39"
>> @@ -79,7 +79,7 @@ jobs:
>>        matrix:
>>          cfg:
>>          - { branch: "${{ github.ref_name }}", testsuiteflags: ""}
>> -        - { branch: "branch-22.03", testsuiteflags: "-k 'basic test'" }
>> +        - { branch: "branch-24.03", testsuiteflags: "-k 'basic test'" }
>>      name: multinode tests ${{ join(matrix.cfg.*, ' ') }}
>>      env:
>>        RUNC_CMD: podman
>> @@ -125,12 +125,12 @@ jobs:
>>
>>      - uses: actions/download-artifact@v4
>>        with:
>> -        name: test-branch-22.03-image
>> +        name: test-branch-24.03-image
>>
>>      - name: Load podman image
>>        run: |
>>          sudo podman load --input ovn_${{ github.ref_name }}_image.tar
>> -        sudo podman load --input ovn_branch-22.03_image.tar
>> +        sudo podman load --input ovn_branch-24.03_image.tar
>>
>>      - name: Check out ovn-fake-multi-node
>>        uses: actions/checkout@v4
>> --
>> 2.48.1
>>
>> _______________________________________________
>> dev mailing list
>> d...@openvswitch.org
>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>>
>>
> Looks good to me, thanks.
> Acked-by: Ales Musil <amu...@redhat.com>
> 

Thanks, Ales, for the review!

Applied to main.

Regards,
Dumitru


_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to