<div dir="ltr">Sorry folks for the late reply, I'll try to also weigh in the Gerrit change.<br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 24, 2018 at 2:55 PM, Jay Pipes <span dir="ltr"><<a href="mailto:jaypipes@gmail.com" target="_blank">jaypipes@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 04/23/2018 05:51 PM, Arvind N wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Thanks for the detailed options Matt/eric/jay.<br>
<br>
Just few of my thoughts,<br>
<br>
For #1, we can make the explanation very clear that we rejected the request because the original traits specified in the original image and the new traits specified in the new image do not match and hence rebuild is not supported.<br>
</blockquote>
<br></span>
I believe I had suggested that on the spec amendment patch. Matt had concerns about an error message being a poor user experience (I don't necessarily disagree with that) and I had suggested a clearer error message to try and make that user experience slightly less sucky.<span class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
For #3,<br>
<br>
Even though it handles the nested provider, there is a potential issue.<br>
<br>
Lets say a host with two SRIOV nic. One is normal SRIOV nic(VF1), another one with some kind of offload feature(VF2).(Described by alex)<br>
<br>
Initial instance launch happens with VF:1 allocated, rebuild launches with modified request with traits=HW_NIC_OFFLOAD_X, so basically we want the instance to be allocated VF2.<br>
<br>
But the original allocation happens against VF1 and since in rebuild the original allocations are not changed, we have wrong allocations.<br>
</blockquote>
<br></span>
Yep, that is certainly an issue. The only solution to this that I can see would be to have the conductor ask the compute node to do the pre-flight check. The compute node already has the entire tree of providers, their inventories and traits, along with information about providers that share resources with the compute node. It has this information in the ProviderTree object in the reportclient that is contained in the compute node resource tracker.<br>
<br>
The pre-flight check, if run on the compute node, would be able to grab the allocation records for the instance and determine if the required traits for the new image are present on the actual resource providers allocated against for the instance (and not including any child providers not allocated against).<br>
<br></blockquote><div><br></div><div>Yup, that. We also have pre-flight checks for move operations like live and cold migrations, and I'd really like to keep all the conditionals in the conductor, because it knows better than the scheduler which operation is asked.<br></div><div>I'm not really happy with adding more in the scheduler about "yeah, it's a rebuild, so please do something exceptional", and I'm also not happy with having a filter (that can be disabled) calling the Placement API.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Or... we chalk this up as a "too bad" situation and just either go with option #1 or simply don't care about it.</blockquote><div><br></div><div>Also, that too. Maybe just provide an error should be enough, nope? Operators, what do you think ? (cross-calling openstack-operators@)<br><br></div><div> -Sylvain<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
<br>
Best,<br>
-jay<br>
<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.op<wbr>enstack.org?subject:unsubscrib<wbr>e</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi<wbr>-bin/mailman/listinfo/openstac<wbr>k-dev</a><br>
</div></div></blockquote></div><br></div></div>