Re: RFR: 8352926: New test TestDockerMemoryMetricsSubgroup.java fails [v6]

2025-05-21 Thread Mikhailo Seledtsov
On Wed, 21 May 2025 06:54:37 GMT, PAWAN CHAWDHARY wrote: >> 8352926: New test TestDockerMemoryMetricsSubgroup.java fails > > PAWAN CHAWDHARY has updated the pull request incrementally with one > additional commit since the last revision: > > address review comments Thanks for the updates. Ch

Re: RFR: 8352926: New test TestDockerMemoryMetricsSubgroup.java fails [v5]

2025-05-15 Thread Mikhailo Seledtsov
On Thu, 15 May 2025 10:03:11 GMT, PAWAN CHAWDHARY wrote: >> 8352926: New test TestDockerMemoryMetricsSubgroup.java fails > > PAWAN CHAWDHARY has updated the pull request incrementally with one > additional commit since the last revision: > > Remove unused import Marked as reviewed by mseledt

Re: RFR: 8354475: TestDockerMemoryMetricsSubgroup.java fails with exitValue = 1 [v2]

2025-05-14 Thread Mikhailo Seledtsov
On Tue, 13 May 2025 04:06:34 GMT, PAWAN CHAWDHARY wrote: >> 8354475: TestDockerMemoryMetricsSubgroup.java fails with exitValue = 1 > > PAWAN CHAWDHARY has updated the pull request incrementally with one > additional commit since the last revision: > > Update TestDockerMemoryMetricsSubgroup.ja

Re: RFR: 8354475: TestDockerMemoryMetricsSubgroup.java fails with exitValue = 1 [v2]

2025-05-14 Thread Mikhailo Seledtsov
On Tue, 13 May 2025 04:06:34 GMT, PAWAN CHAWDHARY wrote: >> 8354475: TestDockerMemoryMetricsSubgroup.java fails with exitValue = 1 > > PAWAN CHAWDHARY has updated the pull request incrementally with one > additional commit since the last revision: > > Update TestDockerMemoryMetricsSubgroup.ja

Re: RFR: 8352926: New test TestDockerMemoryMetricsSubgroup.java fails [v3]

2025-05-14 Thread Mikhailo Seledtsov
On Fri, 2 May 2025 10:00:07 GMT, PAWAN CHAWDHARY wrote: >> 8352926: New test TestDockerMemoryMetricsSubgroup.java fails > > PAWAN CHAWDHARY has updated the pull request incrementally with one > additional commit since the last revision: > > Refactor container runtime version code Overall cha

Re: RFR: 8341138: Rename jtreg property docker.support as container.support [v2]

2024-10-10 Thread Mikhailo Seledtsov
On Thu, 10 Oct 2024 14:27:08 GMT, Ramkumar Sunderbabu wrote: >> The System property "docker.support" defined in VMProps gives a wrong >> impression that it is tied to docker alone. The property is common for any >> container runtime. Hence, it needs to be renamed as "container.support". >> >>

Withdrawn: 8313718: make container at requires command configurable

2023-09-20 Thread Mikhailo Seledtsov
On Tue, 29 Aug 2023 22:01:22 GMT, Mikhailo Seledtsov wrote: > Container ecosystem is growing. It would be beneficial to define custom > command to figure out whether a specific test host or environment allows for > container testing. This enhancement seeks to make the command used

Re: RFR: 8313718: make container at requires command configurable [v2]

2023-08-31 Thread Mikhailo Seledtsov
On Wed, 30 Aug 2023 03:25:54 GMT, Mikhailo Seledtsov wrote: >> Container ecosystem is growing. It would be beneficial to define custom >> command to figure out whether a specific test host or environment allows for >> container testing. This enhancement seeks to make the com

Re: RFR: 8313718: make container at requires command configurable [v2]

2023-08-29 Thread Mikhailo Seledtsov
ifically > test/jtreg-ext/requires/VMProps.java checkContainerSupport(). Mikhailo Seledtsov has updated the pull request incrementally with one additional commit since the last revision: Fixed initialization issue - Changes: - all: https://git.openjdk.org/jdk/pull/1

Re: RFR: 8313718: make container at requires command configurable

2023-08-29 Thread Mikhailo Seledtsov
On Tue, 29 Aug 2023 22:01:22 GMT, Mikhailo Seledtsov wrote: > Container ecosystem is growing. It would be beneficial to define custom > command to figure out whether a specific test host or environment allows for > container testing. This enhancement seeks to make the command used

Re: RFR: 8313718: make container at requires command configurable

2023-08-29 Thread Mikhailo Seledtsov
On Wed, 30 Aug 2023 02:49:09 GMT, David Holmes wrote: >> Container ecosystem is growing. It would be beneficial to define custom >> command to figure out whether a specific test host or environment allows for >> container testing. This enhancement seeks to make the command used by jtreg >> "re

RFR: 8313718: make container at requires command configurable

2023-08-29 Thread Mikhailo Seledtsov
Container ecosystem is growing. It would be beneficial to define custom command to figure out whether a specific test host or environment allows for container testing. This enhancement seeks to make the command used by jtreg "requires" extension configurable, specifically test/jtreg-ext/requires

Re: RFR: 8308090: Add container tests for on-the-fly resource quota updates [v2]

2023-06-01 Thread Mikhailo Seledtsov
On Tue, 23 May 2023 09:04:11 GMT, Severin Gehwolf wrote: >> Please review these test changes which implement automatic testing of >> container resource updates without JVM restart. Note that this merely tests >> container detection code handling this case. It doesn't do anything special >> for

Integrated: 8296611: Problemlist several sun/security tests until JDK-8295343 is resolved

2022-11-09 Thread Mikhailo Seledtsov
On Tue, 8 Nov 2022 21:39:03 GMT, Mikhailo Seledtsov wrote: > Trivial change, updated jdk problem list. This pull request has now been integrated. Changeset: d6e2d0d0 Author: Mikhailo Seledtsov URL: https://git.openjdk.org/jdk/commit/d6e2d0d03d2161f934474fa1d4299513d14cb9c5 St

Re: RFR: 8296611: Problemlist several sun/security tests until JDK-8295343 is resolved [v2]

2022-11-09 Thread Mikhailo Seledtsov
On Wed, 9 Nov 2022 17:36:12 GMT, Mikhailo Seledtsov wrote: >> Trivial change, updated jdk problem list. > > Mikhailo Seledtsov has updated the pull request incrementally with one > additional commit since the last revision: > > Addressed review feedback Thank you Raja

Re: RFR: 8296611: Problemlist several sun/security and java/security tests until JDK-8295343 is resolved [v2]

2022-11-09 Thread Mikhailo Seledtsov
On Wed, 9 Nov 2022 17:36:12 GMT, Mikhailo Seledtsov wrote: >> Trivial change, updated jdk problem list. > > Mikhailo Seledtsov has updated the pull request incrementally with one > additional commit since the last revision: > > Addressed review feedback I have update

Re: RFR: 8296611: Problemlist several sun/security and java/security tests until JDK-8295343 is resolved

2022-11-09 Thread Mikhailo Seledtsov
On Tue, 8 Nov 2022 21:39:03 GMT, Mikhailo Seledtsov wrote: > Trivial change, updated jdk problem list. Thanks Sean. I will remove the fixed tests from the list. - PR: https://git.openjdk.org/jdk/pull/11045

Re: RFR: 8296611: Problemlist several sun/security and java/security tests until JDK-8295343 is resolved [v2]

2022-11-09 Thread Mikhailo Seledtsov
> Trivial change, updated jdk problem list. Mikhailo Seledtsov has updated the pull request incrementally with one additional commit since the last revision: Addressed review feedback - Changes: - all: https://git.openjdk.org/jdk/pull/11045/files - new: ht

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

Re: RFR: 8275662: remove test/lib/sun/hotspot

2022-07-08 Thread Mikhailo Seledtsov
On Fri, 8 Jul 2022 19:46:17 GMT, Coleen Phillimore wrote: > This change removes the last remnants of sun/hotspot/WhiteBox.java and other > classes, and uses the versions in jdk/test/whitebox. > I used sed to change sun.hotspot.{gc,code,cpuinfo} to jdk.test.whitebox and > deleted the old files a