Re: [ovs-dev] [PATCH OVN] ovn-nbctl.c: Add an optional way to delete router policy by uuid

2020-03-23 Thread taoyunxi...@cmss.chinamobile.com
Hi Ben,     I have added ovn-nbctl documentation, and summitted v2.  Thanks, Yun -- taoyunxi...@cmss.chinamobile.com >On Fri, Mar 20, 2020 at 05:34:33PM +0800, Tao YunXiang wrote: >> We can delete router policy by specify lr and more parameters. >> If CMS want to delete it exactly,

Re: [ovs-dev] [PATCH OVN] ovn-nbctl.c: Add an optional way to delete router policy by uuid

2020-03-20 Thread Ben Pfaff
On Fri, Mar 20, 2020 at 05:34:33PM +0800, Tao YunXiang wrote: > We can delete router policy by specify lr and more parameters. > If CMS want to delete it exactly, it must specify detailed "match" field. > It's not an easy way, also maybe deleted by mistake. > This change adds a way to specify lr

Re: [ovs-dev] [PATCH OVN] ovn-nbctl.c: Add an optional way to delete router policy by uuid

2020-03-20 Thread 0-day Robot
Bleep bloop. Greetings Tao YunXiang, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Line lacks whitespace around operator #34 FILE: utilities/ovn-nbctl.c:699:

[ovs-dev] [PATCH OVN] ovn-nbctl.c: Add an optional way to delete router policy by uuid

2020-03-20 Thread Tao YunXiang
We can delete router policy by specify lr and more parameters. If CMS want to delete it exactly, it must specify detailed "match" field. It's not an easy way, also maybe deleted by mistake. This change adds a way to specify lr and uuid, which is optional. You can still use the previous method to