> 4) Periodically, code from the new project(s) must be merged into Nova. > Only Nova core reviewers will have obviously +2a rights here. > I propose to do it on scheduled days before every milestone, differentiated > per driver to distribute the review effort (what about also having Nova core > reviewers assigned to each driver? Dan was suggesting something similar some > time ago).
FWIW, this is not what I had intended. I think that if you want (or need) to be in the "extras" tree, then that's where you are. Periodic syncs generate extra work and add the previously-mentioned confusion of "which driver is the official/best one?" I think that any driver that gets put into -extra gets removed from the mainline nova tree. If that driver has full CI testing and wants to be moved into the main tree, then that happens once. Having commit rights to the extras tree and periodic nearly-unattended or too-large-to-reasonably-review sync patches just sidesteps the process. That gains you the recgonition of being in the tree, without having to undergo the aggressive review and participate in the planning and coordination of the process that goes with it. That is NOT okay, IMHO. Sorry if that was unclear with the previous discussion. I'm not sure who else was thinking that those drivers would exist in both places, but I definitely was not. --Dan _______________________________________________ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
