Open Stack

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