[Yesterday while discussing the following issue on IRC, jgriffith suggested that I post to the dev list in preparation for a discussion in Wednesday's cinder meeting.]
Please take a look at the 10 "Low" priority reviews in the cinder Liberty 3 etherpad that were punted to Mitaka yesterday. [1] Six of these *never* [2] received a vote from a core reviewer. With the exception of the first in the list, which has 35 patch sets, none of the others received a vote before Friday, August 28. Of these, none had more than -1s on minor issues, and these have been remedied. Review https://review.openstack.org/#/c/213855 "Implement manage/unmanage snapshot in Pure drivers" is a great example: * approved blueprint for a valuable feature * pristine code * passes CI and Jenkins (and by the deadline) * never reviewed We have 11 core reviewers, all of whom were very busy doing reviews during L3, but evidently this set of reviews didn't really have much chance of making it. This looks like a classic case where the individually rational priority decisions of each core reviewer collectively resulted in starving the Low Priority review queue. One way to remedy would be for the 11 core reviewers to devote a day or two to cleaning up this backlog of 10 outstanding reviews rather than punting all of them out to Mitaka. Thanks for your time and consideration. Respectfully, -- Tom Barron [1] https://etherpad.openstack.org/p/cinder-liberty-3-reviews [2] At the risk of stating the obvious, in this count I ignore purely procedural votes such as the final -2. __________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev