<div dir="ltr">If we are using Co-Authored-By tags why not add something to automatically look for them in the git logs, I have a hunch other projects will have similar cases?<div><br></div><div>git log --grep=Co-Authored-By --since=6.months | grep "Co-Authored-By" | sort | uniq<br></div><div><br></div><div>Gives a quick and dirty list that can be compared to the regular author list.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Sep 7, 2014 at 4:38 PM, Michael Still <span dir="ltr"><<a href="mailto:mikal@stillhq.com" target="_blank">mikal@stillhq.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><a href="https://review.openstack.org/#/c/119666/" target="_blank">https://review.openstack.org/#/c/119666/</a> is the first pass, I'll do a<br>
catch up closer to the election if its required.<br>
<br>
Michael<br>
<div><div><br>
On Sun, Sep 7, 2014 at 10:28 AM, Anita Kuno <<a href="mailto:anteaya@anteaya.info" target="_blank">anteaya@anteaya.info</a>> wrote:<br>
> On 09/05/2014 11:38 PM, Michael Still wrote:<br>
>> If I knew when you were going to generate the list of voting ATCs, I<br>
>> would be happy to propose a review against extra-atcs which lists all<br>
>> the co-authored-bys for nova before then. It would require a TC<br>
>> review, but we're actually pretty fast at those for mechanical things.<br>
>><br>
>> Michael<br>
> This seems to be the easiest/fastest way to accomplish your goal.<br>
><br>
> The electoral rolls will be generated after September 26, 2014 05:59<br>
> UTC, so please have any and all changes merged to the extra-atcs file<br>
> prior to that date/time to ensure those names are included in the<br>
> intended projects for the ptl and tc electoral rolls. (The cut off date<br>
> for eligibility is the same for both elections.)<br>
><br>
> Thanks, let me know if there are any other questions/details I can help<br>
> with,<br>
> Anita.<br>
>><br>
>> On Sat, Sep 6, 2014 at 12:03 PM, Anita Kuno <<a href="mailto:anteaya@anteaya.info" target="_blank">anteaya@anteaya.info</a>> wrote:<br>
>>> On 09/05/2014 07:44 PM, Russell Bryant wrote:<br>
>>>><br>
>>>><br>
>>>>> On Sep 5, 2014, at 7:09 PM, Michael Still <<a href="mailto:mikal@stillhq.com" target="_blank">mikal@stillhq.com</a>> wrote:<br>
>>>>><br>
>>>>> Hi.<br>
>>>>><br>
>>>>> I bring this up because Nova has a large example of this in Juno, and<br>
>>>>> I want to make sure we handle it nicely. If this is the wrong list,<br>
>>>>> feel free to cc others.<br>
>>>>><br>
>>>>> I'd like to see co-authored-by confer ATC status. Or, I'd like to at<br>
>>>>> least see a discussion on if it should.<br>
>>>>><br>
>>>>> This is of interest for me because the ironic nova driver is in the<br>
>>>>> process of merging, and we've used co-authored-by to acknowledge the<br>
>>>>> contributions of the various authors when we lose the git history from<br>
>>>>> the remote repository. It think its a nice touch, but I'd also like to<br>
>>>>> see those authors get ATC.<br>
>>>><br>
>>>> I thought this was already the case, actually. Absolutely +1 if not.<br>
>>>><br>
>>> Jeremy can correct me if I am wrong, but I believe this is the script<br>
>>> that he uses to create the electoral roll:<br>
>>> <a href="http://git.openstack.org/cgit/openstack-infra/config/tree/tools/atc/email_stats.py" target="_blank">http://git.openstack.org/cgit/openstack-infra/config/tree/tools/atc/email_stats.py</a><br>
>>> This script generates the list of emails I need to use for the poll to<br>
>>> add the emails of the electorate.<br>
>>><br>
>>> So one option is to offer a patch to extend that script to include<br>
>>> co-authored by if we can find a gerrit query that will give results for<br>
>>> Co-Authored. (I'm working on it and have come up empty thus far:<br>
>>> <a href="https://review.openstack.org/Documentation/user-search.html" target="_blank">https://review.openstack.org/Documentation/user-search.html</a>) If we did<br>
>>> this we should probably also ensure a governance resolution is merged<br>
>>> prior to September 26, 2014 05:59 UTC identifying that the definition of<br>
>>> "members who committed a change to a repository" as outlined in the<br>
>>> charter is expanded to include co-authored by, just to dot all the i's<br>
>>> and cross all the t's.<br>
>>><br>
>>> Another option is to submit a list of names of contributors that have<br>
>>> status by way of co-authored-by but not by owner of a commit as<br>
>>> extra-atc contributors.<br>
>>> <a href="http://git.openstack.org/cgit/openstack/governance/tree/reference/extra-atcs" target="_blank">http://git.openstack.org/cgit/openstack/governance/tree/reference/extra-atcs</a><br>
>>> There is already a mechanism to include names in this file, both by<br>
>>> governance decision and the above script. This would require the list of<br>
>>> names to be compiled by nova, submitted to the tc and merged to the<br>
>>> extra-atc file prior to September 26, 2014 05:59 UTC. There is time for<br>
>>> this to happen.<br>
>>><br>
>>> My input is based on finding options to make the technical/governance<br>
>>> side of this work. I have no opinion on whether this should be<br>
>>> undertaken as that decision is up to the tc.<br>
>>><br>
>>> Thanks,<br>
>>> Anita.<br>
>><br>
>><br>
>><br>
><br>
<br>
<br>
<br>
</div></div><span><font color="#888888">--<br>
Rackspace Australia<br>
</font></span><div><div><br>
_______________________________________________<br>
OpenStack-TC mailing list<br>
<a href="mailto:OpenStack-TC@lists.openstack.org" target="_blank">OpenStack-TC@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-tc" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-tc</a><br>
</div></div></blockquote></div><br></div></div>