[ovs-dev] [PATCH 3/3] ovn-northd: SNAT in either direction of gateway router.

sformation, we should not do it again. Signed-off-by: Gurucharan Shetty --- ovn/lib/actions.c | 8 ovn/lib/logical-fields.c| 2 ++ ovn/lib/logical-fields.h| 4 ovn/northd/ovn-northd.8.xml | 49 +++-- ovn/northd/ovn-northd.

[ovs-dev] [PATCH 2/3] ovn: Ability to skip some IP addresses for SNAT.

ss-through without any SNAT done on it. For e.g a subnet that is routable in the external network. This commit adds a "nosnat" option to the NAT table. Signed-off-by: Gurucharan Shetty --- ovn/northd/ovn-northd.8.xml | 8 ovn/northd/ovn-northd.c | 38 +++

[ovs-dev] [PATCH 1/3] logical-fields: Add ct.dnat and ct.snat symbols.

Upcoming commits use it. Signed-off-by: Gurucharan Shetty --- ovn/lib/logical-fields.c | 2 ++ ovn/ovn-sb.xml | 19 +-- tests/ovn.at | 4 +++- 3 files changed, 22 insertions(+), 3 deletions(-) diff --git a/ovn/lib/logical-fields.c b/ovn/lib/logical

[ovs-dev] [PATCH v2] ovn: Add a case of policy based routing.

t effected in the policy based routing table itself. Signed-off-by: Gurucharan Shetty --- ovn/northd/ovn-northd.c | 24 +++-- ovn/ovn-nb.ovsschema | 8 +- ovn/ovn-nb.xml| 26 + ovn/utilities/ovn-nbctl.8.xml | 8 +- ovn/utilities/ovn-nbctl.c | 43 ++

[ovs-dev] [PATCH v2] ovn-controller: Container can have connection to a hosting VM.

: Gurucharan Shetty --- AUTHORS | 1 + ovn/controller/physical.c | 27 --- tests/ovn.at | 195 ++ 3 files changed, 211 insertions(+), 12 deletions(-) diff --git a/AUTHORS b/AUTHORS index a30a5d8..2525265 100644 --- a

[ovs-dev] [PATCH] ovn-controller: Container can have connection to a hosting VM.

: Gurucharan Shetty --- AUTHORS | 1 + ovn/controller/physical.c | 18 ++--- tests/ovn.at | 179 ++ 3 files changed, 190 insertions(+), 8 deletions(-) diff --git a/AUTHORS b/AUTHORS index a30a5d8..2525265 100644 --- a

[ovs-dev] [PATCH v3] INSTALL.md: Add details about kernel module preference.

Signed-off-by: Gurucharan Shetty --- INSTALL.md | 23 ++- 1 file changed, 22 insertions(+), 1 deletion(-) diff --git a/INSTALL.md b/INSTALL.md index bb40e4a..a073465 100644 --- a/INSTALL.md +++ b/INSTALL.md @@ -322,9 +322,30 @@ Building the Sources 4. Run "make instal

[ovs-dev] [PATCH v2] INSTALL.md: Add details about kernel module preference.

Signed-off-by: Gurucharan Shetty --- INSTALL.md | 23 ++- 1 file changed, 22 insertions(+), 1 deletion(-) diff --git a/INSTALL.md b/INSTALL.md index bb40e4a..6ecefdf 100644 --- a/INSTALL.md +++ b/INSTALL.md @@ -322,9 +322,30 @@ Building the Sources 4. Run "make instal

[ovs-dev] [PATCH] INSTALL.md: Add details about kernel module preference.

Signed-off-by: Gurucharan Shetty --- INSTALL.md | 22 +- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git a/INSTALL.md b/INSTALL.md index bb40e4a..a9b70db 100644 --- a/INSTALL.md +++ b/INSTALL.md @@ -322,9 +322,29 @@ Building the Sources 4. Run "make instal

[ovs-dev] [PATCH v2] ovn: Initialize the databases.

Without initializing the databases, commands of the form 'ovn-nbctl --wait=sb ls-add ls0' will simply hang. Signed-off-by: Gurucharan Shetty --- ovn/northd/ovn-northd.c | 10 +++--- ovn/utilities/ovn-ctl | 2 ++ 2 files changed, 9 insertions(+), 3 deletions(-) diff --git a/

[ovs-dev] [PATCH] ovn-ctl: Initialize the databases.

Without initializing the databases, commands of the form 'ovn-nbctl --wait=sb ls-add ls0' will simply hang. Signed-off-by: Gurucharan Shetty --- ovn/utilities/ovn-ctl | 2 ++ 1 file changed, 2 insertions(+) diff --git a/ovn/utilities/ovn-ctl b/ovn/utilities/ovn-ctl index 07bff8

[ovs-dev] [PATCH 2/2] ovn-northd: Format logical flows a little better.

This is a cosmetic fix. Signed-off-by: Gurucharan Shetty --- ovn/northd/ovn-northd.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c index 648bd88..8b6f6a3 100644 --- a/ovn/northd/ovn-northd.c +++ b/ovn/northd/ovn

[ovs-dev] [PATCH 1/2] ovn-northd: Fix a memory leak.

Signed-off-by: Gurucharan Shetty --- ovn/northd/ovn-northd.c | 1 + 1 file changed, 1 insertion(+) diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c index 5ccb516..648bd88 100644 --- a/ovn/northd/ovn-northd.c +++ b/ovn/northd/ovn-northd.c @@ -2570,6 +2570,7 @@ build_stateful(struct

[ovs-dev] [PATCH] ofctrl: Fix logic for mff_ovn_geneve.

Fixes: 7fa39c6b9376e99 ("Fix memory leak in recv_S_TLV_TABLE_REQUESTED().") Signed-off-by: Gurucharan Shetty --- ovn/controller/ofctrl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ovn/controller/ofctrl.c b/ovn/controller/ofctrl.c index fe72d79..6ea593b 100644

[ovs-dev] [PATCH] ovn-controller: Fix group_id allocation.

installed new groups. This commit fixes the above scenario. Signed-off-by: Gurucharan Shetty --- include/ovn/actions.h | 2 ++ ovn/controller/ofctrl.c | 6 +- ovn/lib/actions.c | 3 +++ 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/include/ovn/actions.h b/include/ovn

[ovs-dev] [PATCH] ovn-northd: Tag allocation for nested containers.

When there are hundreds of nodes controlled by OVN, the workflow to track and allocate unique tags across multiple hosts becomes complicated. It is much easier to let ovn-northd do the allocation. Signed-off-by: Gurucharan Shetty --- ovn/northd/ovn-northd.c | 134

[ovs-dev] [PATCH] ovs-bugtool: Add interfaces file contents.

It is useful to know the contents of interfaces file for debugging in debian based systems. Signed-off-by: Gurucharan Shetty --- utilities/bugtool/ovs-bugtool.in | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/utilities/bugtool/ovs-bugtool.in b/utilities/bugtool/ovs

[ovs-dev] [PATCH 2/2] ovn-northd: Add load-balancers to gateway routers.

chooses a DNAT action. Established connections go through the DNAT table for a regular DNAT. Signed-off-by: Gurucharan Shetty --- ovn/northd/ovn-northd.8.xml | 63 +++ ovn/northd/ovn-northd.c | 150 +--- ovn/ovn-nb.ovsschema|

[ovs-dev] [PATCH 0/2] load-balancer fix for north-south direction.

load_balancer to the gateway router too. It has the added benefit that it works fine with floating IPs for north-south traffic. i.e., floating ips can be load-balanced to logical ips. Gurucharan Shetty (2): ovn-controller: Datapath based conntrack zone for load-balancing. ovn-northd: Add load-balancers

[ovs-dev] [PATCH 1/2] ovn-controller: Datapath based conntrack zone for load-balancing.

Currently ct_lb() logical action is only added for a logical switch and we use the conntrack zone allocated for the logical port. A future commit will use ct_lb() for a logical router too. In that case, use the allocated DNAT zone. Signed-off-by: Gurucharan Shetty --- include/ovn/actions.h

[ovs-dev] [PATCH v2] system-ovn.at: Fix occasional test failures.

A run of the NAT unit tests hundred times showed some occasional unit test failures. With this commit, I haven't seen any failures for 100 runs. Reported-by: Joe Stringer Signed-off-by: Gurucharan Shetty --- tests/system-ovn.at | 8 ++-- 1 file changed, 6 insertions(+), 2 dele

[ovs-dev] [PATCH] ovn-controller: Don't clear desired groups during backlog.

Doing so would mean that we will lose the desired groups forever because of incremental processing (till the next change in NB database). Signed-off-by: Gurucharan Shetty --- ovn/controller/ofctrl.c | 1 - 1 file changed, 1 deletion(-) diff --git a/ovn/controller/ofctrl.c b/ovn/controller

[ovs-dev] [PATCH] system-ovn.at: Fix occasional test failures.

A run of the NAT unit tests hundred times showed some occasional unit test failures. With this commit, I haven't seen any failures for 100 runs. Reported-by: Joe Stringer Signed-off-by: Gurucharan Shetty --- tests/system-ovn.at | 8 ++-- 1 file changed, 6 insertions(+), 2 dele

[ovs-dev] [PATCH v2 1/2] ovn: Ability to set multiple load balancers.

The schema prevented one from setting more than one load balancer to a switch. This removes that anomaly. (The test has a slightly unrelated change where it increases the number of connections to get rid of some occasional unit test failures.) Signed-off-by: Gurucharan Shetty --- ovn/northd

[ovs-dev] [PATCH v2 2/2] system-ovn.at: Add another load-balancing test.

The client and servers are in the same subnet. (This would not have worked without the recent IPv6 patches that let packets to loop back) Signed-off-by: Gurucharan Shetty --- tests/system-ovn.at | 118 1 file changed, 118 insertions(+) diff

[ovs-dev] [PATCH 2/2] system-ovn.at: Add another load-balancing test.

The client and servers are in the same subnet. (This would not have worked without the recent IPv6 patches that let packets to loop back) Signed-off-by: Gurucharan Shetty --- tests/system-ovn.at | 117 1 file changed, 117 insertions(+) diff

[ovs-dev] [PATCH 1/2] ovn: Ability to set multiple load balancers.

The schema prevented one from setting more than one load balancer to a switch. This removes that anamoly. Signed-off-by: Gurucharan Shetty --- ovn/northd/ovn-northd.c | 38 +++--- ovn/ovn-nb.ovsschema| 6 +++--- tests/system-ovn.at | 19

[ovs-dev] [PATCH] ovn-northd: Warn when the peer of a router port is a switch port.

Signed-off-by: Gurucharan Shetty --- ovn/northd/ovn-northd.c | 8 1 file changed, 8 insertions(+) diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c index 7ce509d..0a093c7 100644 --- a/ovn/northd/ovn-northd.c +++ b/ovn/northd/ovn-northd.c @@ -2603,6 +2603,14

[ovs-dev] [PATCH] ovn.at: A "peer" is only for interconnected routers.

s.") Signed-off-by: Gurucharan Shetty --- tests/ovn.at | 20 +--- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git a/tests/ovn.at b/tests/ovn.at index 12de125..4f077d4 100644 --- a/tests/ovn.at +++ b/tests/ovn.at @@ -1508,8 +1508,7 @@ done ovn-nbctl lr-add lr0

[ovs-dev] [PATCH v2] ovn-northd: Combine two NAT loops into one.

Signed-off-by: Gurucharan Shetty --- ovn/northd/ovn-northd.c | 47 ++- 1 file changed, 14 insertions(+), 33 deletions(-) diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c index b1c2c6c..52e3229 100644 --- a/ovn/northd/ovn-northd.c +++ b

[ovs-dev] [PATCH v2 2/2] system-ovn.at: Add a OVN load-balancing unit test.

Signed-off-by: Gurucharan Shetty --- tests/system-ovn.at | 125 1 file changed, 125 insertions(+) diff --git a/tests/system-ovn.at b/tests/system-ovn.at index b4cbc85..13f380f 100644 --- a/tests/system-ovn.at +++ b/tests/system-ovn.at

[ovs-dev] [PATCH v2 1/2] system-ovn.at: Add a OVN NAT test using OVN gateway.

multi-host scenarios (without NAT testing), so it should still be a decent coverage. Signed-off-by: Gurucharan Shetty --- tests/automake.mk | 2 + tests/system-kmod-testsuite.at | 1 + tests/system-ovn.at | 169 tests

[ovs-dev] [PATCH] ovn-northd: Combine two NAT loops into one.

It is wasteful to have two loops that go through all the NAT entries. Signed-off-by: Gurucharan Shetty --- ovn/northd/ovn-northd.c | 45 + 1 file changed, 13 insertions(+), 32 deletions(-) diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn

[ovs-dev] [PATCH 4/4] system-ovn.at: Add a OVN load-balancing unit test.

Signed-off-by: Gurucharan Shetty --- Please note that there are a couple of unit tests around fragmentation (unrelated to OVN) that can cause kernel crashes when you run OVN kernel tests. So, if you intend to run these, run it via: make check-kmod TESTSUITEFLAGS="-k ovn" --- tests/sys

[ovs-dev] [PATCH 2/4] system-common-macros.at: Add a iface-id to interfaces.

IntegrationGuide.md explains the meaning of the key 'iface-id'. Adding it as part of ADD_VETH is useful for OVN related tests. For non-OVN tests, it should have no effect. Upcoming commits take advantage of this. Signed-off-by: Gurucharan Shetty --- tests/system-common-macros.at | 3 +

[ovs-dev] [PATCH 3/4] system-ovn.at: Add a OVN NAT test using OVN gateway.

multi-host scenarios (without NAT testing), so it should still be a decent coverage. Signed-off-by: Gurucharan Shetty --- Please note that there are a couple of unit tests around fragmentation (unrelated to OVN) that can cause kernel crashes when you run OVN kernel tests. So, if you intend to run

[ovs-dev] [PATCH 1/4] system-common-macros: Enhance ADD_VETH to take mac and gateway.

It is useful to set the mac address and gateway while using ADD_VETH to connect a namespace to a OVN logical topology. Upcoming commits use this enhancement. Signed-off-by: Gurucharan Shetty --- tests/system-common-macros.at | 11 ++- 1 file changed, 10 insertions(+), 1 deletion

[ovs-dev] [PATCH v2] ovn-controller: Change strategy for gateway conntrack zone allocation.

n the commit. i.e. ovn-controller would crash. ) Signed-off-by: Gurucharan Shetty --- This applies on top of https://patchwork.ozlabs.org/patch/646268/. I will rebase this if the dependent patch changes. --- ovn/controller/ovn-controller.c | 4 ++-- ovn/controller/ovn-controller.h | 2 +- ovn/contr

[ovs-dev] [PATCH 2/2] ovn-controller: Change strategy for gateway conntrack zone allocation.

processing.") Signed-off-by: Gurucharan Shetty --- This applies on top of https://patchwork.ozlabs.org/patch/646268/. I will rebase this if the dependent patch changes. --- ovn/controller/ovn-controller.c | 4 ++-- ovn/controller/ovn-controller.h | 2 +- ovn/controller/patch.c | 4 +++-

[ovs-dev] [PATCH] INSTALL.Docker.md: Clarify OVS python library path.

Reported-by: Taekho Nam Signed-off-by: Gurucharan Shetty --- AUTHORS | 1 + INSTALL.Docker.md | 23 +++ 2 files changed, 20 insertions(+), 4 deletions(-) diff --git a/AUTHORS b/AUTHORS index 704ba40..5d86dc6 100644 --- a/AUTHORS +++ b/AUTHORS @@ -432,6 +432,7

[ovs-dev] [PATCH 1/2] ovn-northd: Ability to loop-back in a router.

anges that behavior and lets an OVN router loop-back the packet via the same interface. Signed-off-by: Gurucharan Shetty --- ovn/northd/ovn-northd.8.xml | 10 ++ ovn/northd/ovn-northd.c | 10 ++ 2 files changed, 20 insertions(+) diff --git a/ovn/northd/ovn-northd.8.xml b/ovn/n

[ovs-dev] [PATCH 2/2] ovn-nb: Update documentation for Load balancers.

Clarify that the load balancers needs a router in the topology. Signed-off-by: Gurucharan Shetty --- ovn/ovn-nb.xml | 7 +++ 1 file changed, 7 insertions(+) diff --git a/ovn/ovn-nb.xml b/ovn/ovn-nb.xml index ff2e695..a1f560a 100644 --- a/ovn/ovn-nb.xml +++ b/ovn/ovn-nb.xml @@ -567,6

[ovs-dev] [PATCH 1/2] ovn-nbctl: Ability to update a route's output port.

The command 'lr-route-add' currently cannot update the output port. Signed-off-by: Gurucharan Shetty --- ovn/utilities/ovn-nbctl.c | 4 tests/ovn-nbctl.at| 10 +- 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/ovn/utilities/ovn-nbctl.c b/ovn/uti

[ovs-dev] [PATCH 2/2] ovn: Add a case of policy based routing.

. Since each host gets a specific subnet, we can use source IP address based policy routing to decide on the gateway router. Signed-off-by: Gurucharan Shetty --- ovn/northd/ovn-northd.c | 20 +++-- ovn/ovn-nb.ovsschema | 8 +- ovn/ovn-nb.xml| 22 + ovn/utilities/ovn

[ovs-dev] [PATCH 7/7] ovn-northd: Logical flows for load balancers.

on that needs to be load balanced is given a ct_lb("$IP_LIST") action. Signed-off-by: Gurucharan Shetty --- ovn/northd/ovn-northd.c | 201 +--- 1 file changed, 190 insertions(+), 11 deletions(-) diff --git a/ovn/northd/ovn-northd.c b/ovn/northd

[ovs-dev] [PATCH 6/7] ovn-controller: Add support for load balancing.

(192.168.1.2, 192.168.1.3); ct_lb(192.168.1.2:80, 192.168.1.3:80); Creates an OpenFlow group with multiple buckets and equal weights that changes the destination IP address (and port number) of the packet statefully to one of the options provided inside the parenthesis. Signed-off-by: Gurucharan

[ovs-dev] [PATCH 2/7] ovn-northd.8: Update documentation.

: Gurucharan Shetty --- ovn/northd/ovn-northd.8.xml | 40 ++-- 1 file changed, 22 insertions(+), 18 deletions(-) diff --git a/ovn/northd/ovn-northd.8.xml b/ovn/northd/ovn-northd.8.xml index 260cc14..4d712a1 100644 --- a/ovn/northd/ovn-northd.8.xml +++ b/ovn/northd

[ovs-dev] [PATCH 5/7] ovn-nb: Add Load-balancer table to schema.

Also add the ability to run database commands on the new schema using ovn-nbctl. Signed-off-by: Gurucharan Shetty --- ovn/ovn-nb.ovsschema | 22 -- ovn/ovn-nb.xml| 41 + ovn/utilities/ovn-nbctl.c | 4 3 files

[ovs-dev] [PATCH 1/7] ovn-northd: Split ACL and pre-ACL processing.

Future patches introduce more tables between pre-ACL and ACL processing. As such, it looks easier to separate these out into separate functions to enhance code readability. Signed-off-by: Gurucharan Shetty Acked-by: Ben Pfaff --- ovn/northd/ovn-northd.c | 26 ++ 1 file

[ovs-dev] [PATCH 3/7] ovn-northd: Introduce pre-stateful table.

l table in-turn will send the packet to conntrack if 'reg0[0]' is 1. Signed-off-by: Gurucharan Shetty --- ovn/northd/ovn-northd.8.xml | 36 ++- ovn/northd/ovn-northd.c | 46 +++-- 2 files changed, 63 inserti

[ovs-dev] [PATCH 4/7] ovn-northd: Introduce stateful table.

load balancing, this commit introduces a new stateful table that is responsible to commit packets to conntrack via ct_commit action. If ACL table needs to commit a packet, it sets 'reg0[1]' as 1. Stateful table in-turn will commit the packet if 'reg0[1]' is 1. Signed-off-b

[ovs-dev] [PATCH 0/7] OVN load-balancers.

cer \ vips:'"30.0.0.1:8000"'='"172.16.1.2:80,172.16.1.3:80,172.16.1.4:80"' And then attach the created load_balancer to any logical switch where the client resides. e.g.: ovn-nbctl set logical_switch foo \ load_balancer=91facf32-d6f1-43e9-bead-54c59af493e1

[ovs-dev] [PATCH v4] ovn: DNAT and SNAT on a gateway router.

could combine case1 and case2 with nat type="dnat_and_snat" if the IP addresses are the same. case3 : SNAT and DNAT (east-west traffic) When bar1 pings 30.0.0.2, the traffic jumps to the gateway router and loops back to foo1 with a source ip address of 30.0.0.1 Signed-off-by: Gurucharan

[ovs-dev] [PATCH 0/8] Make ovs-bugtool flake8 compatible.

*** BLURB HERE *** Gurucharan Shetty (8): ovs-bugtool.in: Remove unused imports. ovs-bugtool.in: Fix errors around spaces and line length. ovs-bugtool.in: Remove unused variables. ovs-bugtool.in: Remove usage of 'has_key'. ovs-bugtool.in: Test for membership should be '

[ovs-dev] [PATCH 4/8] ovs-bugtool.in: Remove usage of 'has_key'.

Signed-off-by: Gurucharan Shetty --- utilities/bugtool/ovs-bugtool.in | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/utilities/bugtool/ovs-bugtool.in b/utilities/bugtool/ovs-bugtool.in index cd1924d..95ea482 100755 --- a/utilities/bugtool/ovs

[ovs-dev] [PATCH 3/8] ovs-bugtool.in: Remove unused variables.

Signed-off-by: Gurucharan Shetty --- utilities/bugtool/ovs-bugtool.in | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/utilities/bugtool/ovs-bugtool.in b/utilities/bugtool/ovs-bugtool.in index 605a6bf..cd1924d 100755 --- a/utilities/bugtool/ovs-bugtool.in +++ b/utilities

[ovs-dev] [PATCH 2/8] ovs-bugtool.in: Fix errors around spaces and line length.

Signed-off-by: Gurucharan Shetty --- utilities/bugtool/ovs-bugtool.in | 235 ++- 1 file changed, 135 insertions(+), 100 deletions(-) diff --git a/utilities/bugtool/ovs-bugtool.in b/utilities/bugtool/ovs-bugtool.in index 9e1f535..605a6bf 100755 --- a/utilities

[ovs-dev] [PATCH 1/8] ovs-bugtool.in: Remove unused imports.

Also take care of a 'import not at top of file' warning from flake8. Signed-off-by: Gurucharan Shetty --- utilities/bugtool/ovs-bugtool.in | 12 +++- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/utilities/bugtool/ovs-bugtool.in b/utilities/bugtool/ovs-bugtoo

[ovs-dev] [PATCH v3 3/5] ovn-controller: Refactor conntrack zone allocation.

commit moves conntrack zone allocation code to ovn-controller.c where it would be easily accessible for router zone allocation too. Signed-off-by: Gurucharan Shetty --- ovn/controller/binding.c| 61 - ovn/controller/binding.h| 5 ++-- ovn

[ovs-dev] [PATCH v3 5/5] ovn: DNAT and SNAT on a gateway router.

ffic jumps to the gateway router and loops back to foo1 with a source ip address of 30.0.0.3 Signed-off-by: Gurucharan Shetty --- ovn/lib/actions.c | 83 + ovn/northd/ovn-northd.8.xml | 129 +++--- ovn/northd/ovn-northd.c | 214 +++

[ovs-dev] [PATCH v3 4/5] ovn-controller: Assign conntrack zones for gateway router.

OVS NAT currently cannot do snat and dnat in the same zone. So we need two zones per gateway router. Signed-off-by: Gurucharan Shetty --- ovn/controller/ovn-controller.c | 52 +++-- ovn/controller/ovn-controller.h | 2 ++ ovn/controller/patch.c | 6

[ovs-dev] [PATCH v3 2/5] ovn: Introduce l3 gateway router.

#x27;. The ovn-controller on that chassis populates the "chassis" column for this record as an indication for other ovn-controllers of its physical location. Other ovn-controllers treat this port as they would treat a VM/Container port on a different chassis. Signed-off-by: Gurucharan Sh

[ovs-dev] [PATCH v3 1/5] ovn-northd: Support connecting multiple routers to a switch.

tch. With the above goal in mind, this commit gives the general ability to connect multiple routers via a switch. Signed-off-by: Gurucharan Shetty --- ovn/controller/lflow.c | 19 ++-- ovn/northd/ovn-northd.c | 56 +++- ovn/ovn-nb.xml | 7 -- tests/ovn.at

[ovs-dev] [PATCH] tests: Add valgrind targets for ovn utilities and dameons.

Signed-off-by: Gurucharan Shetty --- tests/automake.mk |4 1 file changed, 4 insertions(+) diff --git a/tests/automake.mk b/tests/automake.mk index a5c6074..211a80d 100644 --- a/tests/automake.mk +++ b/tests/automake.mk @@ -152,6 +152,10 @@ check-lcov: all tests/atconfig tests/atlocal

[ovs-dev] [RFC 5/5 v2] ovn: DNAT and SNAT on a gateway router.

Add a SNAT rule in R2: ovn-nbctl set logical_router R2 snat:"192.168.1.0/24"=30.0.0.1 (You need a static route in R1 to send packets destined to outside world to go through R2) When foo1 pings alice1, alice1 receives traffic from 30.0.0.1 case3 : SNAT and DNAT (east-west traffic) You

[ovs-dev] [PATCH 4/5] ovn-controller: Assign conntrack zones for gateway router.

OVS NAT currently cannot do snat and dnat in the same zone. So we need two zones per gateway router. Signed-off-by: Gurucharan Shetty --- No v1. Newly added as part of v2. --- ovn/controller/ovn-controller.c | 50 +-- ovn/controller/ovn-controller.h |2

[ovs-dev] [PATCH 3/5 v2] ovn-controller: Refactor conntrack zone allocation.

commit moves conntrack zone allocation code to ovn-controller.c where it would be easily accessible for router zone allocation too. Signed-off-by: Gurucharan Shetty --- No v1. Newly added as part of v2. --- ovn/controller/binding.c| 61 --- ovn

[ovs-dev] [PATCH 2/5] ovn: Introduce l3 gateway router.

#x27;. The ovn-controller on that chassis populates the "chassis" column for this record as an indication for other ovn-controllers of its physical location. Other ovn-controllers treat this port as they would treat a VM/Container port on a different chassis. Signed-off-by: Gurucharan She

[ovs-dev] [PATCH 1/5 v2] ovn-northd: Support connecting multiple routers to a switch.

tch. With the above goal in mind, this commit gives the general ability to connect multiple routers via a switch. Signed-off-by: Gurucharan Shetty --- v1->v2: No change. --- ovn/controller/lflow.c | 19 ++-- ovn/northd/ovn-northd.c | 56 ++- ovn/ovn-nb.xml |7 --

[ovs-dev] [PATCH] ovn: Introduce l3 gateway router.

#x27;. The ovn-controller on that chassis populates the "chassis" column for this record as an indication for other ovn-controllers of its physical location. Other ovn-controllers treat this port as they would treat a VM/Container port on a different chassis. Signed-off-by: Gurucharan She

[ovs-dev] [PATCH] ovn-northd: Support connecting multiple routers to a switch.

tch. With the above goal in mind, this commit gives the general ability to connect multiple routers via a switch. Signed-off-by: Gurucharan Shetty --- This needs the following 2 commits under review to first go in. 1. ofproto-dpif: Rename "recurse" to "indentation". 2. ofproto-

[ovs-dev] [PATCH] ovs-save: Handle MTU changes correctly.

ntation about this. VMware-BZ: #1638654 Signed-off-by: Gurucharan Shetty --- utilities/ovs-save |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/utilities/ovs-save b/utilities/ovs-save index 90320b1..d4d3c35 100755 --- a/utilities/ovs-save +++ b/utilities/ovs-save @@ -65,7 +

[ovs-dev] [PATCH] ovn-northd: Add support for static_routes.

static routes are useful when connecting multiple routers with each other. Signed-off-by: Gurucharan Shetty --- This patch does not add a outport to the schema. I think that enhancement should be easily addable as soon as there is an upstream user. We still have 2-3 months before the next

[ovs-dev] [PATCH] ovn-docker: Update overlay mode for new tcp ports.

There were changes made recently wherein 2 ovsdb-server is started for northbound and southbound databases with tcp ports 6641 and 6642. This breaks Docker integration. This commit fixes it. Signed-off-by: Gurucharan Shetty --- INSTALL.Docker.md | 10 ++ ovn

[ovs-dev] [PATCH] INSTALL.md: Explain the effect of using different configure options.

Over the years, I have seen multiple users inadvertantly end up with 2 copies of OVS executables in their filesystem. In all the cases, it was because of using different configure options while installing a new version of Open vSwitch. Signed-off-by: Gurucharan Shetty --- INSTALL.md | 19

[ovs-dev] [PATCH] ovn-northd: Fix peering of routers.

27;. It looked to me that since the router peer is a logical object that has to be created in OVN-NB database, we always need to statically assign the mac address. So this patch picks the mac address from the database. Signed-off-by: Gurucharan Shetty --- ovn/northd/ovn-northd.c | 28 ++

[ovs-dev] [PATCH] ovs-vtep: Delete flows with untagged vlan carefully.

-off-by: Gurucharan Shetty --- AUTHORS |1 + vtep/ovs-vtep |4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/AUTHORS b/AUTHORS index 96bdc4f..e7c68da 100644 --- a/AUTHORS +++ b/AUTHORS @@ -353,6 +353,7 @@ Michael Shigorinm...@osdn.org.ua Mihir Gangar

[ovs-dev] [RFC 1/8] system-traffic: Load balancing.

From: Jarno Rajahalme Signed-off-by: Jarno Rajahalme --- lib/ofp-actions.c| 3 +- ofproto/ofproto-dpif-xlate.c | 21 tests/system-traffic.at | 225 +++ 3 files changed, 248 insertions(+), 1 deletion(-) diff --git a/lib/ofp-action

[ovs-dev] [RFC 7/8] ovn-northd: Introduce the loadbalancing table.

3). This introduces 2 OVN lflow actions. 1. ct_nat. This is used to send packets to conntrack to NAT them for already established connections. 2. ct_lb($ips). This is used to choose one of the $ips as destination ip address for new connections that need to be loadbalanced. Signed-off-by: Gurucha

[ovs-dev] [RFC 2/8] ovn-northd: Split ACL and pre-ACL processing.

Future patches introduce more tables between pre-ACL and ACL processing. As such, it looks easier to separate these out into separate functions to enhance code readability. Signed-off-by: Gurucharan Shetty --- ovn/northd/ovn-northd.c | 26 ++ 1 file changed, 18

[ovs-dev] [RFC 6/8] ovn-northd: Pre-loadbalancing table.

This new table sits before the pre-Stateful table and sets 'reg0' as 1, if the destination ip address of a packet is a VIP in a loadbalancer object. Setting 'reg0' as 1 will send the packet through conntrack to get its status (or to track it.) Signed-off-by: Gurucharan Shetty

[ovs-dev] [RFC 8/8] ovn-controller: Handle logical flows of loadbalancers.

Signed-off-by: Gurucharan Shetty --- ovn/controller/lflow.c | 3 +- ovn/controller/lflow.h | 2 + ovn/controller/ofctrl.c | 134 +- ovn/controller/ofctrl.h | 3 +- ovn/controller/ovn-controller.c | 22 ++- ovn/lib

[ovs-dev] [RFC 5/8] ovn-nb: Add Load-balancer table to schema.

Also add the ability to run database commands on the new schema using ovn-nbctl. Signed-off-by: Gurucharan Shetty --- ovn/ovn-nb.ovsschema | 18 -- ovn/ovn-nb.xml| 24 ovn/utilities/ovn-nbctl.c | 4 3 files changed, 44 insertions

[ovs-dev] [RFC 4/8] ovn-northd: Introduce stateful table.

NAT and loadbalancing, this commit introduces a new stateful table that is responsible to commit packets to conntrack via ct_commit action. If ACL table needs to commit a packet, it sets 'reg1' as 1. Stateful table in-turn will commit the packet if 'reg1' is 1. Signed-off-b

[ovs-dev] [RFC 3/8] ovn-northd: Introduce pre-stateful table.

le in-turn will send the packet to conntrack if 'reg0' is 1. Signed-off-by: Gurucharan Shetty --- ovn/northd/ovn-northd.8.xml | 37 +--- ovn/northd/ovn-northd.c | 68 ++--- 2 files changed, 73 insertions(+), 32 deletions(

[ovs-dev] [RFC 0/8] OVN east-west loadbalancing.

tream) compiled in. Gurucharan Shetty (7): ovn-northd: Split ACL and pre-ACL processing. ovn-northd: Introduce pre-stateful table. ovn-northd: Introduce stateful table. ovn-nb: Add Load-balancer table to schema. ovn-northd: Pre-loadbalancing table. ovn-northd: Introduce the loadbalancing

[ovs-dev] [PATCH] Revert "ovs-lib: Try to call exit before killing."

e: 1. Any configured ip addresses will be lost 2. Traffic will stop flowing. Reported-by: Edgar Cantu Signed-off-by: Gurucharan Shetty --- AUTHORS |1 + utilities/ovs-lib.in |9 + 2 files changed, 2 insertions(+), 8 deletions(-) diff --git a/AUTHORS b/AUTHORS ind

[ovs-dev] [PATCH] openvswitch-switch.init: Redirect error to /dev/null.

The latest Open vSwitch kernel module from Upstream linux does not list "version". During debian package installation, we would see a non-harmful error in the log. This patch suppresses it. Signed-off-by: Gurucharan Shetty --- debian/openvswitch-switch.init | 13 + 1 fi

[ovs-dev] [PATCH] debian: Move ovs-lib to openvswitch-common.

ovs-lib is used by both ovs-ctl and ovn-ctl. So it is useful to move it to openvswitch-common Signed-off-by: Gurucharan Shetty --- debian/control|3 +-- debian/openvswitch-common.install |1 + debian/openvswitch-switch.install |1 - 3 files changed, 2 insertions

[ovs-dev] [PATCH v2 1/5] ovn-ctl: Add daemon status functions.

Signed-off-by: Gurucharan Shetty Acked-by: Russell Bryant Acked-by: Ben Pfaff --- patch already applied. --- ovn/utilities/ovn-ctl |6 ++ 1 file changed, 6 insertions(+) diff --git a/ovn/utilities/ovn-ctl b/ovn/utilities/ovn-ctl index 3e2ccf9..b171934 100755 --- a/ovn/utilities/ovn

[ovs-dev] [PATCH v2 4/5] debian: Add a package for OVN central components.

Signed-off-by: Gurucharan Shetty --- v1-v2: change name from openvswitch-ovn-central to ovn-central --- debian/.gitignore |1 + debian/automake.mk |7 ++ debian/control | 16 + debian/ovn-central.dirs |1 + debian/ovn-central.init

[ovs-dev] [PATCH v2 5/5] debian: Add a package for OVN docker drivers.

Signed-off-by: Gurucharan Shetty --- v1-v2: change the name from openvswitch-ovn-docker to ovn-docker --- debian/.gitignore |1 + debian/automake.mk|1 + debian/control| 18 ++ debian/ovn-docker.install |2 ++ 4 files changed, 22

[ovs-dev] [PATCH v2 2/5] debian: Add a package for OVN common components.

Signed-off-by: Gurucharan Shetty --- v1-v2: change name of the package from openvswitch-ovn-common to ovn-common --- debian/.gitignore |1 + debian/automake.mk |2 ++ debian/control | 14 ++ debian/ovn-common.install |3 +++ debian/ovn

[ovs-dev] [PATCH v2 3/5] debian: Add a package for host components.

Signed-off-by: Gurucharan Shetty --- v1-v2: change the name of package from openvswitch-ovn-controller to ovn-host --- debian/.gitignore|1 + debian/automake.mk |7 ++ debian/control | 16 ++ debian/ovn-host.dirs |1 + debian/ovn

[ovs-dev] [PATCH 4/5] debian: Add a package for OVN central components.

Signed-off-by: Gurucharan Shetty --- debian/.gitignore |1 + debian/automake.mk |7 debian/control | 16 ++ debian/openvswitch-ovn-central.dirs |1 + debian/openvswitch-ovn-central.init | 53

[ovs-dev] [PATCH 2/5] debian: Add a package for OVN common components.

Signed-off-by: Gurucharan Shetty --- debian/.gitignore |1 + debian/automake.mk |2 ++ debian/control | 14 ++ debian/openvswitch-ovn-common.install |3 +++ debian/openvswitch-ovn-common.manpages |4

[ovs-dev] [PATCH 5/5] debian: Add a package for OVN docker drivers.

Signed-off-by: Gurucharan Shetty --- debian/.gitignore |1 + debian/automake.mk|1 + debian/control| 18 ++ debian/openvswitch-ovn-docker.install |2 ++ 4 files changed, 22 insertions(+) create mode

[ovs-dev] [PATCH 3/5] debian: Add a package for ovn-controller.

Signed-off-by: Gurucharan Shetty --- debian/.gitignore |1 + debian/automake.mk |7 debian/control | 16 + debian/openvswitch-ovn-controller.dirs |1 + debian/openvswitch-ovn

[ovs-dev] [PATCH 1/5] ovn-ctl: Add daemon status functions.

Signed-off-by: Gurucharan Shetty --- ovn/utilities/ovn-ctl |6 ++ 1 file changed, 6 insertions(+) diff --git a/ovn/utilities/ovn-ctl b/ovn/utilities/ovn-ctl index 3e2ccf9..b171934 100755 --- a/ovn/utilities/ovn-ctl +++ b/ovn/utilities/ovn-ctl @@ -225,6 +225,12 @@ case $command in

[ovs-dev] [PATCH] debian: Skip systemctl redirect.

like a call to ifup when invoked in this manner does not return. I am not sure why this is happening. We can avoid the above behavior completely by skipping the systemctl redirect as done in this commit. Signed-off-by: Gurucharan Shetty --- debian/openvswitch-switch.init |2 ++ 1 file chang

  1   2   3   4   5   6   7   8   9   10   >