Integrated: 8350824: New async logging gtest StallingModePreventsDroppedMessages fails

[Johan Sjölen]

On Thu, 27 Feb 2025 09:17:54 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:

> Hi,
> 
> When I added async stalling mode I also added this gtest early in the process, and as tier1/GHA passed I did not look at the test again. However, the test does not run until much deeper in the CI/CD process for us, as that's when we turn on async logging explicitly for all gtests.
> 
> Looking at it now, it's clearly buggy. It doesn't check which async mode UL is in, and as `drop` is default, that makes it bound to fail. Besides, the goal of it is to check whether async stalling mode drops messages or not during stress. We already have an excellent test for that in JTReg, and this does run early as it spawns its own VM with the correct asynchronous mode specified. My suggested fix is to delete the gtest and add a dropped message check to the JTReg test instead.
> 
> Testing: StressAsyncUL.java, GHA

This pull request has now been integrated.

Changeset: ac76d8d6
Author:    Johan Sjölen <jsjolen at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/ac76d8d63ff7b06a3c116559712a8b48f8acfa20
Stats:     24 lines in 2 files changed: 5 ins; 12 del; 7 mod

8350824: New async logging gtest StallingModePreventsDroppedMessages fails

Reviewed-by: mbaesken, dholmes

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

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