<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jun 7, 2015 at 11:48 PM, Yanis Guenane <span dir="ltr"><<a href="mailto:yguenane@redhat.com" target="_blank">yguenane@redhat.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=""><br>
<br>
On 06/03/2015 02:32 PM, Martin Mágr wrote:<br>
><br>
</span><div><div class="h5">> On 06/02/2015 07:05 PM, Mathieu Gagné wrote:<br>
>> On 2015-06-02 12:41 PM, Yanis Guenane wrote:<br>
>>> The openstacklib::db::sync[2] is currently only a wrapper around an<br>
>>> exec<br>
>>> that does the actual db sync, this allow to make any modification to<br>
>>> the<br>
>>> exec into a single place. The main advantage IMO is that a contributor<br>
>>> is provided with the same experience as it is not the case today across<br>
>>> all modules.<br>
>>><br>
>> The amount of possible change to an exec resource is very limited. [1] I<br>
>> don't see a value in this change which outweighs the code churn and<br>
>> review load needed to put it in place. Unless we have real use cases or<br>
>> outrageously genius feature to add to it, I'm not in favor of this<br>
>> change.<br>
>><br>
>> Furthermore, any change to the public interface of<br>
>> openstacklib::db::sync would require changes across all our modules<br>
>> anyway to benefit from this latest hypothetical feature. I think we are<br>
>> starting to nitpick over as little "generic" code we could possibly find<br>
>> to put in openstacklib.<br>
>><br>
>> [1] <a href="https://docs.puppetlabs.com/references/latest/type.html#exec" target="_blank">https://docs.puppetlabs.com/references/latest/type.html#exec</a><br>
>><br>
><br>
> Wearing my consistency hat I must say I like this change. On the other<br>
> hand I agree with Mathieu that delegating single resource from several<br>
> modules to single module is necessary in this case.<br>
><br>
> Regards,<br>
> Martin<br>
<br>
</div></div>Mathieu, Martin, thank you for replying.<br>
<br>
For the wrapper around exec usefulness I understand your concerns.<br>
On a note I was trying to follow the current use of openstacklib.<br>
<br>
If we look at openstacklib::db::postgresql[1] or<br>
openstackib::db::mysql[2] they are simple wrapper around<br>
puppet resources with no extra logic, but a common resource across all<br>
modules.<br></blockquote><div>The openstacklib::db::mysql resource is a wrapper around a couple of resources and contains logic around choosing an allowed hosts lists.</div><div><br></div><div>The openstacklib::db::postgresql is largely useless but makes the database interface consistent with mysql.</div><div><br></div><div>Colleen</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Also, to move forward on this topic I will submit to a vote one of the<br>
three propositions during our next meeting<br>
<br>
 1. Abandon this change<br>
 2. Move everything to X::db::sync, but just run the exec there, no<br>
openstacklib::db::sync<br>
 3. Move forward with current implementation<br>
<br>
Thanks again for the feedbacks,<br>
<br>
[1]<br>
<a href="https://github.com/stackforge/puppet-openstacklib/blob/master/manifests/db/postgresql.pp" target="_blank">https://github.com/stackforge/puppet-openstacklib/blob/master/manifests/db/postgresql.pp</a><br>
[2]<br>
<a href="https://github.com/stackforge/puppet-openstacklib/blob/master/manifests/db/mysql.pp" target="_blank">https://github.com/stackforge/puppet-openstacklib/blob/master/manifests/db/mysql.pp</a><br>
<br>
--<br>
Yanis Guenane<br>
<div class="HOEnZb"><div class="h5"><br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div></div>