On Tue, Apr 14, 2020 at 01:33:28PM -0700, Yi-Hung Wei wrote: > This patch adds a new OpenFlow action, delete field, to delete a > field in packets. Currently, only the tun_metadata fields are > supported. > > One use case to add this action is to support multiple versions > of geneve tunnel metadatas to be exchanged among different versions > of networks. For example, we may introduce tun_metadata2 to > replace old tun_metadata1, but still want to provide backward > compatibility to the older release. In this case, in the new > OpenFlow pipeline, we would like to support the case to receive a > packet with tun_metadata1, do some processing. And if the packet > is going to a switch in the newer release, we would like to delete > the value in tun_metadata1 and set a value into tun_metadata2. > > Currently, ovs does not provide an action to remove a value in > tun_metadata if the value is present. This patch fulfills the gap > by adding the delete_field action. For example, the OpenFlow > syntax to delete tun_metadata1 is: > > actions=delete_field:tun_metadata1 > > Signed-off-by: Yi-Hung Wei <[email protected]> > --- LGTM. Acked-by: William Tu <[email protected]>
Let's see Ben or others have more comments. William _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
