<p dir="ltr">I agree with Walt that reverting this is the right answer. </p>
<p dir="ltr">Please make sure to have your ci running as well for liberty. </p>
<p dir="ltr">Jay</p>
<div class="gmail_quote">On Mar 4, 2015 2:30 AM, "Zhangni" <<a href="mailto:zhangni@huawei.com">zhangni@huawei.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div lang="ZH-CN" link="blue" vlink="purple">
<div>
<p class="MsoNormal"><span lang="EN-US" style="font-family:"Arial","sans-serif";color:black;background:white">Hi Mike, Jay, and Walter,
<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-family:"Arial","sans-serif";color:black;background:white"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-family:"Arial","sans-serif";color:black;background:white">Please revisit this patch
<a href="https://review.openstack.org/#/c/151959/" target="_blank">https://review.openstack.org/#/c/151959/</a></span><span lang="EN-US" style="font-family:"Arial","sans-serif";background:white"> and don’t revert this,
<span style="color:black">thank you very much!<u></u><u></u></span></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-family:"Arial","sans-serif";color:black;background:white"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-family:"Arial","sans-serif";color:black;background:white">I think it’s apposite to merge the SDSHypervisor driver in cinder first, and next to request nova to add a new libvirt volume driver.<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-family:"Arial","sans-serif";color:black;background:white"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-family:"Arial","sans-serif";color:black;background:white">Meanwhile nova side always ask whether the driver is merged into Cinder, please see my comments in nova spec
</span><span lang="EN-US"><a href="https://review.openstack.org/#/c/130919/" target="_blank"><span style="font-family:"Arial","sans-serif";color:#0654ac;background:white;text-decoration:none">https://review.openstack.org/#/c/130919/</span></a></span><span lang="EN-US" style="font-family:"Arial","sans-serif";color:black;background:white">,
 thank you very much!</span><span lang="EN-US"><u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-family:"Arial","sans-serif";color:black;background:white"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">Best regards<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">ZhangNi<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>
</div>
</div>

<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>
<br></blockquote></div>