Re: [ovs-dev] [PATCH ovn v2] Do not set 'up' attribute in the port-binding table if ovn-northd is at an older version.

2022-05-18 Thread Mary Manohar
Hi Numan Sure, I have sent out a v3 with the updated commit message. Thanks Mary From: Numan Siddique Date: Wednesday, May 18, 2022 at 7:37 AM To: Mark Michelson Cc: Mary Manohar , Xavier Simonart , ovs-dev@openvswitch.org Subject: Re: [ovs-dev] [PATCH ovn v2] Do not set 'up' at

[ovs-dev] [PATCH ovn v3] ovn-controller: Set Port_Binding.up field only if the Southbound DB is aware of this field

2022-05-18 Thread mary . manohar
From: Mary Manohar This patch is fixing a backward compatibility issue when using an older ovn-northd (20.09). The 20.09 Southbound schema is not aware of the 'up' field in the Port_Binding table. ovn-controller should set Port_Binding.up field only if the Southbound DB is aware of

Re: [ovs-dev] [PATCH ovn v2] Do not set 'up' attribute in the port-binding table if ovn-northd is at an older version.

2022-05-12 Thread Mary Manohar
at 8:10 AM To: Mark Michelson Cc: Mary Manohar , ovs-dev@openvswitch.org Subject: Re: [ovs-dev] [PATCH ovn v2] Do not set 'up' attribute in the port-binding table if ovn-northd is at an older version. Hi Mark This is fixing a backward compatibility issue when using an older ovn-n

[ovs-dev] [PATCH ovn v2] Do not set 'up' attribute in the port-binding table if ovn-northd is at an older version.

2022-05-09 Thread mary . manohar
From: Mary Manohar Signed-off-by: Mary Manohar --- controller/binding.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/controller/binding.c b/controller/binding.c index e284704..e7dc537 100644 --- a/controller/binding.c +++ b/controller/binding.c @@ -908,7 +908,9

[ovs-dev] [PATCH ovn v1] Do not set 'up' attribute in the port-binding table if ovn-northd is at an older version.

2022-05-06 Thread mary . manohar
From: Mary Manohar Signed-off-by: mary.manohar --- controller/binding.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/controller/binding.c b/controller/binding.c index e284704..e7dc537 100644 --- a/controller/binding.c +++ b/controller/binding.c @@ -908,7 +908,9

[ovs-dev] [PATCH] OVN: Add NEWS for Policy-based routing

2019-04-16 Thread Mary Manohar
Signed-off-by: Mary Manohar --- NEWS | 3 +++ 1 file changed, 3 insertions(+) diff --git a/NEWS b/NEWS index 5a215b2..f3233d2 100644 --- a/NEWS +++ b/NEWS @@ -26,6 +26,9 @@ Post-v2.11.0 * Select IPAM mac_prefix in a random manner if not provided by the user * Added the HA chassis

[ovs-dev] [PATCH v4] Policy-based routing (PBR) in OVN.

2019-04-03 Thread Mary Manohar
ow. When the 'action' is reroute, if the nexthop ip-address is a connected router port or the IP address of a logical port, the logical-flow is constructed to route the packet to the nexthop ip-address. Signed-off-by: Mary Manohar --- ovn/northd/ovn-northd.c | 113

[ovs-dev] [PATCH v4] Policy-based Routing(PBR) in OVN

2019-04-03 Thread Mary Manohar
PBR provides a mechanism to configure permit/deny and reroute policies on the router. Permit/deny policies are similar to OVN ACLs, but exist on the logical-router. Reroute policies are needed for service-insertion and service-chaining. Currently, policies are stateless. Mary Manohar (1

[ovs-dev] [PATCH v3] Policy-based routing (PBR) provides a mechanism to configure permit/deny and reroute policies on the router. Permit/deny policies are similar to OVN ACLs, but exist on the logical

2019-01-16 Thread Mary Manohar
x27;match' string in the logical-flow. The action column is used to determine the action of the logical-flow. When the 'action' is reroute, if the nexthop ip-address is a connected router port or the IP address of a logical port, the logical-flow is constructed to rou

[ovs-dev] [PATCH v3] Policy-based routing

2019-01-16 Thread Mary Manohar
Mary Manohar (1): Policy-based routing (PBR) provides a mechanism to configure permit/deny and reroute policies on the router. Permit/deny policies are similar to OVN ACLs, but exist on the logical-router. Reroute policies are needed for service-insertion and service-chaining

[ovs-dev] [PATCH v2 2/3] Routing policies, add routing-policy commands in ovn-nbctl

2018-11-11 Thread Mary Manohar
610 ip4.dst==12.2.1.0/24 && ip4.src==11.2.1.0/24 reroute 13.2.1.12 600 ip4.dst==0.0.0.0/0 && ip4.src==0.0.0.0/0 drop e. Unit tests to validate these commands Signed-off-by: Mary Manohar --- ovn/utilities/ovn-nbctl.c | 198 +++

[ovs-dev] [PATCH v2 3/3] Routing policies, ovn-northd changes to handle routing policy commands.

2018-11-11 Thread Mary Manohar
a connected router port or the IP address of a logical port, the logical-flow is constructed to route the packet to the nexthop ip-address. Signed-off-by: Mary Manohar --- ovn/northd/ovn-northd.c | 144 ++-- 1 file changed, 138 insertions(+), 6 deleti

[ovs-dev] [PATCH v2 1/3] Routing policies, add routing-policy commands in ovn-nbctl

2018-11-11 Thread Mary Manohar
IP address. Each row in this table represents one routing policy for a logical router. The 'action' column for the highest priority matching row in this table determines a packet's treatment. If no row matches, packets are allowed by default. Signed-off-by: Mary Manohar --- ovn/ov

Re: [ovs-dev] [PATCH v1 0/3] Policy-based routing

2018-11-11 Thread Mary Manohar
traffic. Is there a reason why we can't specify a direction like we do with logical switch ACLs? And finally, the logging in ACLs is a nice feature and should also be in router policies. On 10/22/2018 06:24 PM, Mary Manohar wrote: > This patch series implements polic

[ovs-dev] [PATCH v1 3/3] [3/3]: Routing policies, ovn-northd changes to handle routing policy commands.

2018-10-22 Thread Mary Manohar
This Series: Policy-Based Routing. This Patch: Changes in ovn-northd to handle routing policy commands. --- ovn/northd/ovn-northd.c | 144 ++-- tests/ovn-nbctl.at | 47 2 files changed, 185 insertions(+), 6 deletions(-) diff --gi

[ovs-dev] [PATCH v1 1/3] [1/3]: Routing policies, add config in schema

2018-10-22 Thread Mary Manohar
This Series: Policy-Based Routing. This Patch: Add routing policies in the northbound schema. --- ovn/ovn-nb.ovsschema | 20 - ovn/ovn-nb.xml | 63 2 files changed, 82 insertions(+), 1 deletion(-) diff --git a/ovn/ovn-nb.

[ovs-dev] [PATCH v1 2/3] [2/3] Routing policies, add routing-policies in ovn-nbctl

2018-10-22 Thread Mary Manohar
This Series: Policy-Based Routing This patch: Add commands to ovn-nbctl to add/delete/list routing policies. Routing-policy commands: lr-policy-add ROUTER PRIORITY MATCH ACTION [NEXTHOP] add a policy to router lr-policy-del ROUTER [PRIORITY [MATCH]] lr-policy-list ROUTER print policies for ROUTER

[ovs-dev] [PATCH v1 0/3] Policy-based routing

2018-10-22 Thread Mary Manohar
next-hop. Mary Manohar (3): [1/3]: Routing policies, add config in schema [2/3] Routing policies, add routing-policies in ovn-nbctl [3/3]: Routing policies, ovn-northd changes to handle routing policy commands. ovn/northd/ovn-northd.c | 144 -- ovn/ovn