mbox series

[ovs-dev,0/7] Bump of CI Ubuntu and Fedora versions

Message ID 20240514083851.417951-1-amusil@redhat.com
Headers show
Series Bump of CI Ubuntu and Fedora versions | expand

Message

Ales Musil May 14, 2024, 8:38 a.m. UTC 
The series is pretty small however it is required for the
bump to Ubuntu 24.04 and Fedora 40. Both have newer GCC and
Clang which brought up some issues that needed to be fixed.

The series also includes fix for weekly runs to use Fedora
because the cache string wasn't specific enough.

Ales Musil (7):
  ci: Pin Fedora version for the build-rpm job.
  ovs: Bump the submodule to the tip of branch-3.3.
  ci: Update the Ubuntu container to 24.04.
  tests: Replace wget with curl for failing commands.
  ci: Add missing packages to run Fedora image in GH CI.
  ci: Make sure that we are using proper image.
  ci: Bump the Fedora container to 40.

 .ci/linux-build.sh                     |  3 +++
 .github/workflows/test.yml             |  7 ++++---
 ovs                                    |  2 +-
 tests/system-ovn.at                    | 15 +++++++--------
 utilities/containers/fedora/Dockerfile |  3 ++-
 utilities/containers/ubuntu/Dockerfile | 11 ++++++++++-
 6 files changed, 27 insertions(+), 14 deletions(-)

Comments

Numan Siddique May 16, 2024, 3:57 p.m. UTC | #1 
On Tue, May 14, 2024 at 4:41 AM Ales Musil <amusil@redhat.com> wrote:
>
> The series is pretty small however it is required for the
> bump to Ubuntu 24.04 and Fedora 40. Both have newer GCC and
> Clang which brought up some issues that needed to be fixed.
>
> The series also includes fix for weekly runs to use Fedora
> because the cache string wasn't specific enough.
>
> Ales Musil (7):
>   ci: Pin Fedora version for the build-rpm job.
>   ovs: Bump the submodule to the tip of branch-3.3.
>   ci: Update the Ubuntu container to 24.04.
>   tests: Replace wget with curl for failing commands.
>   ci: Add missing packages to run Fedora image in GH CI.
>   ci: Make sure that we are using proper image.
>   ci: Bump the Fedora container to 40.

Thanks for the patch series.

I applied this patch series to the main.  Before applying I removed
"dnf update" from .ci/linux-build.sh  as per
the comments in the patch 5.

Thanks
Numan

>
>  .ci/linux-build.sh                     |  3 +++
>  .github/workflows/test.yml             |  7 ++++---
>  ovs                                    |  2 +-
>  tests/system-ovn.at                    | 15 +++++++--------
>  utilities/containers/fedora/Dockerfile |  3 ++-
>  utilities/containers/ubuntu/Dockerfile | 11 ++++++++++-
>  6 files changed, 27 insertions(+), 14 deletions(-)
>
> --
> 2.44.0
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
Ales Musil May 16, 2024, 4 p.m. UTC | #2 
On Thu, May 16, 2024 at 5:57 PM Numan Siddique <numans@ovn.org> wrote:

> On Tue, May 14, 2024 at 4:41 AM Ales Musil <amusil@redhat.com> wrote:
> >
> > The series is pretty small however it is required for the
> > bump to Ubuntu 24.04 and Fedora 40. Both have newer GCC and
> > Clang which brought up some issues that needed to be fixed.
> >
> > The series also includes fix for weekly runs to use Fedora
> > because the cache string wasn't specific enough.
> >
> > Ales Musil (7):
> >   ci: Pin Fedora version for the build-rpm job.
> >   ovs: Bump the submodule to the tip of branch-3.3.
> >   ci: Update the Ubuntu container to 24.04.
> >   tests: Replace wget with curl for failing commands.
> >   ci: Add missing packages to run Fedora image in GH CI.
> >   ci: Make sure that we are using proper image.
> >   ci: Bump the Fedora container to 40.
>
> Thanks for the patch series.
>
> I applied this patch series to the main.  Before applying I removed
> "dnf update" from .ci/linux-build.sh  as per
> the comments in the patch 5.
>


Thank you, would you mind backporting only the first patch to 24.03?

>
> Thanks
> Numan
>
> >
> >  .ci/linux-build.sh                     |  3 +++
> >  .github/workflows/test.yml             |  7 ++++---
> >  ovs                                    |  2 +-
> >  tests/system-ovn.at                    | 15 +++++++--------
> >  utilities/containers/fedora/Dockerfile |  3 ++-
> >  utilities/containers/ubuntu/Dockerfile | 11 ++++++++++-
> >  6 files changed, 27 insertions(+), 14 deletions(-)
> >
> > --
> > 2.44.0
> >
> > _______________________________________________
> > dev mailing list
> > dev@openvswitch.org
> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> >
>
>
Thanks,
Ales
Numan Siddique May 16, 2024, 4:21 p.m. UTC | #3 
On Thu, May 16, 2024 at 12:01 PM Ales Musil <amusil@redhat.com> wrote:
>
> On Thu, May 16, 2024 at 5:57 PM Numan Siddique <numans@ovn.org> wrote:
>
> > On Tue, May 14, 2024 at 4:41 AM Ales Musil <amusil@redhat.com> wrote:
> > >
> > > The series is pretty small however it is required for the
> > > bump to Ubuntu 24.04 and Fedora 40. Both have newer GCC and
> > > Clang which brought up some issues that needed to be fixed.
> > >
> > > The series also includes fix for weekly runs to use Fedora
> > > because the cache string wasn't specific enough.
> > >
> > > Ales Musil (7):
> > >   ci: Pin Fedora version for the build-rpm job.
> > >   ovs: Bump the submodule to the tip of branch-3.3.
> > >   ci: Update the Ubuntu container to 24.04.
> > >   tests: Replace wget with curl for failing commands.
> > >   ci: Add missing packages to run Fedora image in GH CI.
> > >   ci: Make sure that we are using proper image.
> > >   ci: Bump the Fedora container to 40.
> >
> > Thanks for the patch series.
> >
> > I applied this patch series to the main.  Before applying I removed
> > "dnf update" from .ci/linux-build.sh  as per
> > the comments in the patch 5.
> >
>
>
> Thank you, would you mind backporting only the first patch to 24.03?

Sure.

Numan

>
> >
> > Thanks
> > Numan
> >
> > >
> > >  .ci/linux-build.sh                     |  3 +++
> > >  .github/workflows/test.yml             |  7 ++++---
> > >  ovs                                    |  2 +-
> > >  tests/system-ovn.at                    | 15 +++++++--------
> > >  utilities/containers/fedora/Dockerfile |  3 ++-
> > >  utilities/containers/ubuntu/Dockerfile | 11 ++++++++++-
> > >  6 files changed, 27 insertions(+), 14 deletions(-)
> > >
> > > --
> > > 2.44.0
> > >
> > > _______________________________________________
> > > dev mailing list
> > > dev@openvswitch.org
> > > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> > >
> >
> >
> Thanks,
> Ales
> --
>
> Ales Musil
>
> Senior Software Engineer - OVN Core
>
> Red Hat EMEA <https://www.redhat.com>
>
> amusil@redhat.com
> <https://red.ht/sig>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev