[ovs-dev] [PATCH v2] ofp-actions: Add delete field action

[William Tu]

On Sun, Apr 19, 2020 at 07:17:57AM -0700, William Tu wrote:
> 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 <yihung.wei at gmail.com>
> > ---
> LGTM.
> Acked-by: William Tu <u9012063 at gmail.com>
> 
> Let's see Ben or others have more comments.
> William
>  
Applied to master, thanks.