<div dir="ltr">Hi, @jichen:<div><br></div><div>Something I found, FYI:<div>The commit history in here: <a href="https://review.openstack.org/#/c/138795">https://review.openstack.org/#/c/138795</a></div><div><br></div><div>I noticed that the unquience change call to cast in PS5 and PS6:</div><div><a href="https://review.openstack.org/#/c/138795/5..6/nova/compute/rpcapi.py">https://review.openstack.org/#/c/138795/5..6/nova/compute/rpcapi.py</a><br></div><div>and some comments on <a href="https://review.openstack.org/#/c/138795/5/nova/compute/api.py@2235">https://review.openstack.org/#/c/138795/5/nova/compute/api.py@2235</a></div><div><br></div><div>According history comments, the reason of "use cast for unquience": it adds _wait_for_snapshots_completion operation on unquience method in PS6,</div><div>it will cause the rpc timeout before snapshot finished if we use call rpc.</div><div><br></div><div>The reason of "use call for quience", I think is just quience is a short operation, no need to change cast to call?</div><div>In the other word, call or cast is okay for quience operation, so, he didn't change it. </div><div><br></div><div>Hope this helps, : ) </div><div><br></div><div><div class="gmail_extra"><div><div class="gmail_signature"><div>Regards,</div><div>Yikun</div><div><font color="#666666">----------------------------------------</font></div><div><font color="#666666">Jiang Yikun（</font><span style="color:rgb(102,102,102)">Kero）</span></div><div><font color="#666666">Mail: <a href="mailto:yikunkero@gmail.com" target="_blank">yikunkero@gmail.com</a></font></div><div><font color="#666666">Tel: (+86) 13572822142</font></div></div></div>
<br><div class="gmail_quote">2017-12-21 16:48 GMT+08:00 Chen CH Ji <span dir="ltr"><<a href="mailto:jichenjc@cn.ibm.com" target="_blank">jichenjc@cn.ibm.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><p><font size="2">During review </font><a href="https://review.openstack.org/#/c/529278/2" target="_blank"><font size="2">https://review.openstack.org/#<wbr>/c/529278/2</font></a><font size="2"> ,some question on the method for </font><font size="2">quience/unquience</font><br><br><a href="https://github.com/openstack/nova/blob/master/nova/compute/rpcapi.py#L1140" target="_blank"><font size="2">https://github.com/openstack/<wbr>nova/blob/master/nova/compute/<wbr>rpcapi.py#L1140</font></a><font size="2"> use call for quience</font><br><a href="https://github.com/openstack/nova/blob/master/nova/compute/rpcapi.py#L1146" target="_blank"><font size="2">https://github.com/openstack/<wbr>nova/blob/master/nova/compute/<wbr>rpcapi.py#L1146</font></a><font size="2"> use cast for unquience</font><br><br><font size="2">just curious ,any special purpose for use different type here?</font><br><br><font size="2">Best Regards! <br><br>Kevin (Chen) Ji 纪 晨<br><br>Engineer, zVM Development, CSTL<br>Notes: Chen CH Ji/China/IBM@IBMCN   Internet: <a href="mailto:jichenjc@cn.ibm.com" target="_blank">jichenjc@cn.ibm.com</a><br>Phone: <a href="tel:+86%2010%208245%201493" value="+861082451493" target="_blank">+86-10-82451493</a><br>Address: 3/F Ring Building, ZhongGuanCun Software Park, Haidian District, Beijing 100193, PRC </font><br>
</p></div>
<br>______________________________<wbr>______________________________<wbr>______________<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.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
<br></blockquote></div><br></div></div></div></div>