Adrian Moreno <amore...@redhat.com> writes:

> diff --git a/python/ovs/flows/ofp.py b/python/ovs/flows/ofp.py
> new file mode 100644
> index 000000000..1af06fa01
> --- /dev/null
> +++ b/python/ovs/flows/ofp.py
> @@ -0,0 +1,453 @@

[...]

> +
> +class OFPFlow(Flow):

[...]

> +    def __init__(self, ofp_string, id=None):
> +        """Create a OFPFlow from a flow string.
> +
> +        The string is expected to have the follwoing format:

following


[...]

> +    @staticmethod
> +    def _field_decoder_args():
> +        """Returns the KVDecoder arguments needed to decode match fields
> +        decoding match fields.

Huh?

> diff --git a/python/ovs/flows/ofp_act.py b/python/ovs/flows/ofp_act.py
> new file mode 100644
> index 000000000..9e04445ba
> --- /dev/null
> +++ b/python/ovs/flows/ofp_act.py
> @@ -0,0 +1,243 @@
> +""" Defines decoders for openflow actions.

Leading whitespace, and 'OpenFlow' for consistency?


[...]

> +def decode_learn(action_decoders):

[...]

> +    For this we need to create a wrapper of field_decoders that, for each
> +    "field=X" key-value we check if X is a field_name or if it's acually

actually

-- 
James
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to