<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Aug 28, 2013 at 4:34 PM, Alex Glikson <span dir="ltr"><<a href="mailto:GLIKSON@il.ibm.com" target="_blank">GLIKSON@il.ibm.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><tt><font>Joe Gordon <<a href="mailto:joe.gordon0@gmail.com" target="_blank">joe.gordon0@gmail.com</a>> wrote on 28/08/2013
11:04:45 PM:<div class="im"><br>
>> Well, first, at the moment each of these filters today duplicate
the<br>
>> code that handles aggregate-based overrides. So, it would make
sense<br>
>> to have it in one place anyway. Second, why duplicating all the
<br>
>> filters if this can be done with a single flag? </div></font></tt>
<br><tt><font>> <br><div class="im">
> We already have too many flags, and i don't want to introduce one
<br>
> that we plan on removing / deprecating in the near future if we can
help it.</div></font></tt>
<br>
<br><tt><font>Wouldn't it make sense to have a flag that enables/disables
aggregate-based policy overrides anyway?</font></tt>
<br></blockquote><div><br></div><div>Why?  </div><div><br></div><div>FWIW that is my default answer to don't we need a flag to do x.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<br><tt><font>> </font></tt><a href="https://github.com/openstack/nova/blob/master/nova/scheduler/" target="_blank"><tt><font>https://github.com/openstack/nova/blob/master/nova/scheduler/</font></tt></a><div class="im">

<tt><font><br>
> filters/ram_filter.py doesn't duplicate all the code, it uses
a base<br>
> class.  The check the aggregate for the value logic is duplicated,
<br>
> but that is easy to fix.</font></tt>
<br>
<br></div><tt><font>Yep, that's exactly what I'm saying -- the first step
would be to put that logic in one place (e.g., scheduler/utils.py, like
the get_config method we have been thinking to introduce originally), and
then we can easily reuse it in all the other filters (regardless of the
decision whether to do it within the existing filters, or to add an "AggregateXYZ"
filter for each existing filter XYZ. Same potentially for weight functions,
etc).</font></tt>
<br></blockquote><div><br></div><div>Sounds like we are in agreement here</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888">
<br><tt><font>Alex</font></tt>
<br>
<br></font></span><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>
<br></blockquote></div><br></div></div>