<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Cyril Hrubis <<a href="mailto:chrubis@suse.cz">chrubis@suse.cz</a>> wrote:<br></div></div><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> Notes:<br>
>     Ps. I also think we might need .min_mem_[avai|total] field but<br>
>         not sure if it is really necessary to add that.<br>
<br>
Actually this would be very useful from the long term perspective. If we<br>
ever manage to run tests in parallel the scheduller should make sure<br>
that we do not hit OOM due to running more than one processe that<br>
consume significant amount of memory at the same time.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Well, sounds useful to avoid OOM in the parallel run.</div><div class="gmail_default" style="font-size:small">Hence the .min_mem should gives the requirement of minimal</div><div class="gmail_default" style="font-size:small">MemAvailable(but not MemTotal) size on test system.</div><div class="gmail_default" style="font-size:small"> </div><div class="gmail_default" style="font-size:small">I will view the whole test to add that when I get a chance.</div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Looks good:<br>
<br>
Reviewed-by: Cyril Hrubis <<a href="mailto:chrubis@suse.cz" target="_blank">chrubis@suse.cz</a>><br></blockquote></div><div class="gmail_default" style="font-size:small"></div><div><div class="gmail_default" style="font-size:small">Pushed (with the typo fix).</div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>