Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer [v4]

2024-02-26 Thread Daniel Fuchs
On Mon, 15 Jan 2024 08:36:43 GMT, Stefan Karlsson wrote: >> Tests using ProcessTools.executeProcess gets the following output written to >> stdout: >> [2023-11-24T09:58:16.797540608Z] Gathering output for process 2517117 >> [2023-11-24T09:58:23.070781345Z] Waiting for completion for process 2517

Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer [v4]

2024-01-15 Thread Stefan Karlsson
> Tests using ProcessTools.executeProcess gets the following output written to > stdout: > [2023-11-24T09:58:16.797540608Z] Gathering output for process 2517117 > [2023-11-24T09:58:23.070781345Z] Waiting for completion for process 2517117 > [2023-11-24T09:58:23.071045055Z] Waiting for completion f

Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer [v3]

2024-01-12 Thread Leonid Mesnik
On Fri, 5 Jan 2024 09:07:57 GMT, Stefan Karlsson wrote: >> Tests using ProcessTools.executeProcess gets the following output written to >> stdout: >> [2023-11-24T09:58:16.797540608Z] Gathering output for process 2517117 >> [2023-11-24T09:58:23.070781345Z] Waiting for completion for process 25171

Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer [v3]

2024-01-12 Thread Stefan Karlsson
On Fri, 12 Jan 2024 01:10:49 GMT, Leonid Mesnik wrote: > Needed to update copyrights now. Even when the code was written in 2023? - PR Comment: https://git.openjdk.org/jdk/pull/16807#issuecomment-1888699636

Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer [v3]

2024-01-11 Thread Leonid Mesnik
On Fri, 5 Jan 2024 09:07:57 GMT, Stefan Karlsson wrote: >> Tests using ProcessTools.executeProcess gets the following output written to >> stdout: >> [2023-11-24T09:58:16.797540608Z] Gathering output for process 2517117 >> [2023-11-24T09:58:23.070781345Z] Waiting for completion for process 25171

Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer [v3]

2024-01-05 Thread Jaikiran Pai
On Fri, 5 Jan 2024 09:07:57 GMT, Stefan Karlsson wrote: >> Tests using ProcessTools.executeProcess gets the following output written to >> stdout: >> [2023-11-24T09:58:16.797540608Z] Gathering output for process 2517117 >> [2023-11-24T09:58:23.070781345Z] Waiting for completion for process 25171

Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer [v3]

2024-01-05 Thread Stefan Karlsson
> Tests using ProcessTools.executeProcess gets the following output written to > stdout: > [2023-11-24T09:58:16.797540608Z] Gathering output for process 2517117 > [2023-11-24T09:58:23.070781345Z] Waiting for completion for process 2517117 > [2023-11-24T09:58:23.071045055Z] Waiting for completion f

Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer [v2]

2023-12-03 Thread David Holmes
On Thu, 30 Nov 2023 16:23:39 GMT, Stefan Karlsson wrote: >> Tests using ProcessTools.executeProcess gets the following output written to >> stdout: >> [2023-11-24T09:58:16.797540608Z] Gathering output for process 2517117 >> [2023-11-24T09:58:23.070781345Z] Waiting for completion for process 2517

Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer [v2]

2023-11-30 Thread Jaikiran Pai
On Thu, 30 Nov 2023 16:23:39 GMT, Stefan Karlsson wrote: >> Tests using ProcessTools.executeProcess gets the following output written to >> stdout: >> [2023-11-24T09:58:16.797540608Z] Gathering output for process 2517117 >> [2023-11-24T09:58:23.070781345Z] Waiting for completion for process 2517

Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer

2023-11-30 Thread Stefan Karlsson
On Thu, 30 Nov 2023 12:19:33 GMT, Jaikiran Pai wrote: > Hello Stefan, > > > The test were I want to use this is a long-running stress test that is > > known to be good at shaking out JVM hangs. It has been written to provide > > its own output about spawned processes and what they are doing, a

Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer [v2]

2023-11-30 Thread Stefan Karlsson
> Tests using ProcessTools.executeProcess gets the following output written to > stdout: > [2023-11-24T09:58:16.797540608Z] Gathering output for process 2517117 > [2023-11-24T09:58:23.070781345Z] Waiting for completion for process 2517117 > [2023-11-24T09:58:23.071045055Z] Waiting for completion f

Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer

2023-11-30 Thread Jaikiran Pai
On Fri, 24 Nov 2023 10:34:02 GMT, Stefan Karlsson wrote: > Tests using ProcessTools.executeProcess gets the following output written to > stdout: > [2023-11-24T09:58:16.797540608Z] Gathering output for process 2517117 > [2023-11-24T09:58:23.070781345Z] Waiting for completion for process 2517117

Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer

2023-11-30 Thread Jaikiran Pai
On Mon, 27 Nov 2023 08:27:25 GMT, Stefan Karlsson wrote: > So, I think you have found a separate issue. I'll open a separate issue for that and propose a change. - PR Comment: https://git.openjdk.org/jdk/pull/16807#issuecomment-1833661517

Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer

2023-11-27 Thread Stefan Karlsson
On Fri, 24 Nov 2023 17:01:29 GMT, Jaikiran Pai wrote: > Hello Stefan, I agree with David that I have found these logs to be useful - > it shows that process launched by the test hasn't yet finished and is being > waited upon. The log message when written will have a timestamp and can then > al

Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer

2023-11-27 Thread Stefan Karlsson
On Fri, 24 Nov 2023 17:13:30 GMT, Jaikiran Pai wrote: > Other than a quick local test run, I haven't tested it for anything else. It > did improve the situation in one of my test and it now only logs it once. In > the setup that you are seeing this large amount of logging, would you be able >

Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer

2023-11-24 Thread David Holmes
On Fri, 24 Nov 2023 17:13:30 GMT, Jaikiran Pai wrote: >> Tests using ProcessTools.executeProcess gets the following output written to >> stdout: >> [2023-11-24T09:58:16.797540608Z] Gathering output for process 2517117 >> [2023-11-24T09:58:23.070781345Z] Waiting for completion for process 2517117

Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer

2023-11-24 Thread Jaikiran Pai
On Fri, 24 Nov 2023 10:34:02 GMT, Stefan Karlsson wrote: > Tests using ProcessTools.executeProcess gets the following output written to > stdout: > [2023-11-24T09:58:16.797540608Z] Gathering output for process 2517117 > [2023-11-24T09:58:23.070781345Z] Waiting for completion for process 2517117

Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer

2023-11-24 Thread Jaikiran Pai
On Fri, 24 Nov 2023 10:34:02 GMT, Stefan Karlsson wrote: > Tests using ProcessTools.executeProcess gets the following output written to > stdout: > [2023-11-24T09:58:16.797540608Z] Gathering output for process 2517117 > [2023-11-24T09:58:23.070781345Z] Waiting for completion for process 2517117

Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer

2023-11-24 Thread Thomas Stuefe
On Fri, 24 Nov 2023 10:34:02 GMT, Stefan Karlsson wrote: > Tests using ProcessTools.executeProcess gets the following output written to > stdout: > [2023-11-24T09:58:16.797540608Z] Gathering output for process 2517117 > [2023-11-24T09:58:23.070781345Z] Waiting for completion for process 2517117

Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer

2023-11-24 Thread Stefan Karlsson
On Fri, 24 Nov 2023 13:06:05 GMT, David Holmes wrote: > I can't help but feel that if you disable this output then you are going to > be missing it in the cases that you need it! Something will go wrong and > there won't be enough info in the log to know what it was. My experience is that I wo

Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer

2023-11-24 Thread David Holmes
On Fri, 24 Nov 2023 10:34:02 GMT, Stefan Karlsson wrote: > Tests using ProcessTools.executeProcess gets the following output written to > stdout: > [2023-11-24T09:58:16.797540608Z] Gathering output for process 2517117 > [2023-11-24T09:58:23.070781345Z] Waiting for completion for process 2517117

Re: RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer

2023-11-24 Thread Stefan Karlsson
On Fri, 24 Nov 2023 10:34:02 GMT, Stefan Karlsson wrote: > Tests using ProcessTools.executeProcess gets the following output written to > stdout: > [2023-11-24T09:58:16.797540608Z] Gathering output for process 2517117 > [2023-11-24T09:58:23.070781345Z] Waiting for completion for process 2517117

RFR: 8320699: Add parameter to skip progress logging of OutputAnalyzer

2023-11-24 Thread Stefan Karlsson
Tests using ProcessTools.executeProcess gets the following output written to stdout: [2023-11-24T09:58:16.797540608Z] Gathering output for process 2517117 [2023-11-24T09:58:23.070781345Z] Waiting for completion for process 2517117 [2023-11-24T09:58:23.071045055Z] Waiting for completion finished fo