[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] cirrus-ci: add myself as maintainer



On Thu, 28 Apr 2022, Andrew Cooper wrote:
> On 28/04/2022 10:55, Roger Pau Monne wrote:
> > Given the testing done by Cirrus-CI is FreeBSD only introduce a new
> > section in the MAINTAINERS file to cover it and add myself as the
> > maintainer.
> >
> > Requested-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
> > Signed-off-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>
> > ---
> > FWIW, I wouldn't mind it being part of the "Continuous Integration
> > (CI)" section, but I understand maintainers there could prefer a
> > separate section since this is ATM FreeBSD only testing.
> 
> I don't think we have enough review bandwidth to separate things like
> this.  Plenty of changes to CI are dependency tweaks which cover all CI
> files in one go, so wouldn't be directly relevant to being FreeBSD. 
> Also some CI changes need superpowers in other systems.


Today, gitlab-ci and cirrus-ci are entirely different systems: they
don't share any containers or any other artifacts. So my preference is
to have a separate entry in the MAINTAINERS file as Roger did in this
patch. That would be more accurate in terms of roles, responsibilities
and expectations. If someone sends a patch for .cirrus.yml, I definitely
think Roger should be the one to have a look. My recommendation is to go
ahead with this patch. We can always merge the sections in the future if
the CI systems become more integrated.

Acked-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.