Integrated: 8332885: Clarify failure_handler self-tests

2024-05-27 Thread Ludvig Janiuk
On Fri, 24 May 2024 12:16:25 GMT, Ludvig Janiuk wrote: > Adding commetns to clarify that the failure_handler selftests are intended to > be run manually. Ideally this would include a more thorough description of > the exepcted outputs, but this is what I have time to add right now.

Re: RFR: 8332885: Clarify failure_handler self-tests [v2]

2024-05-27 Thread Ludvig Janiuk
> Adding commetns to clarify that the failure_handler selftests are intended to > be run manually. Ideally this would include a more thorough description of > the exepcted outputs, but this is what I have time to add right now. Ludvig Janiuk has updated the pull request incrementally w

Integrated: 8332898: failure_handler: log directory of commands

2024-05-27 Thread Ludvig Janiuk
On Fri, 24 May 2024 14:34:39 GMT, Ludvig Janiuk wrote: > Also log the directory in which the command used by failure_handler was > executed. While often the same, it isn't always, and so it this should > simplify troubleshooting for someone looking at this at a glance with

Re: RFR: 8332885: Clarify failure_handler self-tests

2024-05-27 Thread Ludvig Janiuk
On Fri, 24 May 2024 19:26:29 GMT, Erik Joelsson wrote: >> Adding commetns to clarify that the failure_handler selftests are intended >> to be run manually. Ideally this would include a more thorough description >> of the exepcted outputs, but this is what I have time to add right now. > > test/

RFR: 8332898: failure_handler: log directory of commands

2024-05-24 Thread Ludvig Janiuk
Also log the directory in which the command used by failure_handler was executed. While often the same, it isn't always, and so it this should simplify troubleshooting for someone looking at this at a glance without being a failure_handler expert. Example output after this change:

RFR: 8332885: Clarify failure_handler self-tests

2024-05-24 Thread Ludvig Janiuk
Adding commetns to clarify that the failure_handler selftests are intended to be run manually. Ideally this would include a more thorough description of the exepcted outputs, but this is what I have time to add right now. - Commit messages: - 8332885 Clarify failure_handler self-te

Re: RFR: 8330755: ProblemList files have entries referring to non-existent tests [v2]

2024-04-24 Thread Ludvig Janiuk
On Wed, 24 Apr 2024 10:50:44 GMT, Doug Simon wrote: >> This PR adds a check for the format of ProblemList files and ensures they >> only have entries referring to existing tests. >> >> The cleanups in the second commit of this PR were done based on the output >> of `CheckProblemLists`: >> >>>

Integrated: 8302226 failure_handler native.core should wait for coredump to finish

2023-02-17 Thread Ludvig Janiuk
On Fri, 10 Feb 2023 18:19:01 GMT, Ludvig Janiuk wrote: > Update open/test/failure_handler/src/share/conf/(linux,mac).properties to > handle core dumps more correctly. This pull request has now been integrated. Changeset: 6120319a Author:Ludvig Janiuk Committer: Erik Joelsso

Re: RFR: 8302226 failure_handler native.core should wait for coredump to finish [v2]

2023-02-13 Thread Ludvig Janiuk
> Update open/test/failure_handler/src/share/conf/linux.properties to handle > core dumps more correctly. Ludvig Janiuk has updated the pull request incrementally with one additional commit since the last revision: add mac - Changes: - all: https://git.openjdk.org/jd

RFR: 8302226 failure_handler native.core should wait for coredump to finish on linux

2023-02-10 Thread Ludvig Janiuk
Update open/test/failure_handler/src/share/conf/linux.properties to handle core dumps more correctly. - Commit messages: - 8302226 failure_handler native.core should wait for coredump to finish on linux Changes: https://git.openjdk.org/jdk/pull/12515/files Webrev: https://webrevs