[ovs-dev] [PATCH ovn] ovn-controller docs: fix typo in ovn-monitor-all description

2023-05-05 Thread Vladislav Odintsov
Make it more clear that ovn-monitor-all option has effect on OVN Southbound database rather than local OVS. Reported-at: https://mail.openvswitch.org/pipermail/ovs-discuss/2023-May/052421.html Reported-by: Ilya Maximets Signed-off-by: Vladislav Odintsov --- controller/ovn-controller.8.xml

Re: [ovs-dev] [PATCH ovn v2] utilities: increase OVSDB inactivity probe interval for ovn-*ctl

2023-05-04 Thread Vladislav Odintsov
Hi Dumitru, Mark, The new version (v4) has been submitted at https://patchwork.ozlabs.org/project/ovn/patch/20230504165510.4026066-1-odiv...@gmail.com/ > On 3 May 2023, at 17:18, Dumitru Ceara wrote: > > On 5/3/23 16:12, Vladislav Odintsov wrote: >> Hi Dumitru and Mark

[ovs-dev] [PATCH ovn v4] utilities: increase OVSDB inactivity probe interval for ovn-*ctl

2023-05-04 Thread Vladislav Odintsov
option was provided, the initial (12 ms) interval is left. Signed-off-by: Vladislav Odintsov --- v3 -> v4: - Rebased on a fresh main branch. v2 -> v3: - Addressed Dumitru's and Mark's suggestion to split ovn-{n,s}bctl configuration option dbctl_probe_interval for nbctl_... and

[ovs-dev] [PATCH ovn v3] utilities: increase OVSDB inactivity probe interval for ovn-*ctl

2023-05-04 Thread Vladislav Odintsov
option was provided, the initial (12 ms) interval is left. Signed-off-by: Vladislav Odintsov --- v2 -> v3: - Addressed Dumitru's and Mark's suggestion to split ovn-{n,s}bctl configuration option dbctl_probe_interval for nbctl_... and sbctl_... . - Added NEWS entry. - Fixes ty

Re: [ovs-dev] [PATCH ovn v2] utilities: increase OVSDB inactivity probe interval for ovn-*ctl

2023-05-03 Thread Vladislav Odintsov
Hi Dumitru and Mark, thanks for the review! > On 3 May 2023, at 16:56, Dumitru Ceara wrote: > > On 5/3/23 15:47, Mark Michelson wrote: >> On 5/3/23 09:03, Dumitru Ceara wrote: >>> Hi Vladislav, >>> >>> On 5/3/23 02:55, Vladislav Odintsov wrot

Re: [ovs-dev] [PATCH ovn] utilities: disable OVSDB inactivity probes for non-daemon ovn-nbctl

2023-05-02 Thread Vladislav Odintsov
1: https://patchwork.ozlabs.org/project/ovn/patch/20230503005531.4005891-1-odiv...@gmail.com/ > On 18 Apr 2023, at 18:31, Vladislav Odintsov wrote: > > > >> On 18 Apr 2023, at 16:59, Dumitru Ceara wrote: >> >> On 4/14/23 15:37, Vladislav Odintsov wrote: >>&g

[ovs-dev] [PATCH ovn v2] utilities: increase OVSDB inactivity probe interval for ovn-*ctl

2023-05-02 Thread Vladislav Odintsov
to configure intervals in OVN Northbound database: OVN_Northbound.NB_Global.options.dbctl_inactivity_probe for ovn-nbctl and ovn-sbctl utilities. If no DB configuration option was provided, the initial (12 ms) interval is left. Signed-off-by: Vladislav Odintsov --- lib/ovn-util.h

Re: [ovs-dev] [PATCH ovn] utilities: disable OVSDB inactivity probes for non-daemon ovn-nbctl

2023-04-18 Thread Vladislav Odintsov
> On 18 Apr 2023, at 16:59, Dumitru Ceara wrote: > > On 4/14/23 15:37, Vladislav Odintsov wrote: >> Hi Dumitru, >> >>> On 13 Apr 2023, at 11:55, Dumitru Ceara wrote: >>> >>> Hi Vladislav, >>> >>> On 4/3/23 12:08, Ilya Maxi

Re: [ovs-dev] [PATCH ovn] utilities: disable OVSDB inactivity probes for non-daemon ovn-nbctl

2023-04-14 Thread Vladislav Odintsov
Hi Dumitru, > On 13 Apr 2023, at 11:55, Dumitru Ceara wrote: > > Hi Vladislav, > > On 4/3/23 12:08, Ilya Maximets wrote: >> On 3/31/23 17:17, Dumitru Ceara wrote: >>> On 3/31/23 16:51, Vladislav Odintsov wrote: >>>> As I understood from Ilya, in case

Re: [ovs-dev] [ovn] ha-chassis-group false positive failover

2023-04-14 Thread Vladislav Odintsov
al_ids:ovn-openflow-probe-interval "60" > On 30 Mar 2023, at 23:19, Vladislav Odintsov wrote: > > Hi all, > > I’m facing a false positive failover in the event of openflow connection got > dropped after inactivity probe timeout due to high load of ovn-controller &

Re: [ovs-dev] [PATCH ovn] utilities: disable OVSDB inactivity probes for non-daemon ovn-nbctl

2023-03-31 Thread Vladislav Odintsov
Ceara wrote: > > On 3/31/23 14:46, Vladislav Odintsov wrote: >> Hi Dumitru, >> >>> On 31 Mar 2023, at 15:01, Dumitru Ceara wrote: >>> >>> On 3/31/23 11:43, Ales Musil wrote: >>>> On Thu, Mar 23, 2023 at 8:25 PM Vladislav Odintsov >>

Re: [ovs-dev] [PATCH ovn] utilities: disable OVSDB inactivity probes for non-daemon ovn-nbctl

2023-03-31 Thread Vladislav Odintsov
Hi Dumitru, > On 31 Mar 2023, at 15:01, Dumitru Ceara wrote: > > On 3/31/23 11:43, Ales Musil wrote: >> On Thu, Mar 23, 2023 at 8:25 PM Vladislav Odintsov >> wrote: >> >>> For large OVN_Southbound DBs defatult interval of 5000 ms could be not >&g

Re: [ovs-dev] [PATCH] ovsdb: add unixctl cmd to show memory-trim-on-compaction setting

2023-03-30 Thread Vladislav Odintsov
> On 21 Mar 2023, at 17:01, Ilya Maximets wrote: > > On 3/20/23 13:51, Vladislav Odintsov wrote: >> No, my usecase with 2.17 is that I just want to check agains a running >> process wether memory compaction is enabled or not without searching >> specific line in

[ovs-dev] [ovn] ha-chassis-group false positive failover

2023-03-30 Thread Vladislav Odintsov
rning on try to release unknown interface, which was relased right before. Is it normal situation or also should be investigated? Thanks. Regards, Vladislav Odintsov ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH ovn 0/6] Inactivity probe configuration for ic, controller-vtep

2023-03-23 Thread Vladislav Odintsov
Hi Numan, thanks for the review and patches applying. I’ve re-submitted the last patch here: https://patchwork.ozlabs.org/project/ovn/patch/20230323192520.3903330-1-odiv...@gmail.com/ > On 23 Mar 2023, at 22:15, Numan Siddique wrote: > > On Sat, Mar 18, 2023 at 7:50 PM Vladislav

[ovs-dev] [PATCH ovn] utilities: disable OVSDB inactivity probes for non-daemon ovn-nbctl

2023-03-23 Thread Vladislav Odintsov
For large OVN_Southbound DBs defatult interval of 5000 ms could be not sufficient. This patch disables OVSDB inactivity probes for ovn-*ctl running in non-daemon mode. Signed-off-by: Vladislav Odintsov --- utilities/ovn-dbctl.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/utilities

Re: [ovs-dev] [PATCH] ovsdb: add unixctl cmd to show memory-trim-on-compaction setting

2023-03-20 Thread Vladislav Odintsov
No, my usecase with 2.17 is that I just want to check agains a running process wether memory compaction is enabled or not without searching specific line in logs, which in addition can be rotated. > On 20 Mar 2023, at 15:27, Ilya Maximets wrote: > > On 3/20/23 13:20, Vladislav Odint

Re: [ovs-dev] [PATCH] ovsdb: add unixctl cmd to show memory-trim-on-compaction setting

2023-03-20 Thread Vladislav Odintsov
. At least in versions were it’s off by default. > On 20 Mar 2023, at 15:07, Ilya Maximets wrote: > > On 3/20/23 10:24, Vladislav Odintsov wrote: >> In commit [1] there was added support to enable memory trimming on OVSDB tlog >> compation. However there was no option to get

[ovs-dev] [PATCH] ovsdb: add unixctl cmd to show memory-trim-on-compaction setting

2023-03-20 Thread Vladislav Odintsov
://github.com/openvswitch/ovs/commit/f38f98a2c0dd7fcaf20fbe11d1e67a9b2afc0b2a Signed-off-by: Vladislav Odintsov --- NEWS | 4 ovsdb/ovsdb-server.c | 28 2 files changed, 32 insertions(+) diff --git a/NEWS b/NEWS index 72b9024e6..8515f4aaa 100644

[ovs-dev] [PATCH ovn 7/6] utilities: disable OVSDB inactivity probes for non-daemon ovn-nbctl

2023-03-18 Thread Vladislav Odintsov
For large OVN_Southbound DBs default interval of 5000 ms could be not sufficient. This patch disables OVSDB inactivity probes for ovn-{n,s}ctl running in non-daemon mode. Signed-off-by: Vladislav Odintsov --- utilities/ovn-dbctl.c | 3 +++ 1 file changed, 3 insertions(+) diff --git

Re: [ovs-dev] [PATCH ovn 1/6] controller, northd: move inactivity probe configuration to lib/

2023-03-18 Thread Vladislav Odintsov
Signed-off-by: Vladislav Odintsov > On 19 Mar 2023, at 02:50, Vladislav Odintsov wrote: > > ovn-northd and ovn-controller had similar code to configure inactivity > probe interval. This patch moves common logic to lib/ovn-util module. > --- > controller/ovn-controller.c |

[ovs-dev] [PATCH ovn 4/6] controller-vtep: add unixctl commands to show DB connection status

2023-03-18 Thread Vladislav Odintsov
New unixctl commands were added: - sb-connection-status - vtep-connection-status Signed-off-by: Vladislav Odintsov --- controller-vtep/ovn-controller-vtep.c | 5 + 1 file changed, 5 insertions(+) diff --git a/controller-vtep/ovn-controller-vtep.c b/controller-vtep/ovn-controller-vtep.c

[ovs-dev] [PATCH ovn 6/6] NEWS: add info about ic/controller-vtep inactivity probe support

2023-03-18 Thread Vladislav Odintsov
Signed-off-by: Vladislav Odintsov --- NEWS | 2 ++ 1 file changed, 2 insertions(+) diff --git a/NEWS b/NEWS index 637adcff3..fc1b68324 100644 --- a/NEWS +++ b/NEWS @@ -2,6 +2,8 @@ Post v23.03.0 - - Enhance LSP.options:arp_proxy to support IPv6, configurable MAC addresses

[ovs-dev] [PATCH ovn 5/6] ovn-ic: add unixctl commands to report db connection status

2023-03-18 Thread Vladislav Odintsov
New unixctl commands were added: - nb-connection-status - sb-connection-status - ic-nb-connection-status - ic-sb-connection-status Signed-off-by: Vladislav Odintsov --- ic/ovn-ic.c | 9 + 1 file changed, 9 insertions(+) diff --git a/ic/ovn-ic.c b/ic/ovn-ic.c index feb100859..6f31037ec

[ovs-dev] [PATCH ovn 3/6] controller-vtep: add support to configure inactivity probe interval

2023-03-18 Thread Vladislav Odintsov
This patch adds new configuration option for ovn-controller-vtep service to supply OVSDB (only OVN_Southbound DB) probe interval value. It can be configured through Hardware_Vtep database, Global table, column other_config:ovn-remote-probe-interval. Signed-off-by: Vladislav Odintsov

[ovs-dev] [PATCH ovn 2/6] ic: add support to configure ovsdb inactivity probe

2023-03-18 Thread Vladislav Odintsov
-by: Vladislav Odintsov --- ic/ovn-ic.c | 28 ovn-ic-nb.xml | 14 ++ ovn-nb.xml| 13 + 3 files changed, 55 insertions(+) diff --git a/ic/ovn-ic.c b/ic/ovn-ic.c index 1d0a062f6..feb100859 100644 --- a/ic/ovn-ic.c +++ b/ic/ovn-ic.c @@ -35,6 +35,7

[ovs-dev] [PATCH ovn 1/6] controller, northd: move inactivity probe configuration to lib/

2023-03-18 Thread Vladislav Odintsov
ovn-northd and ovn-controller had similar code to configure inactivity probe interval. This patch moves common logic to lib/ovn-util module. --- controller/ovn-controller.c | 8 ++-- lib/ovn-util.c | 22 ++ lib/ovn-util.h | 4

[ovs-dev] [PATCH ovn 0/6] Inactivity probe configuration for ic, controller-vtep

2023-03-18 Thread Vladislav Odintsov
added to ovn-ic and ovn-controller-vtep. Vladislav Odintsov (6): controller,northd: move inactivity probe configuration to lib/ ic: add support to configure ovsdb inactivity probe controller-vtep: add support to configure inactivity probe interval controller-vtep: add unixctl commands

Re: [ovs-dev] [PATCH ovn] rhel: pass options to stop daemon command in systemd units

2023-03-08 Thread Vladislav Odintsov
Thanks Dumitru! > On 8 Mar 2023, at 15:45, Dumitru Ceara wrote: > > On 3/7/23 10:19, Ales Musil wrote: >> On Mon, Feb 27, 2023 at 7:41 PM Vladislav Odintsov >> wrote: >> >>> This patch fixes ovn-northd.service and ovn-db@.service systemd units >>>

[ovs-dev] [PATCH ovn] rhel: pass options to stop daemon command in systemd units

2023-02-27 Thread Vladislav Odintsov
/ ovn-db@sb or manually with ovn-ctl, so ovsdb-server processes for these services will be unexpectedly stopped. Fixes: 497ec3fdfbdb ("rhel: add ovn-db@.service systemd-unit") Signed-off-by: Vladislav Odintsov --- rhel/usr_lib_systemd_system_ovn-db@.service|

Re: [ovs-dev] [PATCH v3] utilities: add support to set umask in ovs-ctl

2023-02-20 Thread Vladislav Odintsov
Thanks, Ilya! > On 20 Feb 2023, at 22:21, Ilya Maximets wrote: > > On 2/15/23 22:20, Vladislav Odintsov wrote: >> Hi Ilya, >> >> Thats my bad - they were the initial names of options, which I renamed later >> and missed this place before sending a patch. >

Re: [ovs-dev] [PATCH v3] utilities: add support to set umask in ovs-ctl

2023-02-15 Thread Vladislav Odintsov
Hi Ilya, Thats my bad - they were the initial names of options, which I renamed later and missed this place before sending a patch. I’m absolutely fine with proposed change. Please fold it while applying the patch. Thanks. regards, Vladislav Odintsov > On 15 Feb 2023, at 22:25, Ilya Maxim

Re: [ovs-dev] [PATCH v2] utilities: add support to set umask in ovs-ctl

2023-02-10 Thread Vladislav Odintsov
Thanks Eelco, requested changes addressed here: https://patchwork.ozlabs.org/project/openvswitch/patch/20230210160229.3689298-1-odiv...@gmail.com/ > On 10 Feb 2023, at 11:21, Eelco Chaudron wrote: > > > > On 8 Feb 2023, at 16:22, Vladislav Odintsov wrote: > >> Thi

[ovs-dev] [PATCH v3] utilities: add support to set umask in ovs-ctl

2023-02-10 Thread Vladislav Odintsov
tchd-umask=0002" will set umask to ovs-vswitchd daemon. Previous behaviour (not setting umask) is left as default. Reported-at: https://mail.openvswitch.org/pipermail/ovs-dev/2023-January/401501.html Signed-off-by: Vladislav Odintsov --- v2 -> v3: - addressed Eelco's review comments. v

Re: [ovs-dev] [PATCH] utilities: add support to set umask in ovs-ctl

2023-02-08 Thread Vladislav Odintsov
ote: > > > > On 7 Feb 2023, at 16:04, Vladislav Odintsov wrote: > >> Thanks for the review, Eelco! >> Answers are inline. >> >>> On 7 Feb 2023, at 17:02, Eelco Chaudron wrote: >>> >>> See some comments inline below. >>> >>

[ovs-dev] [PATCH v2] utilities: add support to set umask in ovs-ctl

2023-02-08 Thread Vladislav Odintsov
tchd-umask=0002" will set umask to ovs-vswitchd daemon. Previous behaviour (not setting umask) is left as default. Reported-at: https://mail.openvswitch.org/pipermail/ovs-dev/2023-January/401501.html Signed-off-by: Vladislav Odintsov --- v1 -> v2: - added item in NEWS file as Ilya's su

Re: [ovs-dev] [PATCH] utilities: add support to set umask in ovs-ctl

2023-02-07 Thread Vladislav Odintsov
Thanks for the review, Eelco! Answers are inline. > On 7 Feb 2023, at 17:02, Eelco Chaudron wrote: > > See some comments inline below. > > Cheers, > > Eelco > > On 27 Jan 2023, at 14:29, Vladislav Odintsov wrote: > >> This patch adds a new ovs-ctl option

Re: [ovs-dev] [PATCH] utilities: add support to set umask in ovs-ctl

2023-02-07 Thread Vladislav Odintsov
Hi, sure, no problem. Should I do in a separate patch or incorporate into this one? > On 7 Feb 2023, at 15:35, Ilya Maximets wrote: > > On 1/27/23 14:29, Vladislav Odintsov wrote: >> This patch adds a new ovs-ctl option to pass umask configuration to allow >> OVS dae

Re: [ovs-dev] Wrong unix socket file permissions

2023-01-27 Thread Vladislav Odintsov
I’ve sent a patch [0], please take a look on it when you have time. 0: https://patchwork.ozlabs.org/project/openvswitch/patch/20230127132950.3670502-1-odiv...@gmail.com/ > On 27 Jan 2023, at 13:14, Ilya Maximets wrote: > > On 1/27/23 10:25, Vladislav Odintsov wrote: >> What do

[ovs-dev] [PATCH] utilities: add support to set umask in ovs-ctl

2023-01-27 Thread Vladislav Odintsov
to ovs sockets from that user. Previous behaviour (not setting umask) is left as default. Reported-at: https://mail.openvswitch.org/pipermail/ovs-dev/2023-January/401501.html Signed-off-by: Vladislav Odintsov --- utilities/ovs-ctl.in | 8 1 file changed, 8 insertions(+) diff --git

Re: [ovs-dev] Wrong unix socket file permissions

2023-01-27 Thread Vladislav Odintsov
/utilities/ovs-ctl.in @@ -544,6 +544,7 @@ do done case $command in start) +umask 0002 start_ovsdb || exit 1 start_forwarding || exit 1 add_managers > On 26 Jan 2023, at 14:30, Ilya Maximets wrote: > > On 1/25/23 17:35, Vladislav Odint

Re: [ovs-dev] Wrong unix socket file permissions

2023-01-25 Thread Vladislav Odintsov
Regards, Vladislav Odintsov > On 25 Jan 2023, at 19:16, Ilya Maximets wrote: > > On 1/25/23 16:15, Vladislav Odintsov wrote: >> Hi, >> >> I’m a bit confused about linux unix socket files permissions, created by ovs >> daemons. >> >> From sour

[ovs-dev] Wrong unix socket file permissions

2023-01-25 Thread Vladislav Odintsov
-unix.c#L264 Regards, Vladislav Odintsov ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [OVN RFC 0/7] OVN IC bugfixes & proposals/questions

2023-01-24 Thread Vladislav Odintsov
Thanks Ilya for the quick and useful response! We’ll dig into monitor/db_change_aware logic. Regards, Vladislav Odintsov > On 24 Jan 2023, at 17:00, Ilya Maximets wrote: > > On 1/24/23 14:12, Vladislav Odintsov wrote: >> Hi Ilya, >> >> could you please take a loo

Re: [ovs-dev] [OVN RFC 0/7] OVN IC bugfixes & proposals/questions

2023-01-24 Thread Vladislav Odintsov
Hi Ilya, could you please take a look on this? Maybe you can advice any direction how to investigate this issue? Thanks in advance. Regards, Vladislav Odintsov > On 24 Nov 2022, at 21:10, Anton Vazhnetsov wrote: > > Hi, Terry! > > In continuation to our yesterday’s conversati

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

2023-01-23 Thread Vladislav Odintsov
Thanks Dumitru! Regards, Vladislav Odintsov > On 23 Jan 2023, at 17:58, Dumitru Ceara wrote: > > On 1/23/23 10:15, Simon Horman wrote: >> On Sat, Jan 21, 2023 at 07:46:07PM +0300, Vladislav Odintsov wrote: >>> There were two issues prior to this patch: >>> 1. I

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

2023-01-23 Thread Vladislav Odintsov
Thanks Dumitru! Regards, Vladislav Odintsov > On 23 Jan 2023, at 17:59, Dumitru Ceara wrote: > > On 1/21/23 15:54, Vladislav Odintsov wrote: >> v2->v3: >> - Address Dumitru's comment regarding wrong %systemd_postun rpm macro >> usage. >> - Rebase o

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

2023-01-21 Thread Vladislav Odintsov
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

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

2023-01-21 Thread Vladislav Odintsov
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

2023-01-21 Thread Vladislav Odintsov
s patch also updates relevant testcases to check the changed flows generation and port_binding:options:always-redirect logic. Signed-off-by: Vladislav Odintsov --- northd/northd.c | 26 -- tests/ovn-northd.at | 26 +- 2 files changed, 49 inse

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

2023-01-21 Thread Vladislav Odintsov
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

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

2023-01-21 Thread Vladislav Odintsov
--db-sb-relay-ctrl-sock --db-sb-relay-use-remote-in-db --ovn-sb-relay-db-ssl-key --ovn-sb-relay-db-ssl-cert --ovn-sb-relay-db-ssl-ca-cert Signed-off-by: Vladislav Odintsov Reviewed-by: Ales Musil --- NEWS | 1 + utilities/ovn-ctl | 56

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

2023-01-21 Thread Vladislav Odintsov
anular process management. For instance, it is not needed to restart SB and NB ovsdb-servers anymore, when user just restarts ovn-northd daemon (upgrade scenario). Signed-off-by: Vladislav Odintsov Reviewed-by: Ales Musil --- NEWS| 2 ++ rhel/automake.mk

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

2023-01-21 Thread Vladislav Odintsov
nality in OVN: - add new commands to ovn-ctl script - add new systemd-unit ovn-db@.service to manage ovsdb-servers - add support to run against non-leader to ovn-controller-vtep - add documentation about OVSDB Relay usage in OVN Vladislav Odintsov (4): rhel,ovn-ctl: add support to run OVN SB relay

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

2023-01-21 Thread Vladislav Odintsov
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 t

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

2023-01-20 Thread Vladislav Odintsov
Hi Dumitru, regards, Vladislav Odintsov > On 20 Jan 2023, at 22:51, Dumitru Ceara wrote: > > On 1/20/23 10:51, Ales Musil wrote: >> On Sat, Jan 14, 2023 at 10:28 AM Vladislav Odintsov >> wrote: >> >>> With this universal systemd unit users, whic

Re: [ovs-dev] [PATCH ovn] controller-vtep: monitor only needed SB tables

2023-01-17 Thread Vladislav Odintsov
Hi Dumitru, yes, it was based on my OVSDB Relay support series, I thought it would be accepted first :) Thanks! Regards, Vladislav Odintsov > On 17 Jan 2023, at 19:11, Dumitru Ceara wrote: > > On 1/13/23 21:03, Vladislav Odintsov wrote: >> Signed-off-by: Vla

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

2023-01-14 Thread Vladislav Odintsov
This series was superseded by a new v2: https://patchwork.ozlabs.org/project/ovn/cover/20230114092808.3538014-1-odiv...@gmail.com/ Regards, Vladislav Odintsov > On 6 Jan 2023, at 02:33, Vladislav Odintsov wrote: > > This patch series adds support to run utilize OVSDB Relay func

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

2023-01-14 Thread Vladislav Odintsov
--db-sb-relay-ctrl-sock --db-sb-relay-use-remote-in-db --ovn-sb-relay-db-ssl-key --ovn-sb-relay-db-ssl-cert --ovn-sb-relay-db-ssl-ca-cert Signed-off-by: Vladislav Odintsov --- NEWS | 1 + utilities/ovn-ctl | 56 ++- 2 files changed, 56

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

2023-01-14 Thread Vladislav Odintsov
anular process management. For instance, it is not needed to restart SB and NB ovsdb-servers anymore, when user just restarts ovn-northd daemon (upgrade scenario). Signed-off-by: Vladislav Odintsov --- NEWS| 2 ++ rhel/automake.mk

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

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

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

2023-01-14 Thread Vladislav Odintsov
ort to run against non-leader to ovn-controller-vtep - add documentation about OVSDB Relay usage in OVN Vladislav Odintsov (4): rhel,ovn-ctl: add support to run OVN SB relay rhel: add ovn-db@.service systemd-unit vtep: set SB DB ovsdb_idl to no_leader_only mode docs: add tutorial for ovsdb re

[ovs-dev] [PATCH ovn] controller-vtep: monitor only needed SB tables

2023-01-13 Thread Vladislav Odintsov
Signed-off-by: Vladislav Odintsov --- controller-vtep/ovn-controller-vtep.c | 33 ++- 1 file changed, 32 insertions(+), 1 deletion(-) diff --git a/controller-vtep/ovn-controller-vtep.c b/controller-vtep/ovn-controller-vtep.c index df20816ff..26f8fd115 100644

Re: [ovs-dev] [PATCH ovn branch-21.06] tests: Adapt "conntrack nat implies conntrack" test for branch 21.06.

2023-01-11 Thread Vladislav Odintsov
Thanks Dumitru for the patch. It looks good to me. Acked-by: Vladislav Odintsov Regards, Vladislav Odintsov > On 11 Jan 2023, at 17:50, Dumitru Ceara wrote: > > Commit 384a7c6237da ("northd: Refactor Logical Flows for routers with > DNAT/Load Balancers") chan

[ovs-dev] [PATCH ovn branch-20.03 3/3] ovn-ic: Only monitor useful tables and columns.

2023-01-10 Thread Vladislav Odintsov
From: Dumitru Ceara Signed-off-by: Dumitru Ceara Signed-off-by: Mark Michelson Acked-by: Mark Michelson (cherry picked from commit 2698e7811b41b00ac3b6544430a95be9ae700e23) Signed-off-by: Vladislav Odintsov --- ic/ovn-ic.c | 105 +--- 1 file

[ovs-dev] [PATCH ovn branch-20.03 2/3] Fix compilation issue in fedora 37/rawhide.

2023-01-10 Thread Vladislav Odintsov
.html#open This patch fixes this issue. Acked-by: Dumitru Ceara Signed-off-by: Numan Siddique (cherry picked from commit 500982b84280fdc451877c76f5fdb9a0ac19e805) Signed-off-by: Vladislav Odintsov --- Documentation/conf.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Doc

[ovs-dev] [PATCH ovn branch-20.03 1/3] ci: Install wheel before installing any other python packages.

2023-01-10 Thread Vladislav Odintsov
from commit 889037e2905447a601fab8f78e6ece46c5d45773) Signed-off-by: Vladislav Odintsov --- .ci/linux-prepare.sh | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/.ci/linux-prepare.sh b/.ci/linux-prepare.sh index 37e8d4250..e0c528479 100755 --- a/.ci/linux-prepare.sh +++

[ovs-dev] [PATCH ovn branch-20.06 3/3] ovn-ic: Only monitor useful tables and columns.

2023-01-10 Thread Vladislav Odintsov
From: Dumitru Ceara Signed-off-by: Dumitru Ceara Signed-off-by: Mark Michelson Acked-by: Mark Michelson (cherry picked from commit 2698e7811b41b00ac3b6544430a95be9ae700e23) Signed-off-by: Vladislav Odintsov --- ic/ovn-ic.c | 106 +--- 1 file

[ovs-dev] [PATCH ovn branch-20.06 1/3] ci: Install wheel before installing any other python packages.

2023-01-10 Thread Vladislav Odintsov
from commit 889037e2905447a601fab8f78e6ece46c5d45773) Signed-off-by: Vladislav Odintsov --- .ci/linux-prepare.sh | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/.ci/linux-prepare.sh b/.ci/linux-prepare.sh index 37e8d4250..e0c528479 100755 --- a/.ci/linux-prepare.sh +++

[ovs-dev] [PATCH ovn branch-20.06 2/3] Fix compilation issue in fedora 37/rawhide.

2023-01-10 Thread Vladislav Odintsov
.html#open This patch fixes this issue. Acked-by: Dumitru Ceara Signed-off-by: Numan Siddique (cherry picked from commit 500982b84280fdc451877c76f5fdb9a0ac19e805) Signed-off-by: Vladislav Odintsov --- Documentation/conf.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Doc

[ovs-dev] [PATCH ovn branch-20.09 4/4] ovn-ic: Only monitor useful tables and columns.

2023-01-10 Thread Vladislav Odintsov
From: Dumitru Ceara Signed-off-by: Dumitru Ceara Signed-off-by: Mark Michelson Acked-by: Mark Michelson (cherry picked from commit 2698e7811b41b00ac3b6544430a95be9ae700e23) Signed-off-by: Vladislav Odintsov --- ic/ovn-ic.c | 108 +--- 1 file

[ovs-dev] [PATCH ovn branch-20.09 3/4] Fix compilation issue in fedora 37/rawhide.

2023-01-10 Thread Vladislav Odintsov
.html#open This patch fixes this issue. Acked-by: Dumitru Ceara Signed-off-by: Numan Siddique (cherry picked from commit 500982b84280fdc451877c76f5fdb9a0ac19e805) Signed-off-by: Vladislav Odintsov --- Documentation/conf.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Doc

[ovs-dev] [PATCH ovn branch-20.09 2/4] ci: Install wheel before installing any other python packages.

2023-01-10 Thread Vladislav Odintsov
from commit 889037e2905447a601fab8f78e6ece46c5d45773) Signed-off-by: Vladislav Odintsov --- .ci/linux-prepare.sh | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/.ci/linux-prepare.sh b/.ci/linux-prepare.sh index 37e8d4250..e0c528479 100755 --- a/.ci/linux-prepare.sh +++

[ovs-dev] [PATCH ovn branch-20.09 1/4] ci: Make linux-prepare trust system installs.

2023-01-10 Thread Vladislav Odintsov
4bd37558104b5d8376c52d5d11ced67a18d51aa4) (cherry picked from commit 13770dba7910fec97c17e69f9155e0d75209c072) Signed-off-by: Vladislav Odintsov --- .ci/linux-prepare.sh | 1 - 1 file changed, 1 deletion(-) diff --git a/.ci/linux-prepare.sh b/.ci/linux-prepare.sh index 83ad3958b..37e8d4250 100755

[ovs-dev] [PATCH ovn branch-20.12 2/4] Fix compilation issue in fedora 37/rawhide.

2023-01-10 Thread Vladislav Odintsov
.html#open This patch fixes this issue. Acked-by: Dumitru Ceara Signed-off-by: Numan Siddique (cherry picked from commit 500982b84280fdc451877c76f5fdb9a0ac19e805) Signed-off-by: Vladislav Odintsov --- Documentation/conf.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Doc

[ovs-dev] [PATCH ovn branch-20.12 4/4] ovn-ic: Only monitor useful tables and columns.

2023-01-10 Thread Vladislav Odintsov
From: Dumitru Ceara Signed-off-by: Dumitru Ceara Signed-off-by: Mark Michelson Acked-by: Mark Michelson (cherry picked from commit 2698e7811b41b00ac3b6544430a95be9ae700e23) Signed-off-by: Vladislav Odintsov --- ic/ovn-ic.c | 108 +--- 1 file

[ovs-dev] [PATCH ovn branch-20.12 3/4] ci: fix build: pin OVS to version 2.17

2023-01-10 Thread Vladislav Odintsov
Signed-off-by: Vladislav Odintsov --- .ci/linux-build.sh | 1 + .ci/osx-build.sh | 1 + 2 files changed, 2 insertions(+) diff --git a/.ci/linux-build.sh b/.ci/linux-build.sh index 0e9f87fa8..4217b4365 100755 --- a/.ci/linux-build.sh +++ b/.ci/linux-build.sh @@ -11,6 +11,7 @@ function

[ovs-dev] [PATCH ovn branch-20.12 1/4] ci: Install wheel before installing any other python packages.

2023-01-10 Thread Vladislav Odintsov
from commit 889037e2905447a601fab8f78e6ece46c5d45773) Signed-off-by: Vladislav Odintsov --- .ci/linux-prepare.sh | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/.ci/linux-prepare.sh b/.ci/linux-prepare.sh index 37e8d4250..e0c528479 100755 --- a/.ci/linux-prepare.sh +++

[ovs-dev] [PATCH ovn branch-21.03 3/3] ovn-ic: Only monitor useful tables and columns.

2023-01-10 Thread Vladislav Odintsov
From: Dumitru Ceara Signed-off-by: Dumitru Ceara Signed-off-by: Mark Michelson Acked-by: Mark Michelson (cherry picked from commit 2698e7811b41b00ac3b6544430a95be9ae700e23) Signed-off-by: Vladislav Odintsov --- ic/ovn-ic.c | 108 +--- 1 file

[ovs-dev] [PATCH ovn branch-21.03 1/3] ci: Install wheel before installing any other python packages.

2023-01-10 Thread Vladislav Odintsov
from commit 889037e2905447a601fab8f78e6ece46c5d45773) Signed-off-by: Vladislav Odintsov --- .ci/linux-prepare.sh | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/.ci/linux-prepare.sh b/.ci/linux-prepare.sh index 37e8d4250..e0c528479 100755 --- a/.ci/linux-prepare.sh +++

[ovs-dev] [PATCH ovn branch-21.03 2/3] Fix compilation issue in fedora 37/rawhide.

2023-01-10 Thread Vladislav Odintsov
.html#open This patch fixes this issue. Acked-by: Dumitru Ceara Signed-off-by: Numan Siddique (cherry picked from commit 500982b84280fdc451877c76f5fdb9a0ac19e805) Signed-off-by: Vladislav Odintsov --- Documentation/conf.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Doc

[ovs-dev] [PATCH ovn branch-21.06 1/3] ci: Install wheel before installing any other python packages.

2023-01-10 Thread Vladislav Odintsov
from commit 889037e2905447a601fab8f78e6ece46c5d45773) Signed-off-by: Vladislav Odintsov --- .ci/linux-prepare.sh | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/.ci/linux-prepare.sh b/.ci/linux-prepare.sh index 37e8d4250..e0c528479 100755 --- a/.ci/linux-prepare.sh +++

[ovs-dev] [PATCH ovn branch-21.06 3/3] ovn-ic: Only monitor useful tables and columns.

2023-01-10 Thread Vladislav Odintsov
From: Dumitru Ceara Signed-off-by: Dumitru Ceara Signed-off-by: Mark Michelson Acked-by: Mark Michelson (cherry picked from commit 2698e7811b41b00ac3b6544430a95be9ae700e23) Signed-off-by: Vladislav Odintsov --- ic/ovn-ic.c | 108 +--- 1 file

[ovs-dev] [PATCH ovn branch-21.06 2/3] Fix compilation issue in fedora 37/rawhide.

2023-01-10 Thread Vladislav Odintsov
.html#open This patch fixes this issue. Acked-by: Dumitru Ceara Signed-off-by: Numan Siddique (cherry picked from commit 500982b84280fdc451877c76f5fdb9a0ac19e805) Signed-off-by: Vladislav Odintsov --- Documentation/conf.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Doc

[ovs-dev] [PATCH ovn branch-21.09 3/3] ovn-ic: Only monitor useful tables and columns.

2023-01-10 Thread Vladislav Odintsov
From: Dumitru Ceara Signed-off-by: Dumitru Ceara Signed-off-by: Mark Michelson Acked-by: Mark Michelson (cherry picked from commit 2698e7811b41b00ac3b6544430a95be9ae700e23) Signed-off-by: Vladislav Odintsov --- ic/ovn-ic.c | 108 +--- 1 file

[ovs-dev] [PATCH ovn branch-21.09 2/3] Fix compilation issue in fedora 37/rawhide.

2023-01-10 Thread Vladislav Odintsov
.html#open This patch fixes this issue. Acked-by: Dumitru Ceara Signed-off-by: Numan Siddique (cherry picked from commit 500982b84280fdc451877c76f5fdb9a0ac19e805) Signed-off-by: Vladislav Odintsov --- Documentation/conf.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Doc

[ovs-dev] [PATCH ovn branch-21.09 1/3] ci: Install wheel before installing any other python packages.

2023-01-10 Thread Vladislav Odintsov
from commit 889037e2905447a601fab8f78e6ece46c5d45773) Signed-off-by: Vladislav Odintsov --- .ci/linux-prepare.sh | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/.ci/linux-prepare.sh b/.ci/linux-prepare.sh index 37e8d4250..e0c528479 100755 --- a/.ci/linux-prepare.sh +++

[ovs-dev] [PATCH ovn branch-21.12 1/2] system-ovn: fix CoPP test failures

2023-01-10 Thread Vladislav Odintsov
id=2073060 Signed-off-by: Lorenzo Bianconi Acked-by: Dumitru Ceara Signed-off-by: Numan Siddique (cherry picked from commit e922c05dbb0c0ca70c98bd0b24833248ff5d458b) Signed-off-by: Vladislav Odintsov --- tests/system-ovn.at | 24 1 file changed, 12 insertions(+), 12

[ovs-dev] [PATCH ovn branch-21.12 2/2] ovn-ic: Only monitor useful tables and columns.

2023-01-10 Thread Vladislav Odintsov
From: Dumitru Ceara Signed-off-by: Dumitru Ceara Signed-off-by: Mark Michelson Acked-by: Mark Michelson (cherry picked from commit 2698e7811b41b00ac3b6544430a95be9ae700e23) Signed-off-by: Vladislav Odintsov --- ic/ovn-ic.c | 110 +--- 1 file

[ovs-dev] [PATCH ovn branch-22.03] ovn-ic: Only monitor useful tables and columns.

2023-01-10 Thread Vladislav Odintsov
From: Dumitru Ceara Signed-off-by: Dumitru Ceara Signed-off-by: Mark Michelson Acked-by: Mark Michelson (cherry picked from commit 2698e7811b41b00ac3b6544430a95be9ae700e23) Signed-off-by: Vladislav Odintsov --- ic/ovn-ic.c | 110 +--- 1 file

[ovs-dev] [PATCH ovn branch-22.06] ovn-ic: Only monitor useful tables and columns.

2023-01-10 Thread Vladislav Odintsov
From: Dumitru Ceara Signed-off-by: Dumitru Ceara Signed-off-by: Mark Michelson Acked-by: Mark Michelson (cherry picked from commit 2698e7811b41b00ac3b6544430a95be9ae700e23) Signed-off-by: Vladislav Odintsov --- ic/ovn-ic.c | 110 +--- 1 file

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

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

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

2023-01-05 Thread Vladislav Odintsov
anular process management. For instance, it is not needed to restart SB and NB ovsdb-servers anymore, when user just restarts ovn-northd daemon (upgrade scenario). Signed-off-by: Vladislav Odintsov --- NEWS| 2 ++ rhel/automake.mk

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

2023-01-05 Thread Vladislav Odintsov
--db-sb-relay-ctrl-sock --db-sb-relay-use-remote-in-db --ovn-sb-relay-db-ssl-key --ovn-sb-relay-db-ssl-cert --ovn-sb-relay-db-ssl-ca-cert Signed-off-by: Vladislav Odintsov --- NEWS | 1 + utilities/ovn-ctl | 56 ++- 2 files changed, 56

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

2023-01-05 Thread Vladislav Odintsov
Vladislav Odintsov (4): rhel,ovn-ctl: add support to run OVN SB relay rhel: add ovn-db@.service systemd-unit vtep: set SB DB ovsdb_idl to no_leader_only mode docs: add tutorial for ovsdb relay usage Documentation/automake.mk | 4 + .../tutorials/images/ovsdb-relay-1.png

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

2022-12-22 Thread Vladislav Odintsov
l3gw chassis, now in 'to vtep lport' direction goes directly from hypervisor handling lport to VTEP (RAMP) switch. In the opposite direction traffic still goes from VTEP (RAMP) switch through l3gw chassis and then to hypervisor. Signed-off-by: Vladislav Odintsov --- northd/north

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

2022-12-22 Thread Vladislav Odintsov
Signed-off-by: Vladislav Odintsov --- northd/northd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/northd/northd.c b/northd/northd.c index 07fb0ab9a..9ee9b590d 100644 --- a/northd/northd.c +++ b/northd/northd.c @@ -11612,7 +11612,7 @@ build_gateway_mtu_flow(struct hmap

Re: [ovs-dev] [PATCH ovn v3 0/5] OVN IC multiple same routes fixes

2022-12-16 Thread Vladislav Odintsov
Thanks Dumitru! Regards, Vladislav Odintsov > On 16 Dec 2022, at 15:17, Dumitru Ceara wrote: > > Hi Vladislav, > > I pushed the series to main branch and branch-22.12. I also backported > the first 4 patches all the way down to branch-22.03. > > Thanks a lot for the

Re: [ovs-dev] [PATCH ovn v3 0/5] OVN IC multiple same routes fixes

2022-12-16 Thread Vladislav Odintsov
Hi Dumitru, Regards, Vladislav Odintsov > On 16 Dec 2022, at 02:16, Dumitru Ceara wrote: > > On 12/15/22 22:30, Vladislav Odintsov wrote: >> Thanks Numan for the review. >> >> It seems that for some reason appeared a memory leak, which constantly >> repr

Re: [ovs-dev] [PATCH ovn v3 0/5] OVN IC multiple same routes fixes

2022-12-15 Thread Vladislav Odintsov
Thanks Numan for the review. It seems that for some reason appeared a memory leak, which constantly reproduces. I’ll give it a time tomorrow to find the source of a problem. Regards, Vladislav Odintsov > On 16 Dec 2022, at 00:21, Numan Siddique wrote: > > On Thu, Dec 15, 2022 at

<    1   2   3   4   5   >