<div dir="ltr"><div><div><div><div><div><div><div><div>Hi Tian,<br><br></div>A couple of things that came out of the V3 API discussions which are relevant to your patches:<br><br></div>- In Icehouse we want to merge the V3 API version of patches either before the V2 one or in the same patch. With<br>
</div>  yours split its probably easiest to do that by setting the V2 patches dependent on the V3 ones.<br><br></div>- I've added this previously in the review comments for some of the patches that we also would<br></div>
  like a specification written up for the REST API which you are adding (not just what data is returned,<br></div>  but the methods (GET/POST/PUT/DELETE), URLs, data format in and data format out).<br><br>  At this point in time in the blueprint is probably best approach with a summary in the commit message<br>
  for the portion which is added (since the API changes are split up - which is good)<br><br></div></div><div>Regards,<br><br>Chris<br></div><div><br><br> </div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Mon, Nov 11, 2013 at 7:23 PM, Tian, Shuangtai <span dir="ltr"><<a href="mailto:shuangtai.tian@intel.com" target="_blank">shuangtai.tian@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">






<div link="blue" vlink="purple" lang="ZH-CN">
<div>
<p class="MsoNormal"><span style="color:#1f497d" lang="EN-US">Hi,All<u></u><u></u></span></p>
<p class="MsoNormal"><span style="color:#1f497d" lang="EN-US"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="color:#1f497d" lang="EN-US">PCI pass-through support has been added to nova. Now we are doing the apis to support PCI pass-through.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="color:#1f497d" lang="EN-US">Pls review the code :</span><span lang="EN-US"><a href="https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:pci-api-support,n,z" target="_blank">https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:pci-api-support,n,z</a><u></u><u></u></span></p>

<p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="color:#1f497d" lang="EN-US">Thank you very much.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="color:#1f497d" lang="EN-US"><u></u> <u></u></span></p>
<p class="MsoNormal" style="text-align:left;line-height:12.0pt" align="left"><span style="font-size:11.0pt" lang="EN-US">Best regards,<u></u><u></u></span></p>
<p class="MsoNormal" style="text-align:left;line-height:12.0pt" align="left"><span style="font-size:11.0pt" lang="EN-US">Tian, Shuangtai
</span><span style="font-size:11.0pt" lang="EN-US"><u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>
</div>
</div>

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