The serie of patchsets is available here: https://review.openstack.org/#/q/status:open+branch:master+topic:db_sync,n,z
Thanks in advance for your feedbacks, -- Yanis Guenane On 06/25/2015 10:02 AM, Yanis Guenane wrote: > > On 06/09/2015 04:07 PM, Mathieu Gagné wrote: >> On 2015-06-08 2:48 AM, Yanis Guenane wrote: >>> If we look at openstacklib::db::postgresql[1] or >>> openstackib::db::mysql[2] they are simple wrapper around >>> puppet resources with no extra logic, but a common resource across all >>> modules. >> I see a lot of resources, relationships and logic in >> openstackib::db::mysql. We can clearly see added value in that one. >> >>> Also, to move forward on this topic I will submit to a vote one of the >>> three propositions during our next meeting >>> >>> 1. Abandon this change >>> 2. Move everything to X::db::sync, but just run the exec there, no >>> openstacklib::db::sync >>> 3. Move forward with current implementation >>> >> I vote for 2 if we can make it happen in all modules and expose the same >> interface (parameters) for consistency. >> > As decided during weekly meeting #37[1] option 2 was chosen. > > Hence a series of fixed patchset will be submitted to reflect this decision. > > Thank you all for your feedbacks, > > [1] > http://eavesdrop.openstack.org/meetings/puppet_openstack/2015/puppet_openstack.2015-06-09-15.00.log.html > > -- > Yanis Guenane > > __________________________________________________________________________ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: [email protected]?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev __________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: [email protected]?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
