7.98529-1-odiv...@gmail.com/
were already sent to list separately, but other changes are based on them
so they're included.
Once those patches are accepts, I can drop them from this series.
- Added NEWS item.
- Added myself to authors list.
Vladislav Odintsov (4):
ic: process o
tcpdump is used in tests. When RPM package is built it
appropriate test fails if tcpdump package is not installed.
This commit fixes this issue by adding a new conditional
BuildRequires: tcpdump to specfile.
Signed-off-by: Vladislav Odintsov
---
rhel/ovn-fedora.spec.in | 9 ++---
1 file
for 21.06 here:
https://patchwork.ozlabs.org/project/ovn/patch/20210918125121.8257-1-odiv...@gmail.com/
21.03 and older branches have more non-trivial conflicts, and backporting
should be done more carefully.
If one needs that, he/she can try to do it by its own.
Regards,
Vladislav Odintsov
> On
.
In ls_in_pre_acl and ls_in_pre_lb we add new flow with priority 110
to skip such packets.
Signed-off-by: Vladislav Odintsov
Signed-off-by: Numan Siddique
(cherry picked from commit 62ca8b9620cc1168ace6905575b7d36438363aed)
---
northd/ovn-northd.8.xml | 28
northd/ovn-northd.c
You’re absolutely right.
The difference with normal chassis is only the lack of output port.
It is determined in l2_lkp table. This is the only one reason to send packet to
ingress pipeline.
Regards,
Vladislav Odintsov
> On 18 Sep 2021, at 01:25, Numan Siddique wrote:
>
> On Fri, Sep
Hi Numan,
I’ve posted a new patch version here:
https://patchwork.ozlabs.org/project/ovn/patch/20210917215602.10633-1-odiv...@gmail.com/
I’ve tried to answer your question about ACLs in documentation.
Please let me know if it is clear.
Regards,
Vladislav Odintsov
> On 17 Sep 2021, at 22
.
In ls_in_pre_acl and ls_in_pre_lb we add new flow with priority 110
to skip such packets.
Signed-off-by: Vladislav Odintsov
---
northd/northd.c | 14 ++
northd/ovn-northd.8.xml | 29 +
northd/ovn_northd.dl| 33 +++--
tests/ovn
ransit switch port and configured as ECMP routes.
Note 2: static routes within route tables will be advertised
and learned only if interconnecting transit switch's LRPs
will have options:route_table same value as route's route_table
value.
Signed-off-by: Vladislav Odintsov
---
NEWS
associated with appropriate route_table.
Signed-off-by: Vladislav Odintsov
---
northd/northd.c | 160 ---
northd/ovn-northd.8.xml | 63 --
ovn-nb.ovsschema| 5 +-
ovn-nb.xml | 30 +++
tests/ovn-ic.at | 4 +
tests/ovn-nbctl.at | 196
availability_zone.
Signed-off-by: Vladislav Odintsov
---
ic/ovn-ic.c | 17 +++--
1 file changed, 11 insertions(+), 6 deletions(-)
diff --git a/ic/ovn-ic.c b/ic/ovn-ic.c
index 99356253d..92c83d730 100644
--- a/ic/ovn-ic.c
+++ b/ic/ovn-ic.c
@@ -68,6 +68,7 @@ struct ic_context {
struct
based on them
so they're included.
Once those patches are accepts, I can drop them from this series.
- Added NEWS item.
- Added myself to authors list.
Vladislav Odintsov (3):
ic: process only local port_bindings
northd,utils: support for RouteTables in LRs
ic: add support for routing
Hi,
done:
https://patchwork.ozlabs.org/project/ovn/patch/20210917150104.6143-1-odiv...@gmail.com/
Regards,
Vladislav Odintsov
> On 17 Sep 2021, at 17:44, Numan Siddique wrote:
>
> On Wed, Sep 15, 2021 at 8:07 PM Vladislav Odintsov <mailto:odiv...@gmail.com>> wrote:
>
.
In ls_in_pre_acl and ls_in_pre_lb we add new flow with priority 110
to skip such packets.
Signed-off-by: Vladislav Odintsov
---
v1 -> v2:
- Patch rebased on upstream changes.
Please note: I've got no experience in DDLog and have no ability to extensively
test these chan
systemctl try-restart. Also,
upgrade ovn-ctl script to support --restart argument in it.
Ideally this should be done by systemd when restart is
called, but it's impossible to pass restart command to
systemd.
Signed-off-by: Vladislav Odintsov
---
rhel/ovn-fedora.spec.in | 8
utilities
igured STT tunnel (best of VXLAN
and STT) and GENEVE on another one ("best" of GENEVE and STT).
No traffic could be sent between these chassis. Though the common STT
could be used.
Now we configure only matching encaps between nodes.
Signed-off-by: Vladislav Odintsov
---
controller/e
Sorry, by OOT I meant non-inbox kmod.
I’ve tried to use inbox kernel module (from kernel package) and problem
resolved.
Regards,
Vladislav Odintsov
> On 16 Sep 2021, at 17:17, Vladislav Odintsov wrote:
>
> Hi Dumitru,
>
> I’ve tried to exclude OOT OVS kernel module.
> With
Hi Dumitru,
I’ve tried to exclude OOT OVS kernel module.
With OVN 20.06.3 + OVS 2.13.4 the problem solved.
Could you please try with OOT kmod? For me it looks like a bug in OOT OVS
kernel module code.
Thanks.
Regards,
Vladislav Odintsov
> On 16 Sep 2021, at 11:02, Dumitru Ceara wr
Hi Dumitru,
thanks for your reply.
Regards,
Vladislav Odintsov
> On 15 Sep 2021, at 11:24, Dumitru Ceara wrote:
>
> Hi Vladislav,
>
> On 9/13/21 6:14 PM, Vladislav Odintsov wrote:
>> Hi Numan,
>>
>> I’ve checked with OVS 2.16.0 and OVN master. The problem p
Hi Numan,
thanks for the suggestion. I’ll update patch with requested change.
Have you looked at ovn-ic changes? Should I wait for a review of that patch and
submit v3 after that?
Regards,
Vladislav Odintsov
> On 14 Sep 2021, at 19:13, Numan Siddique wrote:
>
> On Fri, Sep 3, 2021 a
.
In ls_in_pre_acl and ls_in_pre_lb we add new flow with priority 110
to skip such packets.
Signed-off-by: Vladislav Odintsov
---
Please note: I've got no experience in DDLog and have no ability to extensively
test these changes.
Just local ./configure --with-ddlog=...; make; make
Signed-off-by: Vladislav Odintsov
---
tests/ovn-controller-vtep.at | 64
1 file changed, 64 insertions(+)
diff --git a/tests/ovn-controller-vtep.at b/tests/ovn-controller-vtep.at
index 21d79c66b..2d1ebad56 100644
--- a/tests/ovn-controller-vtep.at
+++ b
ovn_start function for the
purpose of starting OVN.
Signed-off-by: Vladislav Odintsov
---
tests/ovn-controller-vtep.at | 128 +--
1 file changed, 62 insertions(+), 66 deletions(-)
diff --git a/tests/ovn-controller-vtep.at b/tests/ovn-controller-vtep.at
index
-controller recompute` must be removed from test before merge.
It is placed there to show where the bug currently is.
[1]: https://mail.openvswitch.org/pipermail/ovs-dev/2021-September/387435.html
Vladislav Odintsov (3):
tests: use ovn_start in controller-vtep tests
tests: check that flow
Hi Numan,
yes, you’re correct.
Regards,
Vladislav Odintsov
> On 14 Sep 2021, at 21:41, Numan Siddique wrote:
>
> On Fri, Sep 3, 2021 at 6:35 AM Vladislav Odintsov <mailto:odiv...@gmail.com>> wrote:
>>
>> When IC port_binding exists and transit switch is delet
Hi Numan,
I’ve checked with OVS 2.16.0 and OVN master. The problem persists.
Symptoms are the same.
# grep ct_zero_snat /var/log/openvswitch/ovs-vswitchd.log
2021-09-13T16:10:01.792Z|00019|ofproto_dpif|INFO|system@ovs-system: Datapath
supports ct_zero_snat
Regards,
Vladislav Odintsov
> On
There’s a typo in my previous mail. Please, see inline fix.
Regards,
Vladislav Odintsov
> On 13 Sep 2021, at 15:09, Vladislav Odintsov wrote:
>
> Hi,
>
> we’ve encountered a next problem with stateful ACLs.
>
> Suppose, we have one logical switch (ls1) and atta
can be fixed.
The problem is actual with OVS 2.13.4 and latest OVN master branch, however
we’ve met it on 20.06.3 with same OVS and it’s very important for us.
Thanks.
Regards,
Vladislav Odintsov
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Hi Numan,
Regards,
Vladislav Odintsov
> On 9 Sep 2021, at 20:36, Numan Siddique wrote:
>
> On Thu, Sep 9, 2021 at 10:43 AM Vladislav Odintsov <mailto:odiv...@gmail.com>> wrote:
>>
>> I’ve forgot to add:
>> there warnings appear while starting ov
I’ve forgot to add:
there warnings appear while starting ovn-controller service.
Regards,
Vladislav Odintsov
> On 9 Sep 2021, at 17:41, Vladislav Odintsov wrote:
>
> Hi,
>
> with ovn master code and OVS 2.16.0 with OOT kernel module I see error in
> ovs-vswitchd:
>
somebody of developers know
the reason and the possible impact?
Regards,
Vladislav Odintsov
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Hi Han,
Yes, somehow I’ve not seen your comments for the test, sorry.
I’m okay with those changes, thanks.
Regards,
Vladislav Odintsov
> On 7 Sep 2021, at 21:02, Han Zhou wrote:
>
> On Tue, Sep 7, 2021 at 4:51 AM Vladislav Odintsov wrote:
>>
>> When IC port_bindi
I’ve submitted version 2 of this patch:
https://patchwork.ozlabs.org/project/ovn/patch/20210907115052.7913-1-odiv...@gmail.com/
Regards,
Vladislav Odintsov
> On 7 Sep 2021, at 11:42, Vladislav Odintsov wrote:
>
> Hi Han,
>
> thanks for the review.
>
> Regards,
> Vladi
When IC port_binding exists and transit switch is deleted,
the orphan port_binding is left in the IC_SB_DB.
This patch fixes such situation and adds test for this case.
Signed-off-by: Vladislav Odintsov
---
v1 -> v2:
- moved port_binding cleanup from ts_run() to port_binding_
Hi Han,
thanks for the review.
Regards,
Vladislav Odintsov
> On 7 Sep 2021, at 08:37, Han Zhou wrote:
>
> On Tue, Aug 24, 2021 at 11:45 AM Vladislav Odintsov <mailto:odiv...@gmail.com>>
> wrote:
>>
>> When IC port_binding exists and transit switch is de
Signed-off-by: Vladislav Odintsov
---
AUTHORS.rst | 1 +
1 file changed, 1 insertion(+)
diff --git a/AUTHORS.rst b/AUTHORS.rst
index 3f93874a3..e03c6b407 100644
--- a/AUTHORS.rst
+++ b/AUTHORS.rst
@@ -388,6 +388,7 @@ Venkata Anil Kommaddi vkomm...@redhat.com
Venu Iyer
ransit switch port and configured as ECMP routes.
Note 2: static routes within route tables will be advertised
and learned only if interconnecting transit switch's LRPs
will have options:route_table same value as route's route_table
value.
Signed-off-by: Vladislav Odintsov
---
NEWS
Signed-off-by: Vladislav Odintsov
---
northd/ovn-northd.8.xml | 63 --
northd/ovn-northd.c | 198 +++---
ovn-nb.ovsschema| 5 +-
ovn-nb.xml | 30 +++
tests/ovn-ic.at | 4 +
tests/ovn-nbctl.at | 196 +-
tests/ovn
-org/ovn/commit/e3ae68ed9a1bb231f7b069539693474508f3ed86
[2]
https://github.com/ovn-org/ovn/commit/c0085228893e7bf07190fcccf50cf588b028edaa
Signed-off-by: Vladislav Odintsov
---
tests/ovn-northd.at | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tests/ovn-northd.at b
availability_zone.
Signed-off-by: Vladislav Odintsov
---
ic/ovn-ic.c | 17 +++--
1 file changed, 11 insertions(+), 6 deletions(-)
diff --git a/ic/ovn-ic.c b/ic/ovn-ic.c
index 418f19147..98d3e 100644
--- a/ic/ovn-ic.c
+++ b/ic/ovn-ic.c
@@ -68,6 +68,7 @@ struct ic_context {
struct
When IC port_binding exists and transit switch is deleted,
the orphan port_binding if left in the IC_SB_DB.
This patch fixes such situation and adds test for this case.
Signed-off-by: Vladislav Odintsov
---
ic/ovn-ic.c | 35 +++--
tests/ovn-ic.at | 52
ded myself to authors list.
Vladislav Odintsov (6):
ic: remove port_binding on ts deletion
ic: process only local port_bindings
tests: remove strict check for table=N for N > 9
northd,utils: support for RouteTables in LRs
ic: add support for routing tables in adv/learn routes
Update A
upcoming soft-freeze and as I’m new to project, so I don’t know if
it is accepted for now.
Regards,
Vladislav Odintsov
> On 30 Aug 2021, at 23:44, Numan Siddique wrote:
>
> On Mon, Aug 16, 2021 at 5:15 PM Vladislav Odintsov <mailto:odiv...@gmail.com>> wrote:
>>
>&
- ICNB DB
- ICSB DB
Only when this condition is met, we can safely run ovn-ic sync
logic.
Signed-off-by: Vladislav Odintsov
---
ic/ovn-ic.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/ic/ovn-ic.c b/ic/ovn-ic.c
index 18066a305..fc608af82 100644
--- a/ic/ovn-ic.c
+++ b/ic
Small optimization in routes advertizement code.
Instead of iterating over all routes and checking for
needed transit_switch and availability_zone, use
ovsdb_index built with appropriate fields.
Signed-off-by: Vladislav Odintsov
---
ic/ovn-ic.c | 22 ++
1 file changed, 14
When IC port_binding exists and transit switch is deleted,
the orphan port_binding if left in the IC_SB_DB.
This patch fixes such situation and adds test for this case.
Signed-off-by: Vladislav Odintsov
---
ic/ovn-ic.c | 35 +++--
tests/ovn-ic.at | 52
Thanks Numan for the information.
Regards,
Vladislav Odintsov
> On 24 Aug 2021, at 18:43, Numan Siddique wrote:
>
> On Mon, Aug 23, 2021 at 1:38 PM Odintsov Vladislav wrote:
>>
>> Hi,
>>
>> I’m wonder if this patch series is interesting for the project.
&g
Hi Numan,
thanks for the quick fix. This worked for me.
Regards,
Vladislav Odintsov
> On 23 Aug 2021, at 18:24, Vladislav Odintsov wrote:
>
> Tested-by: Vladislav Odintsov mailto:odiv...@gmail.com>>
>
> Regards,
> Vladislav Odintsov
>
>> On 21 Aug 2021, at
Tested-by: Vladislav Odintsov
Regards,
Vladislav Odintsov
> On 21 Aug 2021, at 07:58, num...@ovn.org wrote:
>
> From: Numan Siddique
>
> The commit 6fb87aad8c3("controller: Improve ct zone handling.")
> caused a regression. After this commit ovn-controller is
Thanks!
Regards,
Vladislav Odintsov
> On 19 Aug 2021, at 20:06, Han Zhou wrote:
>
> On Thu, Aug 19, 2021 at 8:30 AM Vladislav Odintsov
> wrote:
>>
>> This commit fixes an error where in case of LRs were connected
>> between different AZs with ovn-ic, their rout
Hi Han,
thanks for the review!
All requested changes I’ve addressed in a new version of this patch:
https://patchwork.ozlabs.org/project/ovn/patch/20210819153012.82531-1-odiv...@gmail.com/
Let me know if now patch is okay or needs more improvements.
Regards,
Vladislav Odintsov
> On 19
-by: Vladislav Odintsov
---
v1 -> v2:
- Address Han's review comments:
- Make comments and test more clear to understand.
- Use ovsdb_idl_index to optimize transit switch lookup.
---
ic/ovn-ic.c | 17 ++-
tests/ovn-ic.at | 55 +
-by: Vladislav Odintsov
---
v1 -> v2:
- Address Han's review comments:
- Make comments and test more clear to understand.
- Use ovsdb_idl_index to optimize transit switch lookup.
---
ic/ovn-ic.c | 17 ++-
tests/ovn-ic.at | 55 +
Signed-off-by: Vladislav Odintsov
---
tests/ovn-ic.at | 4 +
tests/ovn-nbctl.at| 165 +-
utilities/ovn-nbctl.c | 124 +--
3 files changed, 284 insertions(+), 9 deletions(-)
diff --git a/tests/ovn-ic.at b/tests/ovn
with appropriate route_table.
Signed-off-by: Vladislav Odintsov
---
northd/ovn-northd.8.xml | 63 --
northd/ovn-northd.c | 198 +++---
ovn-nb.ovsschema| 5 +-
ovn-nb.xml | 30 +++
tests/ovn-northd.at | 72 ++-
tests/ovn.at| 438
=?? and update AT_CHECK
command for it.
This change is relevant for lrouter ingress pipeline table=N
where N > 9, as next patches in this series will modify 10th table.
Signed-off-by: Vladislav Odintsov
---
tests/ovn-northd.at | 200 ++--
tests/ovn
reviewed.
Vladislav Odintsov (3):
tests: remove strict check for table=N for N > 9
northd: support for RouteTables in LRs
utilities: update ovn-nbctl with RouteTables support
northd/ovn-northd.8.xml | 63 --
northd/ovn-northd.c | 198 ++---
ovn-nb.ovsschema|
in old branches too.
Regards,
Vladislav Odintsov
> On 10 Aug 2021, at 18:28, Numan Siddique wrote:
>
> On Tue, Aug 10, 2021 at 8:00 AM Vladislav Odintsov <mailto:odiv...@gmail.com>> wrote:
>>
>> Hi Numan,
>>
>> thanks for the answer.
>>
>>
it was wrongly pointed to
lr_in_ip_input router stage.
Reported-at:
https://mail.openvswitch.org/pipermail/ovs-dev/2021-August/386607.html
Signed-off-by: Vladislav Odintsov
---
northd/ovn-northd.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/northd/ovn-northd.c b/northd/ovn
nstance, reg7) in one stage prior to lr_in_ip_routing?
If yes, I think we should fix router’s registers usage documentation:
xxreg0: ">= IP_INPUT" -> ">= DEFRAG"
xxreg1: ">= IP_INPUT" -> ">= IP_ROUTING"
Right?
Regards,
Vladislav Odintso
-by: Vladislav Odintsov
---
This bugfix should have no conflicts to all branches,
so it's reasonable to apply it down to supported branches.
---
ic/ovn-ic.c | 4 +++-
tests/ovn-ic.at | 34 ++
2 files changed, 37 insertions(+), 1 deletion(-)
diff --git a/ic/ovn-ic.c
that route is global for LR (current behaviour), and when
there is a string with route_table name
Regards,
Vladislav Odintsov
- add new stage in logical router ingress pipeline right before
lr_in_ip_routing - lr_in_ip_routing_pre. In this stage packets are checked
against inport
to update GH Actions with new inputs when I understand these questions.
Regards,
Vladislav Odintsov
> On 16 Jul 2021, at 17:57, Numan Siddique wrote:
>
> On Tue, Jul 13, 2021 at 2:00 PM Vladislav Odintsov <mailto:odiv...@gmail.com>> wrote:
>>
>> Hi Dumitru, Numan,
>&
Hi,
maybe somebody from dev list can take a look and knows an answer?
Thanks in advance.
Regards,
Vladislav Odintsov
> On 7 Jul 2021, at 19:32, Vladislav Odintsov wrote:
>
> Hi all,
>
> I’ve tried to setup OVN interconnection with only two ovn-controller nodes
> (e
Hi Numan,
could you please look at my case if you have time? Earlier I’ve sent you NB DB
dump.
Thanks in advance.
Regards,
Vladislav Odintsov
> On 1 Jul 2021, at 11:44, Vladislav Odintsov wrote:
>
> I looked at the patch and couldn’t understand how to use it in my case.
> Can
Hi Dumitru, Numan,
Regards,
Vladislav Odintsov
> On 12 Jul 2021, at 21:39, Numan Siddique wrote:
>
> On Fri, Jul 9, 2021 at 9:01 AM Dumitru Ceara <mailto:dce...@redhat.com>> wrote:
>>
>> On 7/8/21 6:34 PM, Vladislav Odintsov wrote:
>>> Hi,
>>
&
Signed-off-by: Vladislav Odintsov
Regards,
Vladislav Odintsov
> On 9 Jul 2021, at 16:55, Vladislav Odintsov wrote:
>
> Acked-by: Vladislav Odintsov
>
> Regards,
> Vladislav Odintsov
>
>> On 9 Jul 2021, at 16:10, Vladislav Odintsov wrote:
>>
>> Fro
Acked-by: Vladislav Odintsov
Regards,
Vladislav Odintsov
> On 9 Jul 2021, at 16:10, Vladislav Odintsov wrote:
>
> From: Dumitru Ceara
>
> Remove the use of sbrec_chassis_is_new() for uncommitted records. This
> is not the way IDL *_is_new() functions are supposed to
From: Dumitru Ceara
Remove the use of sbrec_chassis_is_new() for uncommitted records. This
is not the way IDL *_is_new() functions are supposed to be used.
Note: With this change if the system-id changes there will be a
transient error in ovn-controller due to ovn-controller trying to insert
a
ear which OVN versions are suppoted/LTS. Does OVN project have
such decisions?
5. It is not also clear to understand the upgrade policy. Should administrator
upgrade to each major OVN version or some versions can be skipped especially
when since 20.12 release there is a version pinning between ov
I looked at the patch and couldn’t understand how to use it in my case.
Can you clarify your idea please?
Regards,
Vladislav Odintsov
> On 28 Jun 2021, at 23:34, Numan Siddique wrote:
>
> On Fri, Jun 25, 2021 at 4:13 AM Vladislav Odintsov <mailto:odiv...@gmail.com>> wrot
Hi Numan,
did you have a chance to look at db dump?
Regards,
Vladislav Odintsov
> On 18 Jun 2021, at 23:34, Vladislav Odintsov wrote:
>
> Sorry, there was a typo.
> Sure, VM has IP 192.168.1.2/24 and host 192.168.2.2/24.
>
> [root@ovn-1 ~]# ovn-nbctl show
> switch 8e
Thanks Numan for adding ddlog support here and additional tests as well.
Okay, I’ll try to do it by myself next time, if any.
Regards,
Vladislav Odintsov
> On 22 Jun 2021, at 13:47, Numan Siddique wrote:
>
> On Tue, Jun 15, 2021 at 11:34 AM Vladislav Odintsov <mailto:odiv...@gmail
When physical host sends traffic to
its chassis, routing between sw1 and sw2 works.
gateway chassis: [9721a9e9-73ef-4a9a-8a50-afa84811c6ef]
OVN NB DB is in attachment.
Regards,
Vladislav Odintsov
> On 18 Jun 2021, at 21:27, Numan Siddique wrote:
>
> On Fri, Jun 18, 20
answer such ARP request?
Thanks.
Regards,
Vladislav Odintsov
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
manual work later for us to make backport in
our fork. :)
Actually I was going to send backport down to stable branches (21.06->20.03).
These patches should cleanly apply on all the branches, let me know if I have to
send backports or you can do it by yourself.
Regards,
Vladislav Odint
Hi Dumitru,
Thanks for the review, new patch is submitted to the list.
Regards,
Vladislav Odintsov
> On 15 Jun 2021, at 18:50, Dumitru Ceara wrote:
>
> On 6/11/21 1:44 PM, Vladislav Odintsov wrote:
>> Before this patch ovn-controller-vtep created Mcast_Macs_Remote
>>
Signed-off-by: Vladislav Odintsov
Acked-by: Dumitru Ceara
---
controller-vtep/vtep.c | 19 ---
1 file changed, 12 insertions(+), 7 deletions(-)
diff --git a/controller-vtep/vtep.c b/controller-vtep/vtep.c
index 49723b39d..314cc4eae 100644
--- a/controller-vtep/vtep.c
+++ b
locators for
associated datapath are changed. Next, this newly-created physical
locator set is updated in the MMR record.
Signed-off-by: Vladislav Odintsov
Acked-by: Dumitru Ceara
---
controller-vtep/vtep.c | 54
tests/ovn-controller-vtep.at | 70
review comments from Dumitru and Ben.
* Split optimization code to separate patch.
v1 -> v2
* Fixed formatting.
Vladislav Odintsov (2):
controller-vtep: fix mmr and physical locators create/update
ovn-controller-vtep: update tunnel_key and replication_mode only if
needed
contro
DHCP option Hostname (opcode 12) can be configured in
Logical_Switch_Port as well as in DHCP_Options table's
options:hostname column. If both methods of configuration were
used, Logical_Switch_Port's setting takes precedence.
Signed-off-by: Vladislav Odintsov
---
v2 -> v3
* Fixed style warn
new
to mail list patches submission :) How can I close previous patch versions in
patchwork?
Regards,
Vladislav Odintsov
> On 15 Jun 2021, at 15:35, Numan Siddique wrote:
>
> On Tue, Jun 15, 2021 at 5:00 AM Vladislav Odintsov wrote:
>>
>> Hi Numan,
>>
>> Th
DHCP option Hostname (opcode 12) can be configured in
Logical_Switch_Port as well as in DHCP_Options table's
options:hostname column. If both methods of configuration were
used, Logical_Switch_Port's setting takes precedence.
Signed-off-by: Vladislav Odintsov
---
v1 -> v2
* Added supp
Hi,
I’ve posted a new version of this patch:
https://patchwork.ozlabs.org/project/ovn/list/?series=248346
Regards,
Vladislav Odintsov
> On 11 Jun 2021, at 01:45, Ben Pfaff wrote:
>
> On Fri, Jun 11, 2021 at 12:13:46AM +0200, Dumitru Ceara wrote:
>> On 6/10/21 11:47 PM,
, which differ only
in hostnames.
Let me know your thoughts on this, please.
Regards,
Vladislav Odintsov
> On 14 Jun 2021, at 21:29, Numan Siddique wrote:
>
> On Fri, May 28, 2021 at 7:32 AM Vladislav Odintsov <mailto:odiv...@gmail.com>> wrote:
>>
>>
locators for
associated datapath are changed. Next, this newly-created physical
locator set is updated in the MMR record.
Signed-off-by: Vladislav Odintsov
---
controller-vtep/vtep.c | 60 ++-
tests/ovn-controller-vtep.at | 70
2
Signed-off-by: Vladislav Odintsov
---
controller-vtep/vtep.c | 19 ---
1 file changed, 12 insertions(+), 7 deletions(-)
diff --git a/controller-vtep/vtep.c b/controller-vtep/vtep.c
index 49723b39d..314cc4eae 100644
--- a/controller-vtep/vtep.c
+++ b/controller-vtep/vtep.c
ion code to separate patch.
v1 -> v2
* Fixed formatting.
Vladislav Odintsov (2):
controller-vtep: Fix MMR and physical locators create/update
controller-vtep: Update tunnel_key and replication_mode if needed
controller-vtep/vtep.c | 79 +---
tes
? Or in master branch I
should only use new-style and then rewrite it while backporting?
Regards,
Vladislav Odintsov
> On 10 Jun 2021, at 21:18, Dumitru Ceara wrote:
>
> On 6/10/21 7:57 PM, Vladislav Odintsov wrote:
>
> [...]
>
>>>>
>>>> @@ -231,13 +239,19
Hi Dumitru,
First of all, thanks for the review. My comments are inline.
Regards,
Vladislav Odintsov
> On 10 Jun 2021, at 18:50, Dumitru Ceara wrote:
>
> On 5/28/21 10:21 AM, Vladislav Odintsov wrote:
>> Before this patch ovn-controller-vtep created Mcast_Macs_Remote
>>
DHCP Option Hostname is a per-Logical_Switch_Port property, configured in
Logical_Switch_Port's options:hostname field. It is used if DHCPv4 is
enabled for this LSP.
Signed-off-by: Vladislav Odintsov
---
The implementation for ovn-northd-ddlog is absent, it needs help from
somebody, who's
locators for
associated datapath are changed. Next, this newly-created physical
locator set is updated in the MMR record.
Also, update logical switch's tunnel key and replication method only
if needed.
Signed-off-by: Vladislav Odintsov
---
controller-vtep/vtep.c | 66
locators for
associated datapath are changed. Next, this newly-created physical
locator set is updated in the MMR record.
Also, update logical switch's tunnel key and replication method only
if needed.
Signed-off-by: Vladislav Odintsov
---
controller-vtep/vtep.c | 63
401 - 492 of 492 matches
Mail list logo