Thanks ! Looks good to me !

• minor nit (not a holdup) I see you have differentiated density from launchPods.... seems however that there ir "raw density" and "relative density". - Later in a follow on we can rename these vars or add more comments.

By the way... what was the problem with It arguments ? I didnt notice one.... (update) looking closer, i guess youve removed need to access via indices?

@jayunit100
I've differentiated them, because those test are pretty different one from the other.
So what names are you suggesting? [I was never good at naming thing so I'm very open on any suggestions]

Regarding the "It" argument - this wasn't your bug - it was introduced by me before (but I didn't observe it). Basically it seems that "It" is not called synchronously so there is a problem of using variables from the outside that can change in the meantime inside "It".

@wojtek-t
cool :) ... for names? I'd suggest podScalability and podDensity . but that can be done later.... no need to hold up this patch.

the names i suggest above give a good difference between the latter (which launches # pods per minion) and the former (which just launches N pods).

I say lets merge it if you've tested ... its a good update !

Thanks - renaming done.

Thanks @wojtek-t This looks great.

LGTM