Integrated: 8292911: Add a note about jtreg failure handlers in the testing doc

Jaikiran Pai jpai at openjdk.org
Mon Aug 29 02:29:18 UTC 2022


On Thu, 25 Aug 2022 10:44:32 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

> Can I please get a review of this change to the testing documentation which adds a note about failure handlers in jtreg? This addresses https://bugs.openjdk.org/browse/JDK-8292911.
> 
> Recently, there have been users who have been prompted for sudo password when running jtreg tests locally and those tests running into failures. This is because one of the commands that gets executed by the jtreg failure handler uses sudo (the `sudo dmesg` command on macos).
> 
> This commit updates the `testing.md` to explain what the failure handler does and also mentions the configuration that can be used to disable the failure handler. This commit doesn't specify how to configure the `sudoers` file if the user still wants to allow the failure handlers to run, because I thought that would be outside the scope of this documentation.
> 
> I attempted to auto generate the `testing.html` from the changes to `testing.md` using the `make update-build-docs` command, but that generated a lot of unnecessary diffs not just in `testing.html` but other html files in the `doc` directory. So, as suggested in a different PR which involved updates to these files, I manually added the section to the `testing.html` too.

This pull request has now been integrated.

Changeset: 99cab6a6
Author:    Jaikiran Pai <jpai at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/99cab6a68e16b6e39f82a7a4aa98a551d1df292b
Stats:     28 lines in 2 files changed: 28 ins; 0 del; 0 mod

8292911: Add a note about jtreg failure handlers in the testing doc

Reviewed-by: lancea, erikj, jjg

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

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



More information about the build-dev mailing list