Re: Pull Request Auto-labeling

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

 



Great work Ernesto! This will be an excellent building block for further CI automation.

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




[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