Re: RFR: 8294740: Add cgroups keyword to TestDockerBasic.java

2022-10-03 Thread Ioi Lam
On Tue, 4 Oct 2022 01:10:41 GMT, David Holmes wrote: >> Please review this trivial fix. TestDockerBasic.java uses cgroups features >> so it should be tagged with `@key cgroups`. > > Looks good and trivial. Thanks for the quick fix. Thanks @dholmes-ora and @mseledts for the review. ---

Re: RFR: 8294740: Add cgroups keyword to TestDockerBasic.java

2022-10-03 Thread David Holmes
On Tue, 4 Oct 2022 01:03:25 GMT, Ioi Lam wrote: > Please review this trivial fix. TestDockerBasic.java uses cgroups features > so it should be tagged with `@key cgroups`. Looks good and trivial. Thanks for the quick fix. - Marked as reviewed by dholmes (Reviewer). PR: https://gi

Re: RFR: 8294740: Add cgroups keyword to TestDockerBasic.java

2022-10-03 Thread Mikhailo Seledtsov
On Tue, 4 Oct 2022 01:03:25 GMT, Ioi Lam wrote: > Please review this trivial fix. TestDockerBasic.java uses cgroups features > so it should be tagged with `@key cgroups`. Marked as reviewed by mseledtsov (Committer). - PR: https://git.openjdk.org/jdk/pull/10547

RFR: 8294740: Add cgroups keyword to TestDockerBasic.java

2022-10-03 Thread Ioi Lam
Please review this trivial fix. TestDockerBasic.java uses cgroups features so it should be tagged with `@key cgroups`. - Commit messages: - 8294740: Add cgroups keyword to TestDockerBasic.java Changes: https://git.openjdk.org/jdk/pull/10547/files Webrev: https://webrevs.openjdk.o