[ovs-dev] [PATCH v2] ovs-monitor-ipsec: Fix _nss_clear_database() parse error

Ilya Maximets i.maximets at ovn.org
Tue Jan 5 20:23:03 UTC 2021


On 12/23/20 5:01 PM, Mark Gray wrote:
> _nss_clear_database() runs `certutil` in order to get a list
> of certificates currently loaded in NSS. This fails with error:
> 
> "ovs-monitor-ipsec | ERR | Failed to clear NSS database.
> startswith first arg must be bytes or a tuple of bytes, not str"
> 
> Modify subprocess.Popen() to write in 'text' mode so that
> 'startwith' can correctly parse output.
> 
> Signed-off-by: Mark Gray <mark.d.gray at redhat.com>
> ---
> v2: 'text' parameter is unsupported in older versions of Python
>     (i.e. < 3.7), update to use 'universal_newlines'
> 
>  ipsec/ovs-monitor-ipsec.in | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

Thanks!

Applied to master and, since this is actually a bug fix, backported
down to 2.13.

Best regards, Ilya Maximets.


More information about the dev mailing list