On Thu, 2012-08-02 at 14:24 +0200, [email protected] wrote: > This revision includes: > - a new page where user can enable/disable strategies > - fixes to address Jan's and Jozef's findings during their review >
ACK, you can pushed the patchset. As we discussed it's needed to fix ASAP instance variables in patchset according "Coding Guidelines" part Instance Variables. [1] Question at the end: Other story is discussion about probability distribution and throwing out the matches of selected provider accounts. As we weren't able to say exactly if it's ok or not, we'd like to ask community(the best some person who has good knowledge from this field) :) What's happened? We are using random match in two places when we are trying to run deployment. One of that match is thrown out because the calculated match is changed by new one in the final launch. The questions is if we are breaking probability distribution of the provider account's selection ? [1] - https://www.aeolusproject.org/redmine/projects/aeolus/wiki/Coding_Guidelines -- Jozef
