RFR: 8361894: sun/security/krb5/config/native/TestDynamicStore.java ensure that the test is run with sudo [v2]

Rajan Halade rhalade at openjdk.org
Wed Jul 16 17:34:43 UTC 2025


On Tue, 15 Jul 2025 14:10:19 GMT, Mikhail Yankelevich <myankelevich at openjdk.org> wrote:

>> sun/security/krb5/config/native/TestDynamicStore.java is commonly run without sudo, which leads to the access related failure. 
>> Added a ui popup reminding to run the test with sudo, making instructions clearer and more noticeable
>
> Mikhail Yankelevich has updated the pull request incrementally with one additional commit since the last revision:
> 
>   checking for root user automatically

test/jdk/sun/security/krb5/config/native/TestDynamicStore.java line 71:

> 69:                     .awaitAndCheck();
> 70:         }
> 71: 

Can the test fail here if the it is still run without sudo and `user.name` is not `root`? It could happen that executor clicks pass even when not run in sudo leading to unexpected failures which require additional triage. PASS/FAIL frame above is not needed but may be good to have as an instruction reminder.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26293#discussion_r2211071289


More information about the security-dev mailing list