On Thu, Nov 16, 2023 at 6:26 PM Dumitru Ceara <[email protected]> wrote:

> We initially thought always using the latest releases would be more
> maintainable because we didn't have to bump versions manually.  It turns
> out that it's the opposite, CI suddenly starts to fail on versions where
> it used to pass.
>
> To avoid ever changing tools in our CI due to newer releases of
> Python/Fedora/Ubuntu.  We can always bump versions manually.
>
> Fixes: 5ee07b32a01d ("ci: Change all GitHub CI jobs to use ubuntu-latest.")
> Fixes: 60a53abaa38a ("ci: Add automation for building the containers")
> Signed-off-by: Dumitru Ceara <[email protected]>
> ---
>

Hi Dumitru,

thank you for the v3. I have one small question below.


>  .github/workflows/containers.yml               |    2 +-
>  .github/workflows/ovn-fake-multinode-tests.yml |    6 +++---
>  .github/workflows/ovn-kubernetes.yml           |    4 ++--
>  .github/workflows/test.yml                     |    6 +++---
>  utilities/containers/fedora/Dockerfile         |    2 +-
>  5 files changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/.github/workflows/containers.yml
> b/.github/workflows/containers.yml
> index 57e815ed86..bdd1180872 100644
> --- a/.github/workflows/containers.yml
> +++ b/.github/workflows/containers.yml
> @@ -15,7 +15,7 @@ env:
>
>  jobs:
>    container:
> -    runs-on: ubuntu-latest
> +    runs-on: ubuntu-22.04
>      strategy:
>        matrix:
>          distro: [ fedora, ubuntu ]
> diff --git a/.github/workflows/ovn-fake-multinode-tests.yml
> b/.github/workflows/ovn-fake-multinode-tests.yml
> index 9a5cd83a65..25610df534 100644
> --- a/.github/workflows/ovn-fake-multinode-tests.yml
> +++ b/.github/workflows/ovn-fake-multinode-tests.yml
> @@ -13,7 +13,7 @@ concurrency:
>  jobs:
>    build:
>      name: Build ovn-fake-multinode image
> -    runs-on: ubuntu-latest
> +    runs-on: ubuntu-22.04
>      strategy:
>        matrix:
>          cfg:
> @@ -69,7 +69,7 @@ jobs:
>          path: /tmp/_output/ovn_${{ matrix.cfg.branch }}_image.tar
>
>    multinode-tests:
> -    runs-on: ubuntu-latest
> +    runs-on: ubuntu-22.04
>      timeout-minutes: 15
>      needs: [build]
>      strategy:
> @@ -158,7 +158,7 @@ jobs:
>      - name: set up python
>        uses: actions/setup-python@v4
>        with:
> -        python-version: '3.x'
> +        python-version: '3.12'
>
>      - name: Check out ovn
>        uses: actions/checkout@v3
> diff --git a/.github/workflows/ovn-kubernetes.yml
> b/.github/workflows/ovn-kubernetes.yml
> index d9a91874ff..1689396d66 100644
> --- a/.github/workflows/ovn-kubernetes.yml
> +++ b/.github/workflows/ovn-kubernetes.yml
> @@ -24,7 +24,7 @@ env:
>  jobs:
>    build:
>      name: Build
> -    runs-on: ubuntu-latest
> +    runs-on: ubuntu-22.04
>      steps:
>      - name: Enable Docker experimental features
>        run: |
> @@ -62,7 +62,7 @@ jobs:
>    e2e:
>      name: e2e
>      if: github.event_name != 'schedule'
> -    runs-on: ubuntu-latest
> +    runs-on: ubuntu-22.04
>      timeout-minutes: 220
>      strategy:
>        fail-fast: false
> diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
> index 5c5ce6ed10..a5ccb7e4ae 100644
> --- a/.github/workflows/test.yml
> +++ b/.github/workflows/test.yml
> @@ -94,7 +94,7 @@ jobs:
>        SANITIZERS:  ${{ matrix.cfg.sanitizers }}
>
>      name: linux ${{ join(matrix.cfg.*, ' ') }}
> -    runs-on: ubuntu-latest
> +    runs-on: ubuntu-22.04
>
>      strategy:
>        fail-fast: false
> @@ -219,7 +219,7 @@ jobs:
>      - name: set up python
>        uses: actions/setup-python@v4
>        with:
> -        python-version: '3.x'
> +        python-version: '3.12'
>      - name: prepare
>        run:  ./.ci/osx-prepare.sh
>      - name: build
> @@ -233,7 +233,7 @@ jobs:
>
>    build-linux-rpm:
>      name: linux rpm fedora
> -    runs-on: ubuntu-latest
> +    runs-on: ubuntu-22.04
>      container: fedora:latest
>      timeout-minutes: 30
>
> diff --git a/utilities/containers/fedora/Dockerfile
> b/utilities/containers/fedora/Dockerfile
> index 4058d7f5be..066bb0b957 100755
> --- a/utilities/containers/fedora/Dockerfile
> +++ b/utilities/containers/fedora/Dockerfile
> @@ -1,4 +1,4 @@
> -FROM quay.io/fedora/fedora:latest
> +FROM registry.fedoraproject.org/fedora:latest


Why did we switch from quay? I guess it shouldn't matter, just curious.


>
>
>  ARG CONTAINERS_PATH
>
>
>
Thanks,
Ales

-- 

Ales Musil

Senior Software Engineer - OVN Core

Red Hat EMEA <https://www.redhat.com>

[email protected]
<https://red.ht/sig>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to