<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Sep 5, 2017 at 6:47 AM Mohammed Naser <<a href="mailto:mnaser@vexxhost.com">mnaser@vexxhost.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Sep 5, 2017 at 4:15 AM, Emil Enemærke <<a href="mailto:enemarke@gmail.com" target="_blank">enemarke@gmail.com</a>> wrote:<br>
> Hi,<br>
><br>
> I have stated using the puppet-ceph module for deploying ceph, and have<br>
> noticed a heavy use of exec in fx define 'ceph::osd'<br>
> (<a href="https://github.com/openstack/puppet-ceph/blob/master/manifests/osd.pp" rel="noreferrer" target="_blank">https://github.com/openstack/puppet-ceph/blob/master/manifests/osd.pp</a>). Is<br>
> there a reason for not writing this define as an ensurable type/provider?<br>
> Otherwise I will fork the module an start on rewriting it for a<br>
> type/provider.<br>
><br>
<br>
Thanks for helping out.  I'm happy to see folks using the puppet-ceph<br>
modules!  I think the reason why we've relied of the Exec's is purely<br>
historic.  If you have a patch that would convert it to an ensurable<br>
type and provider, we'd be more than happy to merge it!<br></blockquote><div><br></div><div>As manser pointed out, the reason for the exec's is purely historic, in that the initial implementation team  wasn't comfortable with developing ruby providers given our familiarity at the time. It was easier for us to develop and troubleshoot the execs directly.</div><div><br></div><div>We'd be more than happy to have reviews to migrate to a ruby implementation</div><div><br></div><div>If you have any questions, feel free to pop by on #puppet-openstack on freenode</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
><br>
> Cheers<br>
> Emil<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>
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>
</blockquote></div></div><div dir="ltr">-- <br></div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Andrew Woodward</div></div>