<div dir="ltr">Thanks John,<div>I updated to unknown.</div><div><br></div><div>Eduard</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 18, 2014 at 8:09 PM, John Griffith <span dir="ltr"><<a href="mailto:john.griffith8@gmail.com" target="_blank">john.griffith8@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Thu, Dec 18, 2014 at 1:56 AM, Eduard Matei<br>
<<a href="mailto:eduard.matei@cloudfounders.com">eduard.matei@cloudfounders.com</a>> wrote:<br>
> Hi everyone,<br>
><br>
> We're in a bit of a predicament regarding review:<br>
> <a href="https://review.openstack.org/#/c/130733/" target="_blank">https://review.openstack.org/#/c/130733/</a><br>
><br>
> Two days ago it got a -1 from John G asking to change infinite to<br>
> unavailable although the docs clearly say that "If the driver is unable to<br>
> provide a value for free_capacity_gb or total_capacity_gb, keywords can be<br>
> provided instead. Please use ‘unknown’ if the array cannot report the value<br>
> or ‘infinite’ if the array has no upper limit."<br>
> (<a href="http://docs.openstack.org/developer/cinder/devref/drivers.html" target="_blank">http://docs.openstack.org/developer/cinder/devref/drivers.html</a>)<br>
><br>
> After i changed it, came Walter A. Boring IV and gave another -1 saying we<br>
> should return infinite.<br>
><br>
> Since we use S3 as a backend and it has no upper limit (technically there is<br>
> a limit but for the purposes of our driver there's no limit as the backend<br>
> is "elastic") we could return infinite.<br>
><br>
> Anyway, the problem is that now we missed the K-1 merge window although the<br>
> driver passed all tests (including cert tests).<br>
><br>
> So please can someone decide which is the correct value so we can use that<br>
> and get the patched approved (unless there are other issues).<br>
><br>
> Thanks,<br>
> Eduard<br>
> --<br>
><br>
> Eduard Biceri Matei, Senior Software Developer<br>
> <a href="http://www.cloudfounders.com" target="_blank">www.cloudfounders.com</a><br>
>  | <a href="mailto:eduard.matei@cloudfounders.com">eduard.matei@cloudfounders.com</a><br>
><br>
><br>
><br>
> CloudFounders, The Private Cloud Software Company<br>
><br>
> Disclaimer:<br>
> This email and any files transmitted with it are confidential and intended<br>
> solely for the use of the individual or entity to whom they are addressed.<br>
> If you are not the named addressee or an employee or agent responsible for<br>
> delivering this message to the named addressee, you are hereby notified that<br>
> you are not authorized to read, print, retain, copy or disseminate this<br>
> message or any part of it. If you have received this email in error we<br>
> request you to notify us by reply e-mail and to delete all electronic files<br>
> of the message. If you are not the intended recipient you are notified that<br>
> disclosing, copying, distributing or taking any action in reliance on the<br>
> contents of this information is strictly prohibited.<br>
> E-mail transmission cannot be guaranteed to be secure or error free as<br>
> information could be intercepted, corrupted, lost, destroyed, arrive late or<br>
> incomplete, or contain viruses. The sender therefore does not accept<br>
> liability for any errors or omissions in the content of this message, and<br>
> shall have no liability for any loss or damage suffered by the user, which<br>
> arise as a result of e-mail transmission.<br>
><br>
><br>
</div></div>> _______________________________________________<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>
Hi Eduard,<br>
<br>
First, I owe you an apology; I stated "unavailable" in the review, but<br>
should've stated "unknown".  We're in the process of trying to<br>
eliminate the reporting of infinite as it screwed up the weighing<br>
scheduler.<br>
<br>
Note that Zhiteng adjusted the scheduler so this isn't such a big deal<br>
anymore by down-grading the handling of infinite and unknown [1].<br>
<br>
Anyway, my suggestion to not use infinite is because in the coming<br>
weeks I'd like to remove infinite from the stats reporting altogether,<br>
and for those backends that for whatever reason don't know how much<br>
capacity they have use a more accurate report of "unknown".<br>
<br>
Sorry for the confusion, I think the comments on your review have been<br>
updated to reflect this, if not I'll do that next.<br>
<br>
Thanks,<br>
John<br>
<br>
[1]: <a href="https://github.com/openstack/cinder/commit/ee9d30a73a74a2e1905eacc561c1b5188b62ca75" target="_blank">https://github.com/openstack/cinder/commit/ee9d30a73a74a2e1905eacc561c1b5188b62ca75</a><br>
<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>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><pre style="white-space:pre-wrap;font-size:10px;word-wrap:break-word;margin-top:0px;margin-bottom:0px"><div style="font-family:arial,sans-serif;font-size:13px;text-align:-webkit-auto;white-space:normal;margin:0px"><b><span lang="EN-US" style="font-size:10pt;font-family:Arial,sans-serif">Eduard Biceri Matei, Senior Software Developer</span></b></div><div style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px;text-align:-webkit-auto;white-space:normal;margin:0px"><span style="font-size:13.5pt;font-family:Helvetica,sans-serif"><a href="http://www.cloudfounders.com/" style="color:rgb(17,85,204)" target="_blank"><span lang="EN-US" style="font-size:11pt;font-family:Calibri,sans-serif"><font color="#0045a3">www.cloudfounders.com</font></span></a><div style="display:inline-block;width:16px;height:16px"></div><div style="display:inline-block;width:16px;min-height:16px"></div></span><span lang="EN-US" style="font-size:11pt;font-family:Calibri,sans-serif"><font color="#1f497d"> | </font><a href="mailto:eduard.matei@cloudfounders.com" style="color:rgb(17,85,204)" target="_blank"><font color="#0045a3">eduard.matei@cloudfounders.com</font></a></span></div><div style="font-family:arial,sans-serif;font-size:13px;text-align:-webkit-auto;white-space:normal;margin:0px"><span lang="EN-US" style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)"><u></u> <u></u></span></div><br><div style="font-family:arial,sans-serif;font-size:13px;text-align:-webkit-auto;white-space:normal;margin:0px"><span lang="EN-US" style="font-size:13.5pt;font-family:Arial,sans-serif;color:rgb(80,0,80)"><u></u> <u></u></span></div><div style="font-family:arial,sans-serif;font-size:13px;text-align:-webkit-auto;white-space:normal;margin:0px"><b><span lang="EN-US" style="font-size:13.5pt;font-family:Arial,sans-serif">CloudFounders, The Private Cloud Software Company</span></b><span lang="EN-US" style="font-size:13.5pt;font-family:Arial,sans-serif"><u></u><u></u></span></div><div style="font-family:arial,sans-serif;font-size:13px;text-align:-webkit-auto;white-space:normal;margin:0px"><span lang="EN-US" style="font-size:13.5pt;font-family:Helvetica,sans-serif"><u></u> <u></u></span></div><div style="font-family:arial,sans-serif;font-size:13px;text-align:-webkit-auto;white-space:normal;margin:0px"><span lang="EN-US" style="font-family:Helvetica,sans-serif;font-size:11px">Disclaimer:<u></u><u></u></span></div><div style="font-family:arial,sans-serif;font-size:13px;text-align:-webkit-auto;white-space:normal;margin:0px"><span lang="EN-US" style="font-size:8.5pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)">This email and any files transmitted with it are confidential and intended solely for the use of the individual or entity to whom they are addressed.</span><span lang="EN-US" style="font-family:Calibri,sans-serif;color:rgb(31,73,125)"><br><span style="font-size:11px">If you are not the named addressee or an employee or agent responsible for delivering this message to the named addressee, you are hereby notified that you are not authorized to read, print, retain, copy or disseminate this message or any part of it. If you have received this email in error we request you to notify us by reply e-mail and to delete all electronic files of the message. If you are not the intended recipient you are notified that disclosing, copying, distributing or taking any action in reliance on the contents of this information is strictly prohibited. <br>E-mail transmission cannot be guaranteed to be secure or error free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or contain viruses. The sender therefore does not accept liability for any errors or omissions in the content of this message, and shall have no liability for any loss or damage suffered by the user, which arise as a result of e-mail transmission.</span></span></div></pre></div></div>
</div>