diff options
| author | Adrian Moreno <amorenoz@redhat.com> | 2022-12-19 17:13:52 +0100 |
|---|---|---|
| committer | Ilya Maximets <i.maximets@ovn.org> | 2022-12-21 19:48:07 +0100 |
| commit | 863d2e1a8c2a6ced49a49024c094ef6a9aa7e55a (patch) | |
| tree | 3433431716440f832a82a2c56ec7040115a0847e /python | |
| parent | 22eb2243864d42580dd1447cf09906d4d34fbb68 (diff) | |
| download | openvswitch-863d2e1a8c2a6ced49a49024c094ef6a9aa7e55a.tar.gz | |
python: Don't exit OFPFlow constructor.
Returning None in a constructor does not make sense and is just error
prone. Removing what was a leftover from an attempt to handle a common
error case of trying to parse what is commonly outputted by ovs-ofctl.
This should be done by the caller anyway.
Signed-off-by: Adrian Moreno <amorenoz@redhat.com>
Acked-by: Mike Pattrick <mkp@redhat.com>
Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
Diffstat (limited to 'python')
| -rw-r--r-- | python/ovs/flow/ofp.py | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/python/ovs/flow/ofp.py b/python/ovs/flow/ofp.py index eac8d0851..20231fd9f 100644 --- a/python/ovs/flow/ofp.py +++ b/python/ovs/flow/ofp.py @@ -104,9 +104,6 @@ class OFPFlow(Flow): ValueError if the string is malformed. ParseError if an error in parsing occurs. """ - if " reply " in ofp_string: - return None - sections = list() parts = ofp_string.split("actions=") if len(parts) != 2: |