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