Introduce bfd and bfd_consumer nodes to northd I-P engine to track bfd
connections and northd static_route/policy_route changes.
Reported-at: https://issues.redhat.com/browse/FDP-600
Signed-off-by: Lorenzo Bianconi
---
Changes since v1:
- add incremental processing logic for bfd_consumer node
Add missing bfd option in ovn-nbctl manual for lr-policy-add command
Fixes: 62d5491c0155 ("northd: Add BFD support for ECMP route policy.")
Signed-off-by: Lorenzo Bianconi
---
utilities/ovn-nbctl.8.xml | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/utilities/o
> On Thu, May 2, 2024 at 8:22 AM Lorenzo Bianconi
> wrote:
> >
> > Introduce bfd and bfd_consumer nodes to northd I-P engine to track bfd
> > connections and northd static_route/policy_route changes.
> >
> > Reported-at: https://issues.redhat.com/browse/FDP-600
/browse/FDP-96
Signed-off-by: Lorenzo Bianconi
---
northd/northd.c | 15 +--
tests/ovn.at| 8
tests/system-ovn.at | 22 +-
3 files changed, 38 insertions(+), 7 deletions(-)
diff --git a/northd/northd.c b/northd/northd.c
index 0cabda7ea..29dc58ef4
Introduce bfd and bfd_consumer nodes to northd I-P engine to track bfd
connections and northd static_route/policy_route changes.
Reported-at: https://issues.redhat.com/browse/FDP-600
Signed-off-by: Lorenzo Bianconi
---
northd/en-lflow.c| 19 +--
northd/en-northd.c | 92
Reported-at: https://issues.redhat.com/browse/FDP-362
Signed-off-by: Lorenzo Bianconi
---
Changes since v3:
- fix M_NS_DAEMONIZE macro
Changes since v2:
- minor changes
Changes since v1:
- move logic in consider_port_binding
- add more self-test
- fix typos
---
controller/lflow.h| 1
Introduce bfd and bfd_consumer nodes to northd I-P engine to track bfd
connections and northd static_route/policy_route changes.
Signed-off-by: Lorenzo Bianconi
---
northd/en-lflow.c| 19 +--
northd/en-northd.c | 92 +
northd/en-northd.h | 8 ++
northd/inc
Signed-off-by: Lorenzo Bianconi
---
Changes since v2:
- minor changes
Changes since v1:
- move logic in consider_port_binding
- add more self-test
- fix typos
---
controller/lflow.h| 1 +
controller/physical.c | 30 +++-
northd/northd.c | 33 +++--
northd/ovn
tric reply.
We do it or am I missing something?
Regards,
Lorenzo
>
> Instead, the ECMP routes are never changed. Other configuration changes are
> made, and conntrack is flushed between each stage of the test. Am I missing
> something here?
>
> On 3/12/24 11:59, Lorenzo Bianconi wr
for ECMP route policy.")
Reported-at: https://issues.redhat.com/browse/FDP-543
Signed-off-by: Lorenzo Bianconi
---
northd/northd.c | 29 -
tests/system-ovn.at | 13 +
2 files changed, 29 insertions(+), 13 deletions(-)
diff --git a/northd/northd.c b/n
Add missing bfd option to lr-route-add and lr-policy-add commands in
ovn-nbctl manpage.
Reported-at: https://issues.redhat.com/browse/FDP-550
Signed-off-by: Lorenzo Bianconi
---
utilities/ovn-nbctl.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/utilities/ovn-nbctl.c b/utilities/ovn
Signed-off-by: Lorenzo Bianconi
---
Changes since v1:
- move logic in consider_port_binding
- add more self-test
- fix typos
---
controller/lflow.h | 1 +
controller/physical.c | 30 +++-
northd/northd.c | 35 +++---
northd/ovn-northd.8.xml | 16 +++--
tests/multinode.at
Avoid recirculating IGMP/MLD packets more than one time from stage
ls_out_pre_lb in the egress pipeline to ovn table 37 in order to avoid
packet looping for ovn-ic deployment.
Acked-by: Mohammad Heib
Signed-off-by: Lorenzo Bianconi
---
controller/pinctrl.c | 2 ++
include/ovn/logical
> On Thu, Oct 19, 2023 at 7:47 PM Lorenzo Bianconi <
> lorenzo.bianc...@redhat.com> wrote:
>
> > Avoid recirculating IGMP/MLD packets more than one time from stage
> > ls_out_pre_lb in the egress pipeline to ovn table 37 in order to avoid
> > packet looping fo
Introduce ECMP_Nexthop table in the SB db in order to track active
ecmp-symmetric-reply connections and flush stale ones.
Signed-off-by: Lorenzo Bianconi
---
northd/en-northd.c | 4 +++
northd/inc-proc-northd.c | 8 +++--
northd/northd.c | 73
Introduce the nexthop identifier in the ct_label.label field for
ecmp-symmetric replies connections. This field will be used by
ovn-controller to track ct entries and to flush them if requested by the
CMS (e.g. removing the related static routes).
Signed-off-by: Lorenzo Bianconi
---
northd/en
Introduce ecmp_nexthop_monitor in ovn-controller in order to track and
flush ecmp-symmetric reply ct entires when requested by the CMS (e.g
removing the related static routes).
Signed-off-by: Lorenzo Bianconi
---
controller/ofctrl.c | 101 ++
controller/ofctrl.h
Reported-at: https://issues.redhat.com/browse/FDP-56
Changes since v1:
- add ID column in ECMP_Nexthop table in SB db
- remove nexthop-id in logical_router_static_route option column
Lorenzo Bianconi (3):
northd: Introduce ECMP_Nexthop table in SB db.
northd: Add nexhop id in ct_label.label
> Hi Lorenzo,
>
> Just a couple of small comments below.
>
> On 3/7/24 08:19, Lorenzo Bianconi wrote:
> > Introduce the nexthop identifier in the ct_label.label field for
> > ecmp-symmetric replies connections. This field will be used by
> > ovn-controller to t
> Hi Lorenzo,
Hi Mark,
>
> I have some comments below.
thx for the review.
>
> On 3/7/24 08:19, Lorenzo Bianconi wrote:
> > Introduce ECMP_Nexthop table in the SB db in order to track active
> > ecmp-symmetric-reply connections and flush stale ones.
> >
&g
Introduce ECMP_Nexthop table in the SB db in order to track active
ecmp-symmetric-reply connections and flush stale ones.
Signed-off-by: Lorenzo Bianconi
---
northd/en-northd.c | 4 ++
northd/inc-proc-northd.c | 8 +++-
northd/northd.c | 98
Reported-at: https://issues.redhat.com/browse/FDP-56
Lorenzo Bianconi (3):
northd: Introduce ECMP_Nexthop table in SB db.
northd: Add nexhop id in ct_label.label.
ofctrl: Introduce ecmp_nexthop_monitor.
controller/ofctrl.c | 102 ++
controller/ofctrl.h | 2
Introduce ecmp_nexthop_monitor in ovn-controller in order to track and
flush ecmp-symmetric reply ct entires when requested by the CMS (e.g
removing the related static routes).
Signed-off-by: Lorenzo Bianconi
---
controller/ofctrl.c | 102 ++
controller/ofctrl.h
Introduce the nexthop identifier in the ct_label.label field for
ecmp-symmetric replies connections. This field will be used by
ovn-controller to track ct entries and to flush them if requested by the
CMS (e.g. removing the related static routes).
Signed-off-by: Lorenzo Bianconi
---
northd
Enable automatic static route configuration when NAT is created with
--ad-route option for gw routers similar to what is currently supported
for distributed routers with gw_router_ports.
Reported-at: https://issues.redhat.com/browse/FDP-244
Signed-off-by: Lorenzo Bianconi
---
northd/northd.c
Similar to what is already implemented for routed e/w traffic,
introduce pmtud support for e/w traffic between two logical switch ports
connected to the same logical switch, but running on two different
hypervisors.
Reported-at: https://issues.redhat.com/browse/FDP-362
Signed-off-by: Lorenzo
> Hi Lorenzo,
>
> I have a few comments below.
>
> On 1/17/24 06:11, Lorenzo Bianconi wrote:
> > Similar to OVN static routes, introduce the capability to link a BFD
> > session for OVN reroute policies.
> >
> > Reported-at: https://issues.redhat.com/br
Add the capbility to mark (through pkt.mark) incoming/outgoing packets
in logical_switch datapath according to user configured QoS rule.
Co-developed-by: Dumitru Ceara
Reported-at: https://issues.redhat.com/browse/FDP-42
Signed-off-by: Lorenzo Bianconi
---
Changes since v2:
- set max dscp/mark
Similar to OVN static routes, introduce the capability to link a BFD
session for OVN reroute policies.
Reported-at: https://issues.redhat.com/browse/FDP-234
Signed-off-by: Lorenzo Bianconi
---
NEWS | 1 +
northd/northd.c | 86
abase
unnecessarily.")
Signed-off-by: Lorenzo Bianconi
---
utilities/ovn-nbctl.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/utilities/ovn-nbctl.c b/utilities/ovn-nbctl.c
index 526369b68..0586eccdb 100644
--- a/utilities/ovn-nbctl.c
+++ b/utilities/ovn-nbctl.c
@@ -4394,6 +4394,7 @@ nbctl_pre_lr
Fix regression introduce in BFD system tests
Acked-by: Mark Michelson
Fixes: 1655a6c146ca ("northd, utils: support for RouteTables in LRs")
Signed-off-by: Lorenzo Bianconi
---
tests/system-ovn.at | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tests/syste
Changes since v1:
- improve ipv6 address lookup
Lorenzo Bianconi (3):
test: Fix false positive in BFD system test.
ovn-nbctl: Fix nbctl_pre_lr_route_add for BFD.
northd: Add BFD support for ECMP route policy.
NEWS | 1 +
northd/northd.c | 86
Add the capbility to mark (through pkt.mark) incoming/outgoing packets
in logical_switch datapath according to user configured QoS rule.
Co-developed-by: Dumitru Ceara
Reported-at: https://issues.redhat.com/browse/FDP-42
Signed-off-by: Lorenzo Bianconi
---
- move qos packet mark action
> Hi Lorenzo,
>
> I have just a few small comments below.
Hi Mark,
thx for the review.
>
> On 1/30/24 09:53, Lorenzo Bianconi wrote:
> > Add the capbility to mark (through pkt.mark) incoming/outgoing packets
> > in logical_switch datapath according to user confi
/20777962659
Signed-off-by: Lorenzo Bianconi
---
Changes since v3:
- fix flaky ovn multinode test
- rebase on top of ovn main branch
Changes since v2:
- fix openshift regression for regular icmp traffic received from a remote node
- add gw router selftest
Changes since v1:
- add fix for vxlan and stt
Similar to OVN static routes, introduce the capability to link a BFD
session for OVN reroute policies.
Reported-at: https://issues.redhat.com/browse/FDP-234
Signed-off-by: Lorenzo Bianconi
---
NEWS | 1 +
northd/northd.c | 71
Fix regression introduce in BFD system tests
Fixes: 1655a6c146ca ("northd, utils: support for RouteTables in LRs")
Signed-off-by: Lorenzo Bianconi
---
tests/system-ovn.at | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tests/system-ovn.at b/tests/system-ov
Lorenzo Bianconi (3):
test: fix false positive in BFD system test
ovn-nbctl: fix nbctl_pre_lr_route_add for BFD
northd: add BFD support for ECMP route policy
NEWS | 1 +
northd/northd.c | 71
ovn-nb.ovsschema | 9
Signed-off-by: Lorenzo Bianconi
---
utilities/ovn-nbctl.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/utilities/ovn-nbctl.c b/utilities/ovn-nbctl.c
index 526369b68..0586eccdb 100644
--- a/utilities/ovn-nbctl.c
+++ b/utilities/ovn-nbctl.c
@@ -4394,6 +4394,7 @@ nbctl_pre_lr_route_add(struct ctl_co
Introduce specif flows for E/W ICMPv{4,6} packets if tunnelled packets
do not fit path MTU. This patch enable PMTUD for East/West Geneve traffic.
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2241711
Signed-off-by: Lorenzo Bianconi
---
Changes since v2:
- fix openshift regression
> On Tue, Jan 9, 2024 at 8:59 AM Lorenzo Bianconi
> wrote:
> >
> > > On Fri, Dec 22, 2023 at 11:27 AM Lorenzo Bianconi
> > > wrote:
> > > >
> > > > Introduce specif flows for E/W ICMPv{4,6} packets if tunnelled packets
> > > >
> On Fri, Dec 22, 2023 at 11:27 AM Lorenzo Bianconi
> wrote:
> >
> > Introduce specif flows for E/W ICMPv{4,6} packets if tunnelled packets
> > do not fit path MTU. This patch enable PMTUD for East/West Geneve traffic.
> >
> > Reported-at: https://bugzill
Introduce a dedicated test for garp-max-timeout knob
Signed-off-by: Lorenzo Bianconi
---
tests/ovn.at | 65
1 file changed, 65 insertions(+)
diff --git a/tests/ovn.at b/tests/ovn.at
index 5615ba1a9..6093e0ed9 100644
--- a/tests/ovn.at
+++ b
> Hi Lorenzo,
Hi Mark,
thx for the review
>
> I have some comments below
>
> On 12/22/23 10:47, Lorenzo Bianconi wrote:
> > Introduce a dedicated test for garp-max-timeout knob
> >
> > Signed-off-by: Lorenzo Bianconi
Introduce specif flows for E/W ICMPv{4,6} packets if tunnelled packets
do not fit path MTU. This patch enable PMTUD for East/West Geneve traffic.
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2241711
Signed-off-by: Lorenzo Bianconi
---
Changes since v2:
- the icmp error forwarding
Introduce a dedicated test for garp-max-timeout knob
Signed-off-by: Lorenzo Bianconi
---
tests/ovn.at | 69
1 file changed, 69 insertions(+)
diff --git a/tests/ovn.at b/tests/ovn.at
index 918f97a9e..deeb2a201 100644
--- a/tests/ovn.at
+++ b
> On 12/15/23 18:01, Lorenzo Bianconi wrote:
> > Introduce a dedicated test for garp-max-timeout knob
> >
>
> For context of the review, I had asked for this "standalone" test for
> garp-max-timeout so that we can backport the following patch to 21.12:
>
>
> On 12/14/23 18:42, Lorenzo Bianconi wrote:
> > When using VLAN backed networks and OVN routers leveraging the
> > 'ovn-chassis-mac-mappings' option for east-west traffic, the eth.src field
> > is
> > replaced by the chassis mac address in order to not expose the
Introduce a dedicated test for garp-max-timeout knob
Signed-off-by: Lorenzo Bianconi
---
tests/ovn.at | 82
1 file changed, 82 insertions(+)
diff --git a/tests/ovn.at b/tests/ovn.at
index 918f97a9e..25b101a7a 100644
--- a/tests/ovn.at
+++ b
in TOR swtich fdb table.
More into about the issue can be found here [0].
[0]
https://mail.openvswitch.org/pipermail/ovs-discuss/2020-September/050678.html
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2087779
Signed-off-by: Lorenzo Bianconi
Acked-by: Ales Musil
Signed-off-by: Mark
Introduce IPv6 tests for pmtu discovery in multinode test
Signed-off-by: Lorenzo Bianconi
---
tests/multinode.at | 16 +++-
1 file changed, 15 insertions(+), 1 deletion(-)
diff --git a/tests/multinode.at b/tests/multinode.at
index a051ea8c2..0f098000c 100644
--- a/tests
in TOR swtich fdb table.
More into about the issue can be found here [0].
[0]
https://mail.openvswitch.org/pipermail/ovs-discuss/2020-September/050678.html
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2087779
Signed-off-by: Lorenzo Bianconi
Acked-by: Ales Musil
Signed-off-by: Mark
in TOR swtich fdb table.
More into about the issue can be found here [0].
[0]
https://mail.openvswitch.org/pipermail/ovs-discuss/2020-September/050678.html
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2087779
Signed-off-by: Lorenzo Bianconi
Acked-by: Ales Musil
Signed-off-by: Mark
in TOR swtich fdb table.
More into about the issue can be found here [0].
[0]
https://mail.openvswitch.org/pipermail/ovs-discuss/2020-September/050678.html
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2087779
Signed-off-by: Lorenzo Bianconi
Acked-by: Ales Musil
Signed-off-by: Mark
in TOR swtich fdb table.
More into about the issue can be found here [0].
[0]
https://mail.openvswitch.org/pipermail/ovs-discuss/2020-September/050678.html
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2087779
Signed-off-by: Lorenzo Bianconi
Acked-by: Ales Musil
Signed-off-by: Mark
in TOR swtich fdb table.
More into about the issue can be found here [0].
[0]
https://mail.openvswitch.org/pipermail/ovs-discuss/2020-September/050678.html
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2087779
Signed-off-by: Lorenzo Bianconi
Acked-by: Ales Musil
Signed-off-by: Mark
in TOR swtich fdb table.
More into about the issue can be found here [0].
[0]
https://mail.openvswitch.org/pipermail/ovs-discuss/2020-September/050678.html
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2087779
Signed-off-by: Lorenzo Bianconi
Acked-by: Ales Musil
Signed-off-by: Mark
in TOR swtich fdb table.
More into about the issue can be found here [0].
[0]
https://mail.openvswitch.org/pipermail/ovs-discuss/2020-September/050678.html
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2087779
Signed-off-by: Lorenzo Bianconi
Acked-by: Ales Musil
Signed-off-by: Mark
Introduce specif flows for E/W ICMPv{4,6} packets if tunnelled packets
do not fit path MTU. This patch enable PMTUD for East/West Geneve traffic.
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2241711
Tested-by: Jaime Ruiz
Signed-off-by: Lorenzo Bianconi
---
Changes since v1:
- add
Introduce specif flows for E/W ICMPv{4,6} packets if tunnelled packets
do not fit path MTU. This patch enable PMTUD for East/West Geneve traffic.
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2241711
Tested-by: Jaime Ruiz
Signed-off-by: Lorenzo Bianconi
---
NEWS
Avoid recirculating IGMP/MLD packets more than one time from stage
ls_out_pre_lb in the egress pipeline to ovn table 37 in order to avoid
packet looping for ovn-ic deployment.
Signed-off-by: Lorenzo Bianconi
---
controller/pinctrl.c | 2 ++
include/ovn/logical-fields.h | 3 +++
lib
Introduce the capability to split multicast group openflow actions
created in consider_mc_group routine in multiple buffers if the
single buffer size is over netlink buffer size limits.
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2232152
Signed-off-by: Lorenzo Bianconi
---
Changes
Introduce the capability to split multicast group openflow actions
created in consider_mc_group routine in multiple buffers if the
single buffer size is over netlink buffer size limits.
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2232152
Signed-off-by: Lorenzo Bianconi
---
Changes
Introduce ls_datapath_group column in the load_balancer table of the SB
sb and deprecate datapath_group one. This patch make the table symmetric
with lr_datapath_group column in the load_balancer table of SB db.
Reviewed-by: Ales Musil
Signed-off-by: Lorenzo Bianconi
---
controller/chassis.c
Introduce lr_datapath_group column in the load_balancer table of the SB
db.
Sync load_balancers applied to logical_routers to Load_Balancer table in
the SouthBound database.
Reviewed-by: Ales Musil
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2193323
Signed-off-by: Lorenzo Bianconi
RFC v1:
- get rid of patch 1/2: northd: rename table datapath_group in
ls_datapath_group in load_balancer sb db table
Lorenzo Bianconi (2):
northd: sync lb applied to logical routers in sb db lb table
northd: introduce ls_datapath_group column in lb sb db table
controller/chassis.c
Introduce the capability to split multicast group openflow actions
created in consider_mc_group routine in multiple buffers if the
single buffer size is over netlink buffer size limits.
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2232152
Signed-off-by: Lorenzo Bianconi
Remove some debugging code in unit-tests
Signed-off-by: Lorenzo Bianconi
---
tests/ovn.at | 8
1 file changed, 8 deletions(-)
diff --git a/tests/ovn.at b/tests/ovn.at
index ba5ce298a..dfe535f36 100644
--- a/tests/ovn.at
+++ b/tests/ovn.at
@@ -4870,7 +4870,6 @@ test_ip() {
shift
r len for
check_pkt_larger action")
Signed-off-by: Lorenzo Bianconi
---
Changes since v1:
- add more unit-tests
---
northd/northd.c | 8 +++-
tests/ovn-northd.at | 17 +
2 files changed, 24 insertions(+), 1 deletion(-)
diff --git a/northd/northd.c b/northd/northd.c
index 3eaa43f07.
r len for
check_pkt_larger action")
Signed-off-by: Lorenzo Bianconi
---
northd/northd.c | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/northd/northd.c b/northd/northd.c
index 3eaa43f07..b041467a3 100644
--- a/northd/northd.c
+++ b/northd/northd.c
@@ -12282,7
in TOR swtich fdb table.
More into about the issue can be found here [0].
[0]
https://mail.openvswitch.org/pipermail/ovs-discuss/2020-September/050678.html
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2087779
Signed-off-by: Lorenzo Bianconi
---
Changes since v3:
- simplify code logic
> On Thu, Aug 24, 2023 at 6:36 PM Lorenzo Bianconi <
> lorenzo.bianc...@redhat.com> wrote:
>
> > When using VLAN backed networks and OVN routers leveraging the
> > 'ovn-chassis-mac-mappings' option for east-west traffic, the eth.src field
> > is
> > repla
the
chassis column of the corresponding port binding in SB DB to the
remote chassis. This process can be implemented in ovn-northd.
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2217930
Signed-off-by: Lorenzo Bianconi
---
Changes since v2:
- address missing logica cases
- add more unit
in TOR swtich fdb table.
More into about the issue can be found here [0].
[0]
https://mail.openvswitch.org/pipermail/ovs-discuss/2020-September/050678.html
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2087779
Signed-off-by: Lorenzo Bianconi
---
Changes since v2:
- cap exponential backoff
the
chassis column of the corresponding port binding in SB DB to the
remote chassis. This process can be implemented in ovn-northd.
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2217930
Signed-off-by: Lorenzo Bianconi
---
Changes since v1:
- add NEWS entry
- do not remove ovn-ic code
> Hi Lorenzo,
Hi Mark,
thx for the fast review.
>
> Please add a NEWS item about this new feature.
ack, I will add it in v2.
>
> I have more down below.
>
> On 7/24/23 11:52, Lorenzo Bianconi wrote:
> > ovn supports creating remote logical ports. An user
&g
the
chassis column of the corresponding port binding in SB DB to the
remote chassis. This process can be authomized in ovn-northd.
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2217930
Signed-off-by: Lorenzo Bianconi
---
ic/ovn-ic.c | 42
in TOR swtich fdb table.
More into about the issue can be found here [0].
[0]
https://mail.openvswitch.org/pipermail/ovs-discuss/2020-September/050678.html
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2087779
Signed-off-by: Lorenzo Bianconi
---
Changes since v1:
- add uni-test
- add
> On Mon, Jun 5, 2023 at 5:54 PM Lorenzo Bianconi
> wrote:
> >
> > When using VLAN backed networks and OVN routers leveraging the
> > 'ovn-chassis-mac-mappings' option for east-west traffic, the eth.src field
> > is
> > replaced by the chassis mac address i
Introduce ls_datapath_group column in the load_balancer table of the SB
sb and deprecate datapath_group one. This patch make the table symmetric
with lr_datapath_group column in the load_balancer table of SB db.
Signed-off-by: Lorenzo Bianconi
---
controller/chassis.c| 8
Introduce lr_datapath_group column in the load_balancer table of the SB
db.
Sync load_balancers applied to logical_routers to Load_Balancer table in
the SouthBound database.
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2193323
Signed-off-by: Lorenzo Bianconi
---
controller
Lorenzo Bianconi (2):
northd: sync lb applied to logical routers in sb db lb table
northd: introduce ls_datapath_group column in lb sb db table
controller/chassis.c| 8 +++
controller/lflow.c | 10 +++
controller/local_data.c | 16 +
controller/ovn-controller.c | 13
> On Mon, Jun 19, 2023 at 4:43 PM Lorenzo Bianconi <
> lorenzo.bianc...@redhat.com> wrote:
>
> > Introduce ls_datapath_group column in the load_balancer table of the SB
> > db and deprecate datapath_group one. This patch make the table symmetric
> &
Ceara
> ---
> controller/pinctrl.c | 33 ++++-----
> 1 file changed, 16 insertions(+), 17 deletions(-)
thx for fixing it.
Acked-by: Lorenzo Bianconi
>
> diff --git a/controller/pinctrl.c b/controller/pinctrl.c
> index 6027ba0afb..bed90fe0b7
[...]
>
> There is a comment above that should be adjusted.
> "Delete any SB load balancer entries that refer to NB load balancers
> that don't exist anymore or are not applied to switches anymore."
> ->
> "Delete any SB load balancer entries that refer to NB load balancers
> that don't exist
Introduce ls_datapath_group column in the load_balancer table of the SB
db and deprecate datapath_group one. This patch make the table symmetric
with lr_datapath_group column in the load_balancer table of SB db.
Signed-off-by: Lorenzo Bianconi
---
controller/chassis.c| 8
Introduce lr_datapath_group column in the load_balancer table of the SB
db.
Sync load_balancers applied to logical_routers to Load_Balancer table in
the SouthBound database.
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2193323
Signed-off-by: Lorenzo Bianconi
---
controller
Changes since RFC v2:
- introduce ls_datapath_group column and deprecate datapath_group one.
Changes since RFC v1:
- get rid of patch 1/2: northd: rename table datapath_group in
ls_datapath_group in load_balancer sb db table
Lorenzo Bianconi (2):
northd: sync lb applied to logical routers
> On 6/14/23 15:17, Lorenzo Bianconi wrote:
> > In the current codebase for distributed gw router port use-case,
> > it is not possible to add a load balancer that redirects the traffic
> > to a back-end if it is used as internal IP of a FIP NAT rule since
> >
packet for the load balancer.
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2023609
Acked-by: Mark Michelson
Reviewed-by: Simon Horman
Signed-off-by: Lorenzo Bianconi
---
Changes since v5:
- rely on fmt_pkt macro in unit-tests
Changes since v4:
- improve memory footprint
Changes
Introduce lr_datapath_group column in the load_balancer table of the SB
db.
Sync load_balancers applied to logical_routers to Load_Balancer table in
the SouthBound database.
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2193323
Signed-off-by: Lorenzo Bianconi
---
Changes since RFC v1
On Jun 09, Mark Michelson wrote:
> On 6/9/23 13:10, Lorenzo Bianconi wrote:
> > > Hi Lorenzo, I have one note below
> > >
> > > On 6/9/23 12:07, Lorenzo Bianconi wrote:
> > > > Introduce lr_datapath_group column in the load_balancer table of the SB
>
> On 6/9/23 18:38, Lorenzo Bianconi wrote:
> >> On 6/9/23 18:07, Lorenzo Bianconi wrote:
> >>> This is a preliminary patch to sync load_balancers applied to
> >>> logical_routers to Load_Balancer table in the SouthBound database.
> >>>
> >>&
> Hi Lorenzo, I have one note below
>
> On 6/9/23 12:07, Lorenzo Bianconi wrote:
> > Introduce lr_datapath_group column in the load_balancer table of the SB
> > db.
> > Sync load_balancers applied to logical_routers to Load_Balancer table in
> > the South
> On 6/9/23 18:07, Lorenzo Bianconi wrote:
> > This is a preliminary patch to sync load_balancers applied to
> > logical_routers to Load_Balancer table in the SouthBound database.
> >
> > Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2193323
>
> The
Introduce lr_datapath_group column in the load_balancer table of the SB
db.
Sync load_balancers applied to logical_routers to Load_Balancer table in
the SouthBound database.
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2193323
Signed-off-by: Lorenzo Bianconi
---
controller
This is a preliminary patch to sync load_balancers applied to
logical_routers to Load_Balancer table in the SouthBound database.
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2193323
Signed-off-by: Lorenzo Bianconi
---
controller/lflow.c | 7 ---
controller/local_data.c
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2193323
Lorenzo Bianconi (2):
northd: rename table datapath_group in ls_datapath_group in
load_balancer sb db table
northd: sync lb applied to logical routers in sb db lb table
controller/lflow.c | 7 ++-
controller
> If iface does not exist when qos I+P is run the first time
> a qos queue is still created and qos I+P will never apply qos to OVS
> qos might still be applied later, if/when (for any reason) runtime_date must
> be recomputed
>
> Fixes: 7d1d111ff213 ("controller: configure qos through ovs qos
t.
Reviewed-by: Lorenzo Bianconi
>
> Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2213219
> Fixes: 7d1d111ff213 ("controller: configure qos through ovs qos table and do
> not run tc directly")
> Signed-off-by: Xavier Simonart
> ---
> controller/bindi
1 - 100 of 1303 matches
Mail list logo