Open Stack

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.