<div dir="ltr">Hi all,<div>as of my larger proposal on improvements to code review workflow [1], we need to have cores for repositories, not for the whole Fuel. It is the path we are taking for a while, and new core reviewers added to specific repos only. Now we need to complete this work.</div><div><br></div><div>My proposal is:</div><div><ol><li>Get rid of one common fuel-core [2] group, members of which can merge code anywhere in Fuel. Some members of this group may cover a couple of repositories, but can't really be cores in all repos.</li><li>Extend existing groups, such as fuel-library [3], with members from fuel-core who are keeping up with large number of reviews / merges. This data can be queried at Stackalytics.</li><li>Establish a new group "fuel-infra", and ensure that it's included into any other core group. This is for maintenance purposes, it is expected to be used only in exceptional cases. Fuel Infra team will have to decide whom to include into this group.</li><li>Ensure that fuel-plugin-* repos will not be affected by removal of fuel-core group.</li></ol><div>#2 needs specific details. Stackalytics can show active cores easily, we can look at people with *: <a href="http://stackalytics.com/report/contribution/fuel-web/180">http://stackalytics.com/report/contribution/fuel-web/180</a>. This is for fuel-web, change the link for other repos accordingly. If people are added specifically to the particular group, leaving as is (some of them are no longer active. But let's clean them up separately from this group restructure process).</div></div><div><ul><li>fuel-library-core [3] group will have following members: Bogdan D., Sergii G., Alex Schultz, Vladimir Kuklin, Alex Didenko. </li><li>fuel-web-core [4]: Sebastian K., Igor Kalnitsky, Alexey Kasatkin, Vitaly Kramskikh, Julia Aranovich, Evgeny Li, Dima Shulyak</li><li>fuel-astute-core [5]: Vladimir Sharshov, Evgeny Li</li><li>fuel-dev-tools-core [6]: Przemek Kaminski, Sebastian K.</li><li>fuel-devops-core [7]: Tatyana Leontovich, Andrey Sledzinsky, Nastya Urlapova</li><li>fuel-docs-core [8]: Irina Povolotskaya, Denis Klepikov, Evgeny Konstantinov, Olga Gusarenko</li><li>fuel-main-core [9]: Vladimir Kozhukalov, Roman Vyalov, Dmitry Pyzhov, Sergii Golovatyuk, Vladimir Kuklin, Igor Kalnitsky</li><li>fuel-nailgun-agent-core [10]: Vladimir Sharshov, V.Kozhukalov</li><li>fuel-ostf-core [11]: Tatyana Leontovich, Nastya Urlapova, Andrey Sledzinsky, Dmitry Shulyak</li><li>fuel-plugins-core [12]: Igor Kalnitsky, Evgeny Li, Alexey Kasatkin</li><li>fuel-qa-core [13]: Andrey Sledzinsky, Tatyana Leontovich, Nastya Urlapova</li><li>fuel-stats-core [14]: Alex Kislitsky, Alexey Kasatkin, Vitaly Kramskikh</li><li>fuel-tasklib-core [15]: Igor Kalnitsky, Dima Shulyak, Alexey Kasatkin (this project seems to be dead, let's consider to rip it off)</li><li>fuel-specs-core: there is no such a group at the moment. I propose to create one with following members, based on stackalytics data [16]: Vitaly Kramskikh, Bogdan Dobrelia, Evgeny Li, Sergii Golovatyuk, Vladimir Kuklin, Igor Kalnitsky, Alexey Kasatkin, Roman Vyalov, Dmitry Borodaenko, Mike Scherbakov, Dmitry Pyzhov. We would need to reconsider who can merge after Fuel PTL/Component Leads elections</li><li>fuel-octane-core: needs to be created. Members: Yury Taraday, Oleg Gelbukh, Ilya Kharin</li><li>fuel-mirror-core: needs to be created. Sergey Kulanov, Vitaly Parakhin</li><li>fuel-upgrade-core: needs to be created. Sebastian Kalinowski, Alex Schultz, Evgeny Li, Igor Kalnitsky</li><li>fuel-provision: repo seems to be outdated, needs to be removed.</li></ul></div><div>I suggest to make changes in groups first, and then separately address specific issues like removing someone from cores (not doing enough reviews anymore or too many positive reviews, let's say > 95%).</div><div><br></div><div>I hope I don't miss anyone / anything. Please check carefully.</div><div>Comments / objections?</div><div><br></div><div>[1] <a href="http://lists.openstack.org/pipermail/openstack-dev/2015-August/072406.html">http://lists.openstack.org/pipermail/openstack-dev/2015-August/072406.html</a></div><div>[2] <a href="https://review.openstack.org/#/admin/groups/209,members">https://review.openstack.org/#/admin/groups/209,members</a></div><div>[3] <a href="https://review.openstack.org/#/admin/groups/658,members">https://review.openstack.org/#/admin/groups/658,members</a><br></div><div>[4] <a href="https://review.openstack.org/#/admin/groups/664,members">https://review.openstack.org/#/admin/groups/664,members</a></div><div>[5] <a href="https://review.openstack.org/#/admin/groups/655,members">https://review.openstack.org/#/admin/groups/655,members</a></div><div>[6] <a href="https://review.openstack.org/#/admin/groups/646,members">https://review.openstack.org/#/admin/groups/646,members</a><br></div><div>[7] <a href="https://review.openstack.org/#/admin/groups/656,members">https://review.openstack.org/#/admin/groups/656,members</a></div><div>[8] <a href="https://review.openstack.org/#/admin/groups/657,members">https://review.openstack.org/#/admin/groups/657,members</a></div><div>[9] <a href="https://review.openstack.org/#/admin/groups/659,members">https://review.openstack.org/#/admin/groups/659,members</a></div><div>[10] <a href="https://review.openstack.org/#/admin/groups/1000,members">https://review.openstack.org/#/admin/groups/1000,members</a></div><div>[11] <a href="https://review.openstack.org/#/admin/groups/660,members">https://review.openstack.org/#/admin/groups/660,members</a></div><div>[12] <a href="https://review.openstack.org/#/admin/groups/661,members">https://review.openstack.org/#/admin/groups/661,members</a><br></div><div>[13] <a href="https://review.openstack.org/#/admin/groups/662,members">https://review.openstack.org/#/admin/groups/662,members</a></div><div>[14] <a href="https://review.openstack.org/#/admin/groups/663,members">https://review.openstack.org/#/admin/groups/663,members</a></div><div>[15] <a href="https://review.openstack.org/#/admin/groups/624,members">https://review.openstack.org/#/admin/groups/624,members</a></div><div>[16] <a href="http://stackalytics.com/report/contribution/fuel-specs/180">http://stackalytics.com/report/contribution/fuel-specs/180</a><br></div><div><br></div><div><br></div></div><div dir="ltr">-- <br></div><div dir="ltr">Mike Scherbakov<br>#mihgen</div>