RFR: 8233712: Limit default tests jobs based on ulimit -u setting
Severin Gehwolf
sgehwolf at redhat.com
Fri Nov 15 16:31:51 UTC 2019
On Fri, 2019-11-15 at 08:27 -0800, Erik Joelsson wrote:
> 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.
I see.
> 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.
Thanks, Erik. I'll get this pushed on Monday if I hear no objections by
then.
Cheers,
Severin
> /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