<div dir="ltr">So I have had a chance to look over the whole review history again. I agree with Sean Dague and Dean Troyer's concerns that the current patch affects code outside of lib/storage and extras.d. We should make the Devstack extension system more flexible to allow for more extensions. Although I am not sure if this responsibility falls completely in the lap of those wishing to integrate Ceph.<div>

<br></div><div>What is more concerning though is the argument that <i>even when the Ceph patch meets these standards</i> <i>it will still have to be pulled in from some external source. </i>Devstack is a central part of OpenStack's test and development system. Core projects depend upon it to develop and test drivers. As an operator, I use it to understand how changes might affect my production system. Documentation. Bug Triage. Outreach. Each of these tasks and efforts benefit from having a curated and maintained set extras in the mainline codebase. Particularly extras that are already represented by mainline drivers in other projects.</div>

<div><br></div><div>I am hopeful that def core() will help in creating a logical framework for these considerations.</div><div><br></div><div>~ Scott</div><div><br></div><div><br></div><div><br></div></div><div class="gmail_extra">

<br><br><div class="gmail_quote">On Fri, Apr 4, 2014 at 4:25 AM, Chmouel Boudjnah <span dir="ltr"><<a href="mailto:chmouel@enovance.com" target="_blank">chmouel@enovance.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hello,<br>
<br>
We had quite a lengthy discussion on this review :<br>
<br>
<a href="https://review.openstack.org/#/c/65113/" target="_blank">https://review.openstack.org/#/c/65113/</a><br>
<br>
about a patch that seb has sent to add ceph support to devstack.<br>
<br>
The main issues seems to resolve around the fact that in devstack we<br>
support only packages that are in the distros and not having to add<br>
external apt sources for that.<br>
<br>
In devstack we are moving as well toward a nice and solid plugin system<br>
where people can hook externally and not needing to submit patch to add<br>
a feature that change the 'core' of devstack.<br>
<br>
I think the best way to go forward with this would be to :<br>
<br>
* Split the patch mentioned above to get the generic things bit in<br>
their own patch. i.e the storage file :<br>
<br>
<a href="https://review.openstack.org/#/c/65113/19/lib/storage" target="_blank">https://review.openstack.org/#/c/65113/19/lib/storage</a><br>
<br>
and the create_disk (which would need to be used by lib/swift as well) :<br>
<br>
<a href="https://review.openstack.org/#/c/65113/19/functions" target="_blank">https://review.openstack.org/#/c/65113/19/functions</a><br>
<br>
* Get the existing drivers converted to that new storage format.<br>
<br>
* Adding new hooks to the plugin system to be able to do what we want<br>
for this:<br>
<br>
<a href="https://review.openstack.org/#/c/65113/19/lib/cinder" target="_blank">https://review.openstack.org/#/c/65113/19/lib/cinder</a><br>
<br>
and for injecting things in libvirt :<br>
<br>
<a href="https://review.openstack.org/#/c/65113/19/lib/nova" target="_blank">https://review.openstack.org/#/c/65113/19/lib/nova</a><br>
<br>
Hopefully to have folks using devstack and ceph would just need to be :<br>
<br>
$ git clone devstack<br>
$ curl -O lib/storages/ceph http://..../ceph_devstack<br>
(and maybe an another file for extras.d)<br>
<br>
am I missing a step ?<br>
<br>
Cheers,<br>
Chmouel.<br>
<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</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>
</blockquote></div><br></div>