[LTP] [PATCH] ci: add centos stream support
Petr Vorel
pvorel@suse.cz
Mon Nov 6 11:40:23 CET 2023
> Hi Petr,
> On Fri, Nov 3, 2023 at 5:14 PM Petr Vorel <pvorel@suse.cz> wrote:
> > > Hi Petr,
> > > On Thu, Nov 2, 2023 at 5:21 PM Petr Vorel <pvorel@suse.cz> wrote:
> > > > Hi Li,
> > > > > This is base on Peter's patch:
> > > > > CI: ReAplace CentOS 7 with openSUSE Leap 42.2, Ubuntu xenial
> > > > > Ci: https://github.com/wangli5665/ltp/actions/runs/6729379565
> > > > Reviewed-by: Petr Vorel <pvorel@suse.cz>
> > > Thanks!
> > > > ATM we have 18 jobs, IMHO we should not add more without removing
> > others
> > > > just to
> > > > have reasonable CI runtime.
> > > Your concern makes sense, but I have no idea which one could be removed.
> > I wonder if just adding CentOS 9 would be enough. Because in CI we test
> > just
> > compilation. The point of the public CI is to test the oldest distros,
> > newest
> > distros and something in between, that should cover the most of compilation
> > problems (no need to test *all* used distros to be safe).
> > But sure, both can stay if you want. The runtime is still reasonable.
> > NOTE: I did not care much in case of iputils, which have much faster build
> > than
> > LTP.
> > If we want to add both CentOS 8 and 9, we could remove some of
> > debian:oldstable (bullseye)
> > (similar kernel).
> > And, we could add just openSUSE Leap 42.2, which has older glibc and gcc
> > than
> > Ubuntu 16.04 LTS xenial. For sure there are more people which use xenial
> > than
> > old Leap 42.2, but as I said, the goal is to test the oldest distro we
> > maintainers really care about (and do not have extra work for nothing,
> > which is
> > IMHO keeping CentOS 7 compilable).
> > WDYT?
> Your explanation is quite useful, thanks!
> I'm fine to remove the CentOS8 build in this patch,
> should I resent the new patch or push it directly?
You can push it directly. If it does not apply, feel free to push my change
first. I'll post a note about keeping only openSUSE Leap 42.2 in the patchset.
Acked-by: Petr Vorel <pvorel@suse.cz>
Kind regards,
Petr
More information about the ltp
mailing list