Open Stack

On 12/04/2014 05:21 AM, Mathieu Rohon wrote:
> On Thu, Dec 4, 2014 at 8:38 AM, Sumit Naiksatam
> <sumitnaiksatam at gmail.com> wrote:
>> On Wed, Dec 3, 2014 at 9:07 PM, Adam Young <ayoung at redhat.com> wrote:
>>> On 12/03/2014 06:24 PM, Sukhdev Kapur wrote:
>>>
>>> Congratulations Henry and Kevin. It has always been pleasure working with
>>> you guys.....
>>>
>>>
>>> If I may express my opinion, Bob's contribution to ML2 has been quite
>>> substantial. The kind of stability ML2 has achieved makes a statement of his
>>> dedication to this work. I have worked very closely with Bob on several
>>> issues and co-chaired ML2-Subteam with him and have developed tremendous
>>> respect for his dedication.
>>> Reading his email reply makes me believe he wants to continue to contribute
>>> as core developer. Therefore, I would like to take an opportunity to appeal
>>> to the core team to consider granting him his wish - i.e. vote -1 on his
>>> removal.
>>>
>>> If I might venture an outside voice in support of Bob:  you don't want to
>>> chase away the continuity.  Yes, sometimes the day job makes us focus on
>>> things other than upstream work for a while, but I would say that you should
>>> err on the side of keeping someone that is otherwise still engaged.
>>> Especially when that core has been as fundamental on a project as I know Bob
>>> to have been on Quantum....er Neutron.
>>
>> I would definitely echo the above sentiments; Bob has continually made
>> valuable design contributions to ML2 and Neutron that go beyond the
>> review count metric. Kindly consider keeping him as a part of the core
>> team.
> 
> Working with bob in the ML2 sub team was a real pleasure. He is
> providing a good technical and community leadership.
> Its reviews are really valuable, since he always reviews a patch in
> the context of the overall project and other work in progress.
> This takes more time.
> 
>> That said, a big +1 to both, Henry and Kevin, as additions to the core
>> team! Welcome!!
>>
>> Thanks,
>> ~Sumit.
>>
>>>
>>>
>>>
>>>
>>>
>>>
>>> regards..
>>> -Sukhdev
>>>
>>>
>>>
>>>
>>>
>>>
>>> On Wed, Dec 3, 2014 at 11:48 AM, Edgar Magana <edgar.magana at workday.com>
>>> wrote:
>>>>
>>>> I give +2 to Henry and Kevin. So, Congratulations Folks!
>>>> I have been working with both of them and great quality reviews are always
>>>> coming out from them.
>>>>
>>>> Many thanks to Nachi and Bob for their hard work!
>>>>
>>>> Edgar
>>>>
>>>> On 12/2/14, 7:59 AM, "Kyle Mestery" <mestery at mestery.com> wrote:
>>>>
>>>>> Now that we're in the thick of working hard on Kilo deliverables, I'd
>>>>> like to make some changes to the neutron core team. Reviews are the
>>>>> most important part of being a core reviewer, so we need to ensure
>>>>> cores are doing reviews. The stats for the 180 day period [1] indicate
>>>>> some changes are needed for cores who are no longer reviewing.
>>>>>
>>>>> First of all, I'm proposing we remove Bob Kukura and Nachi Ueno from
>>>>> neutron-core. Bob and Nachi have been core members for a while now.
>>>>> They have contributed to Neutron over the years in reviews, code and
>>>>> leading sub-teams. I'd like to thank them for all that they have done
>>>>> over the years. I'd also like to propose that should they start
>>>>> reviewing more going forward the core team looks to fast track them
>>>>> back into neutron-core. But for now, their review stats place them
>>>>> below the rest of the team for 180 days.
>>>>>
>>>>> As part of the changes, I'd also like to propose two new members to
>>>>> neutron-core: Henry Gessau and Kevin Benton. Both Henry and Kevin have
>>>>> been very active in reviews, meetings, and code for a while now. Henry
>>>>> lead the DB team which fixed Neutron DB migrations during Juno. Kevin
>>>>> has been actively working across all of Neutron, he's done some great
>>>>> work on security fixes and stability fixes in particular. Their
>>>>> comments in reviews are insightful and they have helped to onboard new
>>>>> reviewers and taken the time to work with people on their patches.
>>>>>
>>>>> Existing neutron cores, please vote +1/-1 for the addition of Henry
>>>>> and Kevin to the core team.
>>>>>
>>>>> Thanks!
>>>>> Kyle
>>>>>
>>>>> [1] http://stackalytics.com/report/contribution/neutron-group/180
>>>>>
I think we move into very dangerous territory if we are equating a core
review Gerrit permission (it is just a Gerrit permission, if it is
perceived as anything other than that that is a perception we have
created ourselves) with value as an OpenStack contributor.

I work with and value many OpenStack contributors daily who do not have
a core review Gerrit permission. I do not advocate for any decision that
even indirectly conveys the message that I value their contribution any
differently then any other contributor simply based on Gerrit permissions.

As to who in Neutron should or should not have certain Gerrit
permissions is not for me to say, as I do not have the perspective
(other than third party things) to have an opinion.

Thank you,
Anita.