RFR: 8233712: Limit default tests jobs based on ulimit -u setting
Erik Joelsson
erik.joelsson at oracle.com
Fri Nov 15 16:27:27 UTC 2019
On 2019-11-15 04:23, Severin Gehwolf wrote:
> Hi Erik, Magnus,
>
> On Fri, 2019-11-08 at 08:57 -0800, Erik Joelsson wrote:
>> Hello Severin,
>>
>> On 2019-11-08 07:48, Severin Gehwolf wrote:
>>> Right. I believe webrev 03 does this?
>> Yes, that was mostly a reply to Magnus.
>>> Thanks! Tried it, which doesn't seem to work[1]:
>>> http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8233712/04/webrev/
>> Looking at your job, it's failing already in configure on our Linux
>> hosts. At least on Oracle Linux, it seems ulimit is a bash builtin and
>> not found in path. Configure needs to fallback on that.
> Thanks for the info. This updated webrev passes jdk/submit now:
> http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8233712/05/webrev/
>
> Thoughts?
That is an interesting approach. I think it's good to go. Magnus may
want to tweak it, but as he is on vacation, it may take a while to get
feedback from him.
Perhaps we could just put the builtin logic in the default
BASIC_REQUIRE_PROG, but we could also wait and do that later. No need to
stall this fix more I think.
/Erik
> Thanks,
> Severin
>
>>> Is there some way to reproduce locally?
>>>
>>> Thanks,
>>> Severin
>>>
>>> [1] [Mach5] mach5-one-sgehwolf-JDK-8233712-4-20191108-1454-6533841:
>>> FAILED
>>>
>>>> /Erik
>>>>
>>>>> Thanks,
>>>>> Severin
>>>>>
More information about the build-dev
mailing list