Please, don't review it right now. It seems that it breaks some existing 
tests.

On Thursday, October 8, 2015 at 11:42:22 AM UTC+2, Oleg Ponomarev wrote:
>
> Since location tags were introduced, they may affect optimal allocation. 
> Thus, node tags should be initialized in iallocator. 
>
> From my point of view, node tags don't affect instance placement in 
> previous 
> ganeti releases. 
>
> Oleg Ponomarev (2): 
>   Set node tags in iallocator htools backend 
>   Add test for tags accounting in hail 
>
>  Makefile.am                                |   1 + 
>  src/Ganeti/HTools/Backend/IAlloc.hs        |   4 +- 
>  test/data/htools/hail-alloc-nlocation.json | 159 
> +++++++++++++++++++++++++++++ 
>  test/hs/shelltests/htools-hail.test        |   5 + 
>  4 files changed, 168 insertions(+), 1 deletion(-) 
>  create mode 100644 test/data/htools/hail-alloc-nlocation.json 
>
> -- 
> 2.6.0.rc2.230.g3dd15c0 
>
>

Reply via email to