These should be considered active, because pluto is waiting for the
other side to react.  We should not remove them or try to repair.
Such connections have an extra text between the SA number and the
name of the connection.

Ideally, we would like not to parse the output of ipsec status, since
it's very error prone, but there is, unfortunately, no other interface.

Signed-off-by: Ilya Maximets <[email protected]>
---
 ipsec/ovs-monitor-ipsec.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ipsec/ovs-monitor-ipsec.in b/ipsec/ovs-monitor-ipsec.in
index d667c08bc..3b2057389 100755
--- a/ipsec/ovs-monitor-ipsec.in
+++ b/ipsec/ovs-monitor-ipsec.in
@@ -765,7 +765,7 @@ conn prevent_unencrypted_vxlan
         return conns
 
     def get_active_conns(self):
-        return self.get_conns_from_status(r"#\d+: \"(.*)\".*")
+        return self.get_conns_from_status(r"#\d+: .*\"(.*)\".*")
 
     def get_loaded_conns(self):
         return self.get_conns_from_status(r"\"(.*)\":.*(===|---).*")
-- 
2.46.0

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to