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 <[email protected]> > > Signed-off-by: Roger Pau Monné <[email protected]> > > --- > > 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 <[email protected]>
