<div dir="ltr">Hi guys!<div><br></div><div>> <span style="color:rgb(0,0,0);font-size:12.8000001907349px">* Do not merge a patch without at least one review from both groups</span></div><div><span style="color:rgb(0,0,0);font-size:12.8000001907349px">> * </span><span style="color:rgb(0,0,0);font-size:12.8000001907349px">Collaborate to make the module compliant</span></div><div>I'm strongly agree with it, because it'll help to avoid some mistakes.</div><div>Also, someone from puppet-manager-core may to prompt what are we doing wrong or better way for elaboration of murano module.</div><div><br></div><div>Thanks!</div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-06-17 18:49 GMT+03:00 Serg Melikyan <span dir="ltr"><<a href="mailto:smelikyan@mirantis.com" target="_blank">smelikyan@mirantis.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Emilien,<br>
<br>
Thank you for your proposal, I completely agree with it:<br>
<span class=""><br>
> * Move the module under the big tent<br>
</span>Proposed change in infra [1] and corresponding change in governance [2]<br>
<span class=""><br>
> * Adding Puppet OpenStack group part of core permissions<br>
</span>I've added puppet-manager-core to puppet-murano-core<br>
<span class=""><br>
> * Keep Puppet Murano folks part of core permissions for now<br>
</span>I've added Denis Egorenko to puppet-murano-core (group was empty previously).<br>
<span class=""><br>
> * Do not merge a patch without at least one review from both groups<br>
> * Collaborate to make the module compliant<br>
</span>Denis will be responsible for initial review from Murano side in order<br>
to not overburden OpenStack Puppet with helping with Puppet basics. We<br>
will make sure to not merge anything without +2 from someone from<br>
puppet-manager-core. We will start with basic repository structure and<br>
will move existing module bit by bit.<br>
<span class=""><br>
> * When the module is compliant, we only keep Puppet OpenStack<br>
>   group managing the module, like it's done for other modules.<br>
</span>Sure!<br>
<br>
Once again thank you for your help and concerns!<br>
<br>
References:<br>
[1] <a href="https://review.openstack.org/192730" rel="noreferrer" target="_blank">https://review.openstack.org/192730</a><br>
[2] <a href="https://review.openstack.org/192727" rel="noreferrer" target="_blank">https://review.openstack.org/192727</a><br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, Jun 17, 2015 at 5:03 PM, Emilien Macchi <<a href="mailto:emilien@redhat.com">emilien@redhat.com</a>> wrote:<br>
><br>
><br>
> On 06/17/2015 09:50 AM, Serg Melikyan wrote:<br>
>> Thank you for sharing link to list of things that new module should<br>
>> satisfy to! It will be really helpful even if list will change over<br>
>> time. At least we have pointers how to start making our module<br>
>> compliant.<br>
>><br>
>> Regarding figuring out permissions - I don't mind if we will set<br>
>> puppet-core as group responsible for the repository, I believe that<br>
>> through contributing Murano module authors will get enough<br>
>> creditability to be included to the puppet-core. This will help to<br>
>> ensure that module is developed according all rules of Puppet<br>
>> OpenStack Community and nothing will be merged that does not satisfy<br>
>> adopted way of doing things. Emilien, if you agree with this approach<br>
>> I will send appropriate change to review.<br>
>><br>
><br>
> I like Monty's proposal.<br>
><br>
> I propose:<br>
> * Move the module under the big tent<br>
> * Adding Puppet OpenStack group part of core permissions<br>
> * Keep Puppet Murano folks part of core permissions for now<br>
> * Do not merge a patch without at least one review from both groups<br>
> * Collaborate to make the module compliant<br>
> * When the module is compliant, we only keep Puppet OpenStack group<br>
> managing the module, like it's done for other modules.<br>
><br>
>><br>
>><br>
>><br>
>><br>
>> On Wed, Jun 17, 2015 at 4:08 PM, Monty Taylor <<a href="mailto:mordred@inaugust.com">mordred@inaugust.com</a>> wrote:<br>
>> On 06/17/2015 08:53 AM, Emilien Macchi wrote:<br>
>>>>> Hi Serg,<br>
>>>>><br>
>>>>> On 06/17/2015 05:35 AM, Serg Melikyan wrote:<br>
>>>>>> Hi Emilien,<br>
>>>>>><br>
>>>>>> I would like to answer your question regarding<br>
>>>>>> stackforge/puppet-murano repository asked in different thread:<br>
>>>>>><br>
>>>>>>> Someone from Fuel team created first the module in Fuel, 6<br>
>>>>>>> months ago [1] and 3 months later someone from Fuel team<br>
>>>>>>> created an empty repository in Stackforge [2]. By the way,<br>
>>>>>>> Puppet OpenStack community does not have core permissions on<br>
>>>>>>> this module and it's own by Murano team.<br>
>>>>>><br>
>>>>>> Murano was included to Fuel around 2 years ago, our first<br>
>>>>>> official release as part of Fuel was Icehouse - yes, we have<br>
>>>>>> puppet module for Murano for a long time now. But until recently<br>
>>>>>> we didn't had a Big Tent in place and that is why we never<br>
>>>>>> thought that we able to upstream our module.<br>
>>>>>><br>
>>>>>> Once policy regarding upstream puppet modules in Fuel changed and<br>
>>>>>> Big Tent model was adopted we decided to upstream module for<br>
>>>>>> Murano. I am really sorry that I didn't contact you for more<br>
>>>>>> information how to do that properly and just created<br>
>>>>>> corresponding repository.<br>
>>>>><br>
>>>>> Well, in fact, I'm sorry for you; you could not benefit of Puppet<br>
>>>>> OpenStack community. Let's fix that.<br>
>>>>><br>
>>>>>> I didn't give permission to Puppet OpenStack community for this<br>
>>>>>> repository because it would be strange, given I didn't even<br>
>>>>>> contact you. We thought that we would upstream what we have now<br>
>>>>>> and then make sure that this repo will be integrated with Puppet<br>
>>>>>> OpenStack ecosystem.<br>
>>>>>><br>
>>>>>> We still have big desire to upstream our puppet module. Fuel is<br>
>>>>>> not only user of this module, there are other projects who would<br>
>>>>>> like to use Murano as part of they solution and use puppet module<br>
>>>>>> from Fuel for deployment.<br>
>>>>>><br>
>>>>>> Can you advise how we should proceed further?<br>
>>>>><br>
>>>>> The more recent patch to add a module in OpenStack is zaqar:<br>
>>>>> <a href="https://review.openstack.org/#/c/191942/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/191942/</a><br>
>>>>><br>
>>>>> Two things we need to solve is the fact if you move your module to<br>
>>>>> the big tent: * bring the module compliant (I'm working on a<br>
>>>>> blueprint to explain what is that, but you can already read what we<br>
>>>>> said at the Summit:<br>
>>>>> <a href="https://etherpad.openstack.org/p/liberty-summit-design-puppet-compliant" rel="noreferrer" target="_blank">https://etherpad.openstack.org/p/liberty-summit-design-puppet-compliant</a><br>
>>>>><br>
>>>>><br>
>> * figure out Gerrit permissions. If the module is official, it has to be<br>
>>>>> maintained by Puppet OpenStack group, but that would mean you won't<br>
>>>>> be core on it, which is weird to me at this stage. We are facing<br>
>>>>> the same situation with puppet-monasca, (except the module lives in<br>
>>>>> Stackforge for day1).<br>
>>>>><br>
>><br>
>> When we suck sub-projects into infra, we often make the person or team<br>
>> who was maintainer on it before it came into infra core on just that -<br>
>> with the infra-core team on it as well. For instance:<br>
>><br>
>> puppet-murano-core:<br>
>>   serg melikyan<br>
>>   puppet-core<br>
>><br>
>> Or something. Not saying it's the right choice- just that it's the<br>
>> pattern we've followed before over in our neck of the woods.<br>
>><br>
>>>>>><br>
>>>>>> References: [1]<br>
>>>>>> <a href="https://github.com/stackforge/fuel-library/tree/master/deployment/puppet/murano/" rel="noreferrer" target="_blank">https://github.com/stackforge/fuel-library/tree/master/deployment/puppet/murano/</a><br>
>>>>>><br>
>>>>>><br>
>> [2] <a href="https://review.openstack.org/155688" rel="noreferrer" target="_blank">https://review.openstack.org/155688</a><br>
>>>>>><br>
>>>>><br>
>>>>> Thanks for taking care of that!<br>
>>>>><br>
>>>>><br>
>>>>><br>
>>>>> __________________________________________________________________________<br>
>>>>><br>
>>>>><br>
>> OpenStack Development Mailing List (not for usage questions)<br>
>>>>> Unsubscribe:<br>
>>>>> <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
>>>>> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
>>>>><br>
>><br>
>>><br>
>>> __________________________________________________________________________<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.openstack.org?subject:unsubscribe</a><br>
>>> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
>><br>
>><br>
>><br>
><br>
> --<br>
> Emilien Macchi<br>
><br>
><br>
> __________________________________________________________________________<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.openstack.org?subject:unsubscribe</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
><br>
<br>
<br>
<br>
--<br>
</div></div><div class="HOEnZb"><div class="h5">Serg Melikyan, Senior Software Engineer at Mirantis, Inc.<br>
<a href="http://mirantis.com" rel="noreferrer" target="_blank">http://mirantis.com</a> | <a href="mailto:smelikyan@mirantis.com">smelikyan@mirantis.com</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div style="color:rgb(136,136,136)"><span style="font-family:arial;font-size:small">Best Regards,</span><br></div><span style="color:rgb(136,136,136)">Egorenko Denis</span>,</div><div><span style="color:rgb(136,136,136)">Deployment Engineer</span><br style="color:rgb(136,136,136)"><span style="color:rgb(136,136,136)">Mirantis</span><br></div></div></div></div>
</div>