Skip to content

Commit

Permalink
ovs-ofctl: Correctly mark the CT flush commands.
Browse files Browse the repository at this point in the history
Change the ct-flush and ct-flush-zone to RW
as they do in fact modify the state.

Fixes: 2a7c480 ("Add OpenFlow command to flush conntrack table entries.")
Fixes: 08146bf ("openflow: Add extension to flush CT by generic match.")
Signed-off-by: Ales Musil <amusil@redhat.com>
Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
  • Loading branch information
almusil authored and igsilya committed Nov 29, 2023
1 parent 40f651c commit 3626c17
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions utilities/ovs-ofctl.c
Original file line number Diff line number Diff line change
Expand Up @@ -5144,10 +5144,10 @@ static const struct ovs_cmdl_command all_commands[] = {
1, 1, ofctl_dump_ipfix_flow, OVS_RO },

{ "ct-flush-zone", "switch zone",
2, 2, ofctl_ct_flush_zone, OVS_RO },
2, 2, ofctl_ct_flush_zone, OVS_RW },

{ "ct-flush", "switch [zone=N] [ct-orig-tuple [ct-reply-tuple]]",
1, 4, ofctl_ct_flush, OVS_RO },
1, 4, ofctl_ct_flush, OVS_RW },

{ "ofp-parse", "file",
1, 1, ofctl_ofp_parse, OVS_RW },
Expand Down

0 comments on commit 3626c17

Please sign in to comment.