[openstack-dev] [Fuel] Core Reviewers groups restructure

Aleksey Kasatkin akasatkin at mirantis.com
Mon Sep 21 16:07:09 UTC 2015


Hi,

Just a remark: python-fuelclient is missing here.


Aleksey Kasatkin


On Sun, Sep 20, 2015 at 3:56 PM, Mike Scherbakov <mscherbakov at mirantis.com>
wrote:

> Hi all,
> 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.
>
> My proposal is:
>
>    1. 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.
>    2. 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.
>    3. 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.
>    4. Ensure that fuel-plugin-* repos will not be affected by removal of
>    fuel-core group.
>
> #2 needs specific details. Stackalytics can show active cores easily, we
> can look at people with *:
> http://stackalytics.com/report/contribution/fuel-web/180. 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).
>
>    - fuel-library-core [3] group will have following members: Bogdan D.,
>    Sergii G., Alex Schultz, Vladimir Kuklin, Alex Didenko.
>    - fuel-web-core [4]: Sebastian K., Igor Kalnitsky, Alexey Kasatkin,
>    Vitaly Kramskikh, Julia Aranovich, Evgeny Li, Dima Shulyak
>    - fuel-astute-core [5]: Vladimir Sharshov, Evgeny Li
>    - fuel-dev-tools-core [6]: Przemek Kaminski, Sebastian K.
>    - fuel-devops-core [7]: Tatyana Leontovich, Andrey Sledzinsky, Nastya
>    Urlapova
>    - fuel-docs-core [8]: Irina Povolotskaya, Denis Klepikov, Evgeny
>    Konstantinov, Olga Gusarenko
>    - fuel-main-core [9]: Vladimir Kozhukalov, Roman Vyalov, Dmitry
>    Pyzhov, Sergii Golovatyuk, Vladimir Kuklin, Igor Kalnitsky
>    - fuel-nailgun-agent-core [10]: Vladimir Sharshov, V.Kozhukalov
>    - fuel-ostf-core [11]: Tatyana Leontovich, Nastya Urlapova, Andrey
>    Sledzinsky, Dmitry Shulyak
>    - fuel-plugins-core [12]: Igor Kalnitsky, Evgeny Li, Alexey Kasatkin
>    - fuel-qa-core [13]: Andrey Sledzinsky, Tatyana Leontovich, Nastya
>    Urlapova
>    - fuel-stats-core [14]: Alex Kislitsky, Alexey Kasatkin, Vitaly
>    Kramskikh
>    - fuel-tasklib-core [15]: Igor Kalnitsky, Dima Shulyak, Alexey
>    Kasatkin (this project seems to be dead, let's consider to rip it off)
>    - 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
>    - fuel-octane-core: needs to be created. Members: Yury Taraday, Oleg
>    Gelbukh, Ilya Kharin
>    - fuel-mirror-core: needs to be created. Sergey Kulanov, Vitaly
>    Parakhin
>    - fuel-upgrade-core: needs to be created. Sebastian Kalinowski, Alex
>    Schultz, Evgeny Li, Igor Kalnitsky
>    - fuel-provision: repo seems to be outdated, needs to be removed.
>
> 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%).
>
> I hope I don't miss anyone / anything. Please check carefully.
> Comments / objections?
>
> [1]
> http://lists.openstack.org/pipermail/openstack-dev/2015-August/072406.html
> [2] https://review.openstack.org/#/admin/groups/209,members
> [3] https://review.openstack.org/#/admin/groups/658,members
> [4] https://review.openstack.org/#/admin/groups/664,members
> [5] https://review.openstack.org/#/admin/groups/655,members
> [6] https://review.openstack.org/#/admin/groups/646,members
> [7] https://review.openstack.org/#/admin/groups/656,members
> [8] https://review.openstack.org/#/admin/groups/657,members
> [9] https://review.openstack.org/#/admin/groups/659,members
> [10] https://review.openstack.org/#/admin/groups/1000,members
> [11] https://review.openstack.org/#/admin/groups/660,members
> [12] https://review.openstack.org/#/admin/groups/661,members
> [13] https://review.openstack.org/#/admin/groups/662,members
> [14] https://review.openstack.org/#/admin/groups/663,members
> [15] https://review.openstack.org/#/admin/groups/624,members
> [16] http://stackalytics.com/report/contribution/fuel-specs/180
>
>
> --
> Mike Scherbakov
> #mihgen
>
> __________________________________________________________________________
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20150921/dcaafbdc/attachment.html>


More information about the OpenStack-dev mailing list