Can I please get a review of this change which addresses the issue noted in https://bugs.openjdk.org/browse/JDK-8316946?
Failure handler actions can specify a "successArtifacts" property value, which are file name(s) for which hypherlinks will be created in the "processes.html" file. This property value can contain replacable placeholder tokens like "%p" (process id) "%iterCount" (iteration count, if the action's command is repeated). The issue here was caused by a bug in the handling of the "successArtifacts" value. If there were more than one process for which we were running the action (of collecting thread dumps), then the original (raw) value of "successArtifacts" property wasn't being held on to. As a result, for the second process for which the hyperlink was being created, an incorrect link would be generated. The commit here retains the raw value of "successArtifacts" property in the `PatternAction` (just like we do for "args" property) so that it can be used to repeatedly replace the placeholders, each time the action is run for a different process. I've run this change against our internal CI to verify it doesn't cause any regressions. tier1, tier2 and tier3 was run successfully. Additionally, I have run a couple of jtreg test cases which intentionally timeout, so that I can verify that the generated "processes.html" now contains the correct hyperlinks to the thread dump files for each process. ------------- Commit messages: - 8316946: jtreg failure handler pages are mislabelling the jcmd/thread/dump_to_file results. Changes: https://git.openjdk.org/jdk/pull/15939/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=15939&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8316946 Stats: 9 lines in 1 file changed: 6 ins; 0 del; 3 mod Patch: https://git.openjdk.org/jdk/pull/15939.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/15939/head:pull/15939 PR: https://git.openjdk.org/jdk/pull/15939
