
<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On 03 Sep 2016, at 20:07, Duncan Thomas <<a href="mailto:duncan.thomas@gmail.com" class="">duncan.thomas@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">There's also another API limitation to be fixed - whether it goes in the initial API fixup or gets done later, which is a round one cinder serving multiple nova or other consumers: <a href="https://review.openstack.org/#/c/362637/" class="">https://review.openstack.org/#/c/362637/</a></div></div></blockquote><div><br class=""></div><div>Thanks Duncan, I added it to our etherpad to keep an eye on this one too.</div><br class=""><blockquote type="cite" class=""><div class=""><div class="gmail_extra"><br class=""><div class="gmail_quote">On 2 September 2016 at 22:51, Matt Riedemann <span dir="ltr" class=""><<a href="mailto:mriedem@linux.vnet.ibm.com" target="_blank" class="">mriedem@linux.vnet.ibm.com</a>></span> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 9/1/2016 4:09 PM, Ildiko Vancsa wrote:<br class="">

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hi All,<br class="">

<br class=""><br class="">

We had an attempt to remove ‘check_attach’ from the Nova code, as this check should be purely Cinder’s responsibility. The call is partially removed [4] and there’s one patch up for review to finish that work item [5]. The difficulty with this one is that we have one flow that missed the ‘reserve_volume’ call and therefore also the required checks on Cinder side. This is corrected in the patch up for review [5], it needs some more eyes on it to ensure we have the proper fix.<br class="">

</blockquote>

<br class=""></span>

I haven't gone through the new experimental APIs proposed in cinder but while Walter was working on [4] we realized a gap in the os-reserve call in that we should pass the availability zone since nova still has to check that separately:<br class="">

<br class="">

<a href="https://github.com/openstack/nova/blob/96926a8ee182550391c30b16b410da7f598c0f39/nova/volume/cinder.py#L290" rel="noreferrer" target="_blank" class="">https://github.com/openstack/n<wbr class="">ova/blob/96926a8ee182550391c30<wbr class="">b16b410da7f598c0f39/nova/<wbr class="">volume/cinder.py#L290</a><br class="">

<br class="">

That seems like something we could do separately early in ocata with a microversion on the os-reserve API to take an optional availability_zone argument and fail if it doesn't match the AZ for the volume being reserved.<span class="im HOEnZb"><br class=""></span></blockquote></div></div></div></blockquote><div><br class=""></div><div>@Matt: Do you mean to remove check_attach along with check_availability_zone in one step, when we have os-reserve properly fixed?</div><div><br class=""></div><div>Thanks,</div><div>Ildikó</div><div><br class=""></div><br class=""><blockquote type="cite" class=""><div class=""><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="im HOEnZb">

<br class="">

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br class="">[5] <a href="https://review.openstack.org/#/c/335358/" rel="noreferrer" target="_blank" class="">https://review.openstack.org/#<wbr class="">/c/335358/</a><br class=""></blockquote></span><span class="HOEnZb"><font color="#888888" class="">

-- <br class="">

<br class=""></font></span></blockquote></div></div></div></blockquote></div><br class=""></div></body></html>