Re: [ovs-dev] [PATCH ovn 1/2] northd: make traffic routed to vtep lport distributed

Please check out the v2: https://patchwork.ozlabs.org/project/ovn/patch/20230121164609.3625347-1-odiv...@gmail.com/ Regards, Vladislav Odintsov > On 21 Jan 2023, at 15:47, Vladislav Odintsov wrote: > > Hi Simon, > > thanks for the review. > I agree with all your comments and I’ll address them

[ovs-dev] [PATCH ovn v2 2/2] northd: fix typo in consider_l3d{wg, gw}... function

This patch fixes typo in consider_l3dgw_port_is_centralized() function. Signed-off-by: Vladislav Odintsov Reviewed-by: Simon Horman --- northd/northd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/northd/northd.c b/northd/northd.c index 46ed39850..bc2d79164 100644 ---

[ovs-dev] [PATCH ovn v2 1/2] northd: make traffic routed to vtep lport distributed

There were two issues prior to this patch: 1. It was unable to have connectivity to networks over a router in physical network connected through VTEP (ramp) gateway. Consider next topology: ovn-nbctl lr-add lr1 ovn-nbctl lrp-add lr1 lrp1 00:00:00:00:00:01 10.0.0.1/24 ovn-nbctl

Re: [ovs-dev] [PATCH ovn v3 4/4] docs: add tutorial for ovsdb relay usage

Bleep bloop. Greetings Vladislav Odintsov, 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 is 84 characters long (recommended limit is 79) #5177 FILE: Documentation/t

Re: [ovs-dev] [PATCH ovn v3 2/4] rhel: add ovn-db@.service systemd-unit

Bleep bloop. Greetings Vladislav Odintsov, 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 is 80 characters long (recommended limit is 79) #155 FILE: rhel/usr_lib_sys

Re: [ovs-dev] [PATCH ovn v3 1/4] rhel, ovn-ctl: add support to run OVN SB relay

Bleep bloop. Greetings Vladislav Odintsov, 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 is 100 characters long (recommended limit is 79) #51 FILE: utilities/ovn-ct

[ovs-dev] [PATCH ovn v3 3/4] vtep: set SB DB ovsdb_idl to no_leader_only mode

This adds capability to run ovn-controller-vtep against OVN Southbound OVSDB server running in relay mode. Signed-off-by: Vladislav Odintsov Reviewed-by: Ales Musil --- controller-vtep/ovn-controller-vtep.c | 1 + 1 file changed, 1 insertion(+) diff --git a/controller-vtep/ovn-controller-vtep.

[ovs-dev] [PATCH ovn v3 1/4] rhel, ovn-ctl: add support to run OVN SB relay

New ovn-ctl commands were added: - start_sb_relay_ovsdb - stop_sb_relay_ovsdb - restart_sb_relay_ovsdb Required configuration parameters are: --db-sb-relay-remote - needed to supply upstream cluster/server remote. Optionally user may pass next parameters: --db-sb-relay-sock --db-sb-relay-pidfile

[ovs-dev] [PATCH ovn v3 2/4] rhel: add ovn-db@.service systemd-unit

With this universal systemd unit users, which run ovsdb-servers with systemd, will be able to run different OVSDB server instances within separate systemd-units. Now it is possible to run all commands from ovn-ctl script, which start and end with "start_/stop_" and "_ovsdb" respectively. Current l

[ovs-dev] [PATCH ovn v3 0/4] Add support for OVSDB Relay in OVN SB DB

v2->v3: - Address Dumitru's comment regarding wrong %systemd_postun rpm macro usage. - Rebase on current main branch. v1->v2: - Added post/preun/postun scripts to manage systemd-related logic around new service unit. This patch series adds support to run utilize OVSDB Relay functionality in O

Re: [ovs-dev] [PATCH ovn 1/2] northd: make traffic routed to vtep lport distributed

Hi Simon, thanks for the review. I agree with all your comments and I’ll address them in v2. Regards, Vladislav Odintsov > On 18 Jan 2023, at 16:08, Simon Horman wrote: > > On Thu, Dec 22, 2022 at 08:43:08PM +0300, Vladislav Odintsov wrote: >> There were two issues prior to this patch: >> 1. I