Re: [Openvpn-devel] [PATCH v3] introduce V= level to manage t_client.sh output verbosity

2022-09-27 Thread Antonio Quartulli

Hi,

On 27/09/2022 11:43, Frank Lichtenheld wrote:

On Tue, Sep 20, 2022 at 03:23:51PM +0200, Gert Doering wrote:

If t_client.sh is run interactively, more verbose output is useful
to quickly see what it is doing.  If run from a CI environment, going
through lots of output for successful tests just to find the one that
failed is non-useful.

Introduce V= environment variable to control output verbosity

  V=0 - do not print any per-test output at all, just overall summary
  V=1 - print single header line for each successful test
print full output for failing tests
  V=99 - print full output, always, as before

default is V=1 now



Seems to do what it is supposed to do.

Acked-By: Frank Lichtenheld 


I also second this verdict:

Acked-by: Antonio Quartulli 


--
Antonio Quartulli


___
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel


Re: [Openvpn-devel] [PATCH v3] introduce V= level to manage t_client.sh output verbosity

2022-09-27 Thread Frank Lichtenheld
On Tue, Sep 20, 2022 at 03:23:51PM +0200, Gert Doering wrote:
> If t_client.sh is run interactively, more verbose output is useful
> to quickly see what it is doing.  If run from a CI environment, going
> through lots of output for successful tests just to find the one that
> failed is non-useful.
> 
> Introduce V= environment variable to control output verbosity
> 
>  V=0 - do not print any per-test output at all, just overall summary
>  V=1 - print single header line for each successful test
>print full output for failing tests
>  V=99 - print full output, always, as before
> 
> default is V=1 now
> 

Seems to do what it is supposed to do.

Acked-By: Frank Lichtenheld 

Regards,
-- 
  Frank Lichtenheld


___
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel