[ovs-dev] [PATCH OVN] ovn-nbctl.c: Fix lr-policy-del command

Tao YunXiang taoyunxiang at cmss.chinamobile.com
Wed May 13 02:03:39 UTC 2020


This change will check the existence of lr-policy uuid.
If not, it will print "uuid is not found".

Fixes: 1b030874c32("ovn-nbctl.c: Add an optional way to delete router policy by uuid")


Author: Tao YunXiang <taoyunxiang at cmss.chinamobile.com>
Signed-off-by: Tao YunXiang <taoyunxiang at cmss.chinamobile.com>
---
 utilities/ovn-nbctl.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/utilities/ovn-nbctl.c b/utilities/ovn-nbctl.c
index 67b23108f..2dabd27de 100644
--- a/utilities/ovn-nbctl.c
+++ b/utilities/ovn-nbctl.c
@@ -3642,6 +3642,10 @@ nbctl_lr_policy_del(struct ctl_context *ctx)
                     new_policies[n_policies++] = lr->policies[i];
                 }
             }
+            if (n_policies == lr->n_policies) {
+                ctl_error(ctx, "uuid is not found");
+                }
+
     /* If match is not specified, delete all routing policies with the
      * specified priority. */
         } else {
-- 
2.17.1





More information about the dev mailing list