[ovs-dev] [Suspected-Phishing]Re: [Suspected-Phishing]Re: [PATCH v2 1/3] netdev: Dynamic per-port Flow API.
roid at mellanox.com
Tue Jul 2 12:19:17 UTC 2019
On 2019-07-02 2:23 PM, Roi Dayan wrote:
>>> Regarding deprecation, I'd like to remove all the functionality from ovs-dpctl
>>> utility keeping only ability to remove system datapaths, since this, IMHO,
>>> is the only useful function.
>>>> Imagine some flows are in TC and some in OVS. I guess dpctl will only
>>>> show the OVS ones.
>>> Yes, and this is documented.
>> I missed the documentation but bisected to find the offending commit. I
>> think it's better to deprecate as you suggested above, maybe with some
>> message to refer users to the documentation.
> was added when u try to use the type arg.
> # ovs-dpctl dump-flows type=tc
> ovs-dpctl: Invalid argument 'type'. Use 'ovs-appctl dpctl/dump-flows' instead.
> just output the msg all the time would have been annoying.
I didn't understand at first what Eli intended but I do now.
Eli suggest ovs-dpctl will always output a deprecation notice
to use ovs-appctl instead. so all users will see it and start
migrating to ovs-appctl. Is that accepted ?
can be in any ovs-dpctl or specific ones like dump-flows but
not only if type arg used but always.
More information about the dev