<div dir="ltr"><div>Thanks for the recap, got one question for the "block creation": </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> 

<span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:12.8px;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">* An attempt to create an instance should be blocked if the project has instances in a "down" cell (the instance_mappings table has a "project_id" column) because we cannot count instances in "down" cells for the quota check.</span></blockquote><div><br></div><div>Since users are not aware of any cell information, and the cells are mostly randomly selected, there could be high possibility that users(projects) instances are equally spreaded across cells. The proposed behavior seems can</div><div>easily cause a lot of users couldn't create instances because one of the cells is down, isn't it too rude?</div><div><br></div><div>BR,</div><div><br></div><div>Kevin Zheng</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 15, 2018 at 2:26 AM, Chris Dent <span dir="ltr"><<a href="mailto:cdent+os@anticdent.org" target="_blank">cdent+os@anticdent.org</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 Wed, 14 Mar 2018, melanie witt wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I’ve created a summary etherpad [0] for the nova cells session from the PTG and included a plain text export of it on this email.<br>
</blockquote>
<br></span>
Nice summary. Apparently I wasn't there or paying attention when<br>
something was decided:<span class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
 * An attempt to delete an instance in a "down" cell should result in a 500 or 503 error.<br>
</blockquote>
<br></span>
Depending on how we look at it, this doesn't really align with what<br>
500 or 503 are supposed to be used. They are supposed to indicate<br>
that the web server is broken in some fashion: 500 being an<br>
unexpected and uncaught exception in the web server, 503 that the<br>
web server is either overloaded or down for maintenance.<br>
<br>
So, you could argue that 409 is the right thing here (as seems to<br>
always happen when we discuss these things). You send a DELETE to<br>
kill the instance, but the current state of the instance is "on a<br>
cell that can't be reached" which is in "conflict" with the state<br>
required to do a DELETE.<br>
<br>
If a 5xx is really necessary, for whatever reason, then 503 is a<br>
better choice than 500 because it at least signals that the broken<br>
thing is sort of "over there somewhere" rather than the web server<br>
having an error (which is what 500 is supposed to mean).<span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
Chris Dent                       ٩◔̯◔۶           <a href="https://anticdent.org/" rel="noreferrer" target="_blank">https://anticdent.org/</a><br>
freenode: cdent                                         tw: @anticdent</font></span><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>