Re: Pull Request Auto-labeling

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

 



On Fri, Nov 13, 2020 at 8:26 PM Yuri Weinstein <yweinste@xxxxxxxxxx> wrote:
Great thx !
Any chance we can automatically add labels which would indicate what
suites are required to run in order to test them ?

Hey Yuri,

Can't we simply map component labels to teuthology suites? 
 
On Fri, Nov 13, 2020 at 11:22 AM Josh Durgin <jdurgin@xxxxxxxxxx> wrote:
Great work Ernesto! This will be an excellent building block for further
CI automation.

Thanks, Josh!
 
>
> Josh
>
> On 11/13/20 10:51 AM, Ernesto Puerta wrote:
> > Hi all,
> >
> > Kefu & I have just merged these PRs [1] that enable a Github Action to
> > automatically label PRs based on matching paths in the changeset
> > (minimatch syntax) [2]. The file where to specify such mappings is
> > .github/labeler.yml
> > <https://github.com/ceph/ceph/blob/master/.github/labeler.yml>:
> >
> > dashboard:
> >       - src/pybind/mgr/dashboard/**
> >       - qa/suites/rados/dashboard/**
> >       - qa/tasks/mgr/test_dashboard.py ...
> >
> >
> >
> > In this PoC, I only added the 'dashboard' and 'CI' labels, but I'd
> > encourage other teams to do the same with your own subdirs, and get rid,
> > as much as possible, of this repetitive task of manually setting labels.
> >
> > This is an example of how this GH action tagged a PR [3] with the
> > dashboard label:
> >
> > image.png
> >
> > In the long term we could explore how to generate/use both CODEOWNERS
> > and labeler.yml files from the same source, as there's high redundancy
> > between both (but not all teams have a Github team).
> >
> > [1] https://github.com/ceph/ceph/pull/38049
> > <https://github.com/ceph/ceph/pull/38049> and
> > https://github.com/ceph/ceph/pull/38060
> > <https://github.com/ceph/ceph/pull/38060>
> > [2] https://github.com/isaacs/minimatch
> > <https://github.com/isaacs/minimatch>
> > [3] https://github.com/ceph/ceph/pull/38065
> > <https://github.com/ceph/ceph/pull/38065>
> >
> > Kind Regards,
> >
> > Ernesto Puerta
> >
> > _______________________________________________
> > Dev mailing list -- dev@xxxxxxx
> > To unsubscribe send an email to dev-leave@xxxxxxx
> >
> _______________________________________________
> Dev mailing list -- dev@xxxxxxx
> To unsubscribe send an email to dev-leave@xxxxxxx

_______________________________________________
Dev mailing list -- dev@xxxxxxx
To unsubscribe send an email to dev-leave@xxxxxxx

[Index of Archives]     [CEPH Users]     [Ceph Devel]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux