Integrated: 8292206: TestCgroupMetrics.java fails as getMemoryUsage() is lower than expected

Ioi Lam iklam at openjdk.org
Thu Dec 22 20:53:53 UTC 2022


On Tue, 20 Dec 2022 00:03:25 GMT, Ioi Lam <iklam at openjdk.org> wrote:

> The `testMemoryUsage()` test scenario queries the total memory usage of all processes of the current Linux user, including other concurrently running jtreg test cases. Even if the current process allocates 256MB of ram, it's possible for other dying processes to release much more than that amount. Therefore, it's not possible to guarantee that `Metrics.getMemoryUsage()` would return a higher number.
> 
> I am removing this test scenario for now as I don't see it providing any actual value.
> 
> If we want to have more in-depth functional tests for the `Metrics.getMemoryXXX()` APIs, we need to do it inside a container in a more controlled setting.

This pull request has now been integrated.

Changeset: 6ccee839
Author:    Ioi Lam <iklam at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/6ccee839580fd9dc4cd4941b44dbbe3105202561
Stats:     32 lines in 3 files changed: 15 ins; 9 del; 8 mod

8292206: TestCgroupMetrics.java fails as getMemoryUsage() is lower than expected

Reviewed-by: dholmes, sgehwolf

-------------

PR: https://git.openjdk.org/jdk/pull/11734


More information about the hotspot-runtime-dev mailing list