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.

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

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

Changes: https://git.openjdk.org/jdk/pull/10021/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=10021&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8292911
  Stats: 13 lines in 2 files changed: 13 ins; 0 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/10021.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/10021/head:pull/10021

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

Reply via email to