<div dir="ltr">+1 to Lukasz.<div>-1 to the proposal, we had it this way for a quite some time, and it was not good for the project (as Lukasz pointed out), why should a person who merges the code to the library have an access to merge the code to Nailgun/Astute without proper expertise. Those are different areas which require different skills.</div><div><br></div><div>Thanks,</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 6, 2016 at 9:40 AM, Lukasz Oles <span dir="ltr"><<a href="mailto:loles@mirantis.com" target="_blank">loles@mirantis.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, Sep 5, 2016 at 5:39 PM, Andrew Maksimov <<a href="mailto:amaksimov@mirantis.com">amaksimov@mirantis.com</a>> wrote:<br>
> +1<br>
> This is a good proposal, I also think we should have single fuel-core group<br>
> for all repos. In real life core reviewers won't set +2 or merge to repos<br>
> with which they are not familiar with.<br>
</span>Actually one of the reasons why core groups were split was that it<br>
happened a few times :)<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
> Regards,<br>
> Andrey Maximov<br>
><br>
> On Mon, Sep 5, 2016 at 2:11 PM, Vladimir Kozhukalov<br>
> <<a href="mailto:vkozhukalov@mirantis.com">vkozhukalov@mirantis.com</a>> wrote:<br>
>><br>
>> Dear colleagues,<br>
>><br>
>> I'd like to suggest to use common fuel-core group for all Fuel projects<br>
>> instead of having separate independent 'by-project' core groups like<br>
>> 'fuel-astute-core' or 'fuel-agent-core'.<br>
>><br>
>> Pros:<br>
>> 1) It will be easier to access core members (timezone and holiday<br>
>> tolerance)<br>
>> 2) It will be easier to manage single core group (promote new members,<br>
>> remove not active members)<br>
>><br>
>> Cons:<br>
>> 1) Less of flexibility. Permissions will be the same for all core<br>
>> reviewers in all Fuel projects.<br>
>><br>
>> What do you think?<br>
>><br>
>> Vladimir Kozhukalov<br>
>><br>
>> ______________________________<wbr>______________________________<wbr>______________<br>
>> OpenStack Development Mailing List (not for usage questions)<br>
>> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
>> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
>><br>
><br>
><br>
> ______________________________<wbr>______________________________<wbr>______________<br>
> OpenStack Development Mailing List (not for usage questions)<br>
> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
><br>
<br>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
Łukasz Oleś<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
______________________________<wbr>______________________________<wbr>______________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
</div></div></blockquote></div><br></div>