On Fri, 9 Jul 2021 13:53:27 GMT, Matthias Baesken wrote:
> > OK. Please also add a test on the hotspot side. You may want to add
> > relevant parts to `TestMisc.java`.
>
> Thanks for the suggestion, I will look into TestMisc.java .
I added some HS testing code in the latest commit.
Best
On Fri, 9 Jul 2021 13:42:15 GMT, Severin Gehwolf wrote:
> OK. Please also add a test on the hotspot side. You may want to add relevant
> parts to `TestMisc.java`.
Thanks for the suggestion, I will look into TestMisc.java .
-
PR: https://git.openjdk.java.net/jdk/pull/4518
On Fri, 9 Jul 2021 11:35:34 GMT, Matthias Baesken wrote:
> > This looks pretty good now. Looking forward to seeing container tests for
> > this new code.
>
> Hi Severin , I did some adjustments following your suggestions.
> I added docker based test coding for testing pids-limit (with limits
On Tue, 29 Jun 2021 08:21:51 GMT, Severin Gehwolf wrote:
> This looks pretty good now. Looking forward to seeing container tests for
> this new code.
Hi Severin , I did some adjustments following your suggestions.
I added docker based test coding for testing pids-limit (with limits and also
On Wed, 23 Jun 2021 13:37:59 GMT, Matthias Baesken wrote:
>> Hello, please review this PR; it extend the OSContainer API in order to also
>> support the pids controller of cgroups.
>>
>> I noticed that unlike the other controllers "cpu", "cpuset", "cpuacct",
>> "memory" on some older Linux
On Wed, 23 Jun 2021 14:48:22 GMT, Severin Gehwolf wrote:
> > But I think that the testing needs to be enhanced (e.g. with some added
> > docker tests?). Do you have some good suggestions
> > where I could look at existing (docker?) tests and adjust those for the new
> > pids.max ?
>
> Have a
On Wed, 23 Jun 2021 13:38:58 GMT, Matthias Baesken wrote:
> But I think that the testing needs to be enhanced (e.g. with some added
> docker tests?). Do you have some good suggestions
> where I could look at existing (docker?) tests and adjust those for the new
> pids.max ?
Have a look at
On Wed, 23 Jun 2021 13:37:59 GMT, Matthias Baesken wrote:
>> Hello, please review this PR; it extend the OSContainer API in order to also
>> support the pids controller of cgroups.
>>
>> I noticed that unlike the other controllers "cpu", "cpuset", "cpuacct",
>> "memory" on some older Linux
> Hello, please review this PR; it extend the OSContainer API in order to also
> support the pids controller of cgroups.
>
> I noticed that unlike the other controllers "cpu", "cpuset", "cpuacct",
> "memory" on some older Linux distros (SLES 12.1, RHEL 7.1) the pids
> controller might not be