On Sat, Mar 11, 2017 at 11:06:18AM -0700, Orion Poplawski wrote: > Christopher Meng (cicku) was declared unresponsive a while back. > However, there are still a lot of reviews associated with him: > > Submitted by him (60): > https://bugzilla.redhat.com/buglist.cgi?bug_status=NEW&bug_status=ASSIGNED&bug_status=POST&bug_status=MODIFIED&bug_status=ON_DEV&bug_status=ON_QA&bug_status=VERIFIED&bug_status=RELEASE_PENDING&classification=Fedora&component=Package%20Review&email1=i%40cicku.me&emailreporter1=1&emailtype1=substring&list_id=7172450&product=Fedora&query_format=advanced Better search: https://bugzilla.redhat.com/buglist.cgi?bug_status=NEW&bug_status=ASSIGNED&bug_status=POST&bug_status=ON_DEV&bug_status=ON_QA&bug_status=VERIFIED&bug_status=RELEASE_PENDING&classification=Fedora&component=Package%20Review&email1=i%40cicku.me&emailreporter1=1&emailtype1=substring&list_id=7172480&product=Fedora&query_format=advanced > Assigned to him (23): > https://bugzilla.redhat.com/buglist.cgi?bug_status=NEW&bug_status=ASSIGNED&bug_status=POST&bug_status=MODIFIED&bug_status=ON_DEV&bug_status=ON_QA&bug_status=VERIFIED&bug_status=RELEASE_PENDING&classification=Fedora&component=Package%20Review&email1=i%40cicku.me&emailassigned_to1=1&emailtype1=substring&list_id=7172451&product=Fedora&query_format=advanced > > It seems like we need to have a procedure for dealing with reviews > for unresponsive maintainers as well. ... or just extend the unresponsive maintainer procedure to apply to reviews too, and just reassign all reviews as NEW/nobody when the reviewer is declared unresponsive. Review requests are more complicated. My initial comment was to be that they are not a problem, but random sampling shows that people try to ping the submitter in the review tickets, and of course get no answer. So we should do something to avoid this waste of time. Maybe we should drop a comment in each of those, something like: The submitter of this review request has been declared unresponsive (fesco ticket link), and this review is effectively dead. Anyone who wants to pick up this package, should open a new review request ticket, and close this one as DUPLICATE of the new one. An easier thing would be to make those bugs block FE-DEADREVIEW but that's fairly easy to miss, and not as clear, especially to newcomers. Zbyszek _______________________________________________ devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
