<div dir="ltr"><div><div>Hi Sam,<br><br></div>that's a great work and it will be for sure my default driver for my development environment.<br><br></div>I have a question: once the review will be approved and the code merged into master, do you plan to create a driver nova subteam as Xen, HyperV and others do? I would be glad to cooperate on it.<br>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On 29 August 2013 07:54, Sam Alba <span dir="ltr"><<a href="mailto:sam.alba@gmail.com" target="_blank">sam.alba@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On Wed, Aug 28, 2013 at 9:12 AM, Sam Alba <<a href="mailto:sam.alba@gmail.com">sam.alba@gmail.com</a>> wrote:<br>
> Thanks a lot everyone for the nice feedback. I am going to work hard<br>
> to get all those new comments addressed to be able to re-submit a new<br>
> patchset today or tomorrow (the later).<br>
><br>
> On Wed, Aug 28, 2013 at 7:02 AM, Russell Bryant <<a href="mailto:rbryant@redhat.com">rbryant@redhat.com</a>> wrote:<br>
>> On 08/28/2013 05:18 AM, Daniel P. Berrange wrote:<br>
>>> On Wed, Aug 28, 2013 at 06:00:50PM +1000, Michael Still wrote:<br>
>>>> On Wed, Aug 28, 2013 at 4:18 AM, Sam Alba <<a href="mailto:sam.alba@gmail.com">sam.alba@gmail.com</a>> wrote:<br>
>>>>> Hi all,<br>
>>>>><br>
>>>>> We've been working hard during the last couple of weeks with some<br>
>>>>> people. Brian Waldon helped a lot designing the Glance integration and<br>
>>>>> driver testing. Dean Troyer helped a lot on bringing Docker support in<br>
>>>>> Devstack[1]. On top of that, we got several feedback on the Nova code<br>
>>>>> review which definitely helped to improve the code.<br>
>>>>><br>
>>>>> The blueprint[2] explains what Docker brings to Nova and how to use it.<br>
>>>><br>
>>>> I have to say that this blueprint is a fantastic example of how we<br>
>>>> should be writing design documents. It addressed almost all of my<br>
>>>> questions about the integration.<br>
>>><br>
>>> Yes, Sam (& any of the other Docker guys involved) have been great at<br>
>>> responding to reviewers' requests to expand their design document. The<br>
>>> latest update has really helped in understanding how this driver works<br>
>>> in the context of openstack from an architectural and functional POV.<br>
>><br>
>> They've been great in responding to my requests, as well.  The biggest<br>
>> thing was that I wanted to see devstack support so that it's easily<br>
>> testable, both by developers and by CI.  They delivered.<br>
>><br>
>> So, in general, I'm good with this going in.  It's just a matter of<br>
>> getting the code review completed in the next week before feature<br>
>> freeze.  I'm going to try to help with it this week.<br>
>><br>
<br>
</div></div>If someone wants to take another look at<br>
<a href="https://review.openstack.org/#/c/32960/" target="_blank">https://review.openstack.org/#/c/32960/</a>, we answered/fixed all<br>
previous comments.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
--<br>
@sam_alba<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>
</div></div></blockquote></div><br></div>