Open Stack

Ah, I didn't realize it was in the ironic repo. 

Doug

> On Sep 6, 2014, at 2:35 AM, Michael Still <mikal at stillhq.com> wrote:
> 
> Well, we were only counting commits to the nova _driver_, not to
> ironic over all.
> 
> Michael
> 
>> On Sat, Sep 6, 2014 at 2:46 PM, Doug Hellmann <doug at doughellmann.com> wrote:
>> Would it make sense to count commits to the original repo as contributions under both nova and ironic instead of making a separate list?
>> 
>> Doug
>> 
>>> On Sep 5, 2014, at 11:38 PM, Michael Still <mikal at stillhq.com> wrote:
>>> 
>>> If I knew when you were going to generate the list of voting ATCs, I
>>> would be happy to propose a review against extra-atcs which lists all
>>> the co-authored-bys for nova before then. It would require a TC
>>> review, but we're actually pretty fast at those for mechanical things.
>>> 
>>> Michael
>>> 
>>>>> On Sat, Sep 6, 2014 at 12:03 PM, Anita Kuno <anteaya at anteaya.info> wrote:
>>>>> On 09/05/2014 07:44 PM, Russell Bryant wrote:
>>>>> 
>>>>> 
>>>>>> On Sep 5, 2014, at 7:09 PM, Michael Still <mikal at stillhq.com> wrote:
>>>>>> 
>>>>>> Hi.
>>>>>> 
>>>>>> I bring this up because Nova has a large example of this in Juno, and
>>>>>> I want to make sure we handle it nicely. If this is the wrong list,
>>>>>> feel free to cc others.
>>>>>> 
>>>>>> I'd like to see co-authored-by confer ATC status. Or, I'd like to at
>>>>>> least see a discussion on if it should.
>>>>>> 
>>>>>> This is of interest for me because the ironic nova driver is in the
>>>>>> process of merging, and we've used co-authored-by to acknowledge the
>>>>>> contributions of the various authors when we lose the git history from
>>>>>> the remote repository. It think its a nice touch, but I'd also like to
>>>>>> see those authors get ATC.
>>>>> 
>>>>> I thought this was already the case, actually. Absolutely +1 if not.
>>>> Jeremy can correct me if I am wrong, but I believe this is the script
>>>> that he uses to create the electoral roll:
>>>> http://git.openstack.org/cgit/openstack-infra/config/tree/tools/atc/email_stats.py
>>>> This script generates the list of emails I need to use for the poll to
>>>> add the emails of the electorate.
>>>> 
>>>> So one option is to offer a patch to extend that script to include
>>>> co-authored by if we can find a gerrit query that will give results for
>>>> Co-Authored. (I'm working on it and have come up empty thus far:
>>>> https://review.openstack.org/Documentation/user-search.html) If we did
>>>> this we should probably also ensure a governance resolution is merged
>>>> prior to September 26, 2014 05:59 UTC identifying that the definition of
>>>> "members who committed a change to a repository" as outlined in the
>>>> charter is expanded to include co-authored by, just to dot all the i's
>>>> and cross all the t's.
>>>> 
>>>> Another option is to submit a list of names of contributors that have
>>>> status by way of co-authored-by but not by owner of a commit as
>>>> extra-atc contributors.
>>>> http://git.openstack.org/cgit/openstack/governance/tree/reference/extra-atcs
>>>> There is already a mechanism to include names in this file, both by
>>>> governance decision and the above script. This would require the list of
>>>> names to be compiled by nova, submitted to the tc and merged to the
>>>> extra-atc file prior to September 26, 2014 05:59 UTC. There is time for
>>>> this to happen.
>>>> 
>>>> My input is based on finding options to make the technical/governance
>>>> side of this work. I have no opinion on whether this should be
>>>> undertaken as that decision is up to the tc.
>>>> 
>>>> Thanks,
>>>> Anita.
>>> 
>>> 
>>> 
>>> --
>>> Rackspace Australia
>>> 
>>> _______________________________________________
>>> OpenStack-TC mailing list
>>> OpenStack-TC at lists.openstack.org
>>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-tc
> 
> 
> 
> -- 
> Rackspace Australia