Bleep bloop. Greetings Ben Pfaff, 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: Unexpected sign-offs from developers who are not authors or co-authors
or committers: Ben
Bleep bloop. Greetings Ben Pfaff, 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: Comment with 'xxx' marker
#4578 FILE: northd/ovn-northd-ddlog.c:1273:
* XXX If the tr
I'm Mrs.Lea Edem, I work with one of the leading banks here in Lome, the
capital, I sent mail earlier without an answer. I have an important business
that I want to discuss with you. I want to move a substantial amount of money
that belonged to our client who died since 2010 leaving nobody to cl
On 11/4/20 8:54 AM, Numan Siddique wrote:
> On Wed, Nov 4, 2020 at 3:44 AM Mark Michelson wrote:
>>
>> Aside from spelling "update" properly in the title of patch 1, this
Oops, thanks for spotting this!
>> looks good to me.
>>
>> Acked-by: Mark Michelson
Thanks!
>
> Thanks Dumitru for fixing
On Wed, Nov 4, 2020 at 12:33 PM Ben Pfaff wrote:
>
> This change makes the test much more debuggable:
>
> - Make it easier to find the particular failing test.
>
> - Use correct checksums so that the packets can be compared for
> equality instead of omitting the checksum field.
>
> - Factor out
On Wed, Nov 4, 2020 at 12:33 PM Ben Pfaff wrote:
>
> At the time this was written, there weren't any OVN releases independent
> of OVS, but now there are so we should mention them.
>
> Also, I don't think it's worth talking about compilers other than the
> three listed. We haven't tested with the
On Wed, Nov 4, 2020 at 12:34 PM Ben Pfaff wrote:
>
> This makes it possible for the upcoming ovn-northd-ddlog program to call
> into it from Rust.
>
> Signed-off-by: Ben Pfaff
Acked-by: Numan Siddique
Thanks
Numan
> ---
> lib/automake.mk | 1 +
> lib/ovn-l7.c| 39 ++
This series aims at further restricting ARP/ND broadcast domain for self
originated packets. Until now those packets were always flooded on all
ports of the switch.
The first patch of the series is needed in order to remove dependency on
self-originated GARPs to be sent through the network (OVS)
OVN uses GARPs to announce changes to locally owned NAT addresses. This is
OK when updating upstream router caches but is unnecessary for updating OVN
logical router MAC_Bindings.
ovn-controller already has the information required for directly
updating/inserting the MAC_Bindings that would be cr
Considering the following large scale deployment:
external-network -- public-logical-switch -- router-1 -- sw1 -- VIF-1
-- router-2 -- sw2 -- VIF-2
...
-- router-n -- swn --
Bleep bloop. Greetings Dumitru Ceara, 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.
build:
gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I ./include -I ./include -I ./ovn -I
./include -I
/var/lib
Tested-by: Flavio Fernandes mailto:fla...@flaviof.com>>
> On Nov 3, 2020, at 11:43 AM, Lorenzo Bianconi
> wrote:
>
> Fix the following warning reported by ovn-controller when there are no
> active backends for lb health_check and selection_fields have been
> configured for hash computation
>
On 11/4/20 11:59 AM, 0-day Robot wrote:
[...]
> controller/physical.c:387:23: error: ‘struct ofpact_bundle’ has no member
> named ‘n_slaves’
> if (bundle->n_slaves >= BUNDLE_MAX_SLAVES) {
>^
> controller/physical.c:387:37: error: ‘BUNDLE_MAX_SLAVES’ undeclare
On Wed, Nov 4, 2020 at 4:42 PM Dumitru Ceara wrote:
>
> On 11/4/20 11:59 AM, 0-day Robot wrote:
>
> [...]
>
> > controller/physical.c:387:23: error: ‘struct ofpact_bundle’ has no member
> > named ‘n_slaves’
> > if (bundle->n_slaves >= BUNDLE_MAX_SLAVES) {
> >^
On Wed, Nov 4, 2020 at 4:22 PM Dumitru Ceara wrote:
>
> OVN uses GARPs to announce changes to locally owned NAT addresses. This is
> OK when updating upstream router caches but is unnecessary for updating OVN
> logical router MAC_Bindings.
>
> ovn-controller already has the information required f
On 11/4/20 12:57 PM, Numan Siddique wrote:
> On Wed, Nov 4, 2020 at 4:42 PM Dumitru Ceara wrote:
>>
>> On 11/4/20 11:59 AM, 0-day Robot wrote:
>>
>> [...]
>>
>>> controller/physical.c:387:23: error: ‘struct ofpact_bundle’ has no member
>>> named ‘n_slaves’
>>> if (bundle->n_slaves >=
On 11/4/20 12:57 PM, Numan Siddique wrote:
> On Wed, Nov 4, 2020 at 4:42 PM Dumitru Ceara wrote:
>>
>> On 11/4/20 11:59 AM, 0-day Robot wrote:
>>
>> [...]
>>
>>> controller/physical.c:387:23: error: ‘struct ofpact_bundle’ has no member
>>> named ‘n_slaves’
>>> if (bundle->n_slaves >=
Potřebujete rychlou a záruční půjčku k zaplacení účtů nebo zahájení
podnikání? Nabízím osobní i obchodní půjčky, abych uspokojil vaše
finanční potřeby za nízkou úrokovou sazbu 3%. Kontaktujte nás ještě
dnes prostřednictvím hectorleoloancompan...@gmail.com
___
On Wed, Oct 28, 2020 at 3:30 AM Dumitru Ceara wrote:
>
> On 10/27/20 6:16 PM, num...@ovn.org wrote:
> > From: Numan Siddique
> >
> > This patch adds a new table 'Load_Balancer' in SB DB and syncs the
> > Load_Balancer table rows
> > from NB DB to SB DB. An upcoming patch will make use of this ta
On Fri, Oct 30, 2020 at 2:48 AM Dumitru Ceara wrote:
>
> On 10/27/20 6:18 PM, num...@ovn.org wrote:
> > From: Numan Siddique
> >
>
> Hi Numan,
>
> > Presently to handle the load balancer hairpin traffic (the traffic destined
> > to the
> > load balancer VIP is dnatted to the backend which origin
On Fri, Oct 30, 2020 at 3:13 AM Dumitru Ceara wrote:
>
> On 10/27/20 6:19 PM, num...@ovn.org wrote:
> > From: Numan Siddique
> >
> > Signed-off-by: Numan Siddique
> > ---
> > utilities/ovn-sbctl.c | 55 +++
> > 1 file changed, 55 insertions(+)
> >
> > dif
On Wed, Nov 04, 2020 at 03:36:20PM +0530, Numan Siddique wrote:
> On Wed, Nov 4, 2020 at 12:33 PM Ben Pfaff wrote:
> >
> > This change makes the test much more debuggable:
> >
> > - Make it easier to find the particular failing test.
> >
> > - Use correct checksums so that the packets can be compa
On Wed, Nov 04, 2020 at 03:38:44PM +0530, Numan Siddique wrote:
> On Wed, Nov 4, 2020 at 12:33 PM Ben Pfaff wrote:
> >
> > At the time this was written, there weren't any OVN releases independent
> > of OVS, but now there are so we should mention them.
> >
> > Also, I don't think it's worth talkin
On Wed, Nov 04, 2020 at 03:40:57PM +0530, Numan Siddique wrote:
> On Wed, Nov 4, 2020 at 12:34 PM Ben Pfaff wrote:
> >
> > This makes it possible for the upcoming ovn-northd-ddlog program to call
> > into it from Rust.
> >
> > Signed-off-by: Ben Pfaff
>
>
> Acked-by: Numan Siddique
Thanks, I
On 11/3/20 5:46 PM, Ben Pfaff wrote:
> On Tue, Nov 03, 2020 at 03:53:51AM +0100, Ilya Maximets wrote:
>> On 11/3/20 12:28 AM, Ben Pfaff wrote:
>>> match_format() prints out "igmp" for IGMP flows, but
>>> ofp_parse_protocol() didn't accept it, which meant that OVS would print
>>> out a flow that it
I get the impression (from the article) that 802.11 wireless framing
includes a sequence number. Maybe it is the "sequence control" in the
diagram here: https://en.wikipedia.org/wiki/802.11_Frame_Types. If so,
OVS doesn't have or provide access to it, so it can't calculate gaps.
On Wed, Nov 04,
On Wed, Nov 4, 2020 at 12:33 PM Ben Pfaff wrote:
>
> From: Justin Pettit
>
> The upcoming ddlog implementation of northd needs these functions too,
> so they should be in a common library.
>
> Signed-off-by: Justin Pettit
Acked-by: Numan Siddique
One question: Does a submitter of a patch need
On Wed, Nov 4, 2020 at 12:34 PM Ben Pfaff wrote:
>
> From: Leonid Ryzhyk
>
> Export `ddlog_warn` and `ddlog_err` functions that are just wrappers
> around `VLOG_WARN` and `VLOG_ERR`. This is not ideal because the
> functions are exported by `ovn_util.c` and the resulting log messages use
> `ovn_
Hi,
I would like to know if you are interested in acquiring IBM Watson Users
Records for your marketing and sales needs?
We also have: Microsoft Azure, TensorFlow, AWS, ServiceNow, OpenShift, VMware
Tanzu, Kubernetes and many more.
Data Fields: Name, Company's Name, Phone Number, Fax Num
On Wed, Nov 4, 2020 at 12:34 PM Ben Pfaff wrote:
>
> The idea is to run each test twice, once with ovn-northd, once
> with ovn-northd-ddlog. To do that, we add a macro OVN_FOR_EACH_NORTHD
> and bracket each test that uses ovn-northd in it.
>
> Signed-off-by: Ben Pfaff
> ---
> tests/ovn-ic.at
On Wed, Nov 04, 2020 at 11:19:45PM +0530, Numan Siddique wrote:
> On Wed, Nov 4, 2020 at 12:33 PM Ben Pfaff wrote:
> >
> > From: Justin Pettit
> >
> > The upcoming ddlog implementation of northd needs these functions too,
> > so they should be in a common library.
> >
> > Signed-off-by: Justin Pe
On Wed, Nov 04, 2020 at 11:20:31PM +0530, Numan Siddique wrote:
> On Wed, Nov 4, 2020 at 12:34 PM Ben Pfaff wrote:
> >
> > From: Leonid Ryzhyk
> >
> > Export `ddlog_warn` and `ddlog_err` functions that are just wrappers
> > around `VLOG_WARN` and `VLOG_ERR`. This is not ideal because the
> > fun
On Wed, Nov 04, 2020 at 11:55:38PM +0530, Numan Siddique wrote:
> On Wed, Nov 4, 2020 at 12:34 PM Ben Pfaff wrote:
> >
> > The idea is to run each test twice, once with ovn-northd, once
> > with ovn-northd-ddlog. To do that, we add a macro OVN_FOR_EACH_NORTHD
> > and bracket each test that uses o
Hi All,
Please see the updated patch review list from this week's call. Note we've
updated the list to include patches deferred in addition to the previously
listed states.
Regards
Ian
Patch List
=== Merge Candidate (Close to merging) ===
name: ovsdb: Remove read permission of *.db from oth
Paying more than $500/month in Visa/MasterCard Credit Card Processing Fees?
Give me 3 minutes to explain your alternative.
1) How much of your profit margin is 'spent' on fees every month? Tired of
paying?
Your processor isn't telling you about all the available options.
Why don't they
On 11/3/2020 5:07 PM, Yi-Hung Wei wrote:
On Tue, Nov 3, 2020 at 4:06 PM Gregory Rose wrote:
On 11/3/2020 2:17 PM, Yi-Hung Wei wrote:
[snip]
Hi Greg,
I thought your change in datapath.c is to fix the following new added
lines in 9ba57fc7 ("datapath: Add hash info to upcall"). I fou
Webinar en Vivo:
Contabilidad Forense. El control interno como herramienta para evitar fraudes
Jueves 26 de Noviembre - Horario de 10:00 a 17:00 Hrs.
Identificaremos y conoceremos los puntos finos sobre la contabilidad forense,
el control interno y la auditoría como elementos fundamentales para
When enabling '--ovs' or when not using '-y', ovs-bugtool crashes due to
Traceback (most recent call last):
File "/usr/local/sbin/ovs-bugtool", line 1410, in
sys.exit(main())
File "/usr/local/sbin/ovs-bugtool", line 690, in main
for (k, v) in data.items():
RuntimeError: dictionary chan
On 11/4/2020 3:16 PM, William Tu wrote:
When enabling '--ovs' or when not using '-y', ovs-bugtool crashes due to
Traceback (most recent call last):
File "/usr/local/sbin/ovs-bugtool", line 1410, in
sys.exit(main())
File "/usr/local/sbin/ovs-bugtool", line 690, in main
for (k, v
Hello ,
I'm Gabie from Shaodong Gabi Bags Co., Ltd. ,school bag, laptop bag, shopping
bag ,sports bag are main products .
How about your bags' business ,can I send our catalogue to you ?
Wish everything is very good in 2020 .
Best regards
Gabie
___
dev m
Buenos días
Llegadas estas fechas y como cada año, recordamos a todas las empresas
Españolas su derecho a consumir el Crédito de Formación del que disponen para
la formación de sus empleados en activo o en ERTE, antes de su caducidad a
final de año.
Actualmente se encuentra abierto el plazo
It seems worthwhile to ensure that this call does in fact time out.
Signed-off-by: Ben Pfaff
---
tests/ovn-northd.at | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/tests/ovn-northd.at b/tests/ovn-northd.at
index 0bf20c1a7053..26376c367178 100644
--- a/tests/ovn-northd.at
atlocal.in sets a global default 30-second timeout for all utility
definitions, via OVS_CTL_TIMEOUT=30, so there's no need for individual
shorter timeouts, and they can be harmful in cases where the machine
is running slowly for one reason or another.
Signed-off-by: Ben Pfaff
---
tests/ovn-contr
v1->v2 (thanks Numan!):
- Applied several patches.
- New commit to add documentation for the system-userspace testsuite.
- New patches to improve the testsuite a couple of ways.
- Revised "Prepare for multiple northd types" to fix system-userspace
testsuite.
- Updated DDlog implementa
From: Leonid Ryzhyk
Export `ddlog_warn` and `ddlog_err` functions that are just wrappers
around `VLOG_WARN` and `VLOG_ERR`. This is not ideal because the
functions are exported by `ovn_util.c` and the resulting log messages use
`ovn_util` as module name. More importantly, these functions do not
I hadn't run this before so I had to discover how it worked.
Signed-off-by: Ben Pfaff
---
Documentation/topics/testing.rst | 35
tests/automake.mk| 6 --
2 files changed, 39 insertions(+), 2 deletions(-)
diff --git a/Documentation/topics/tes
The idea is to run each test twice, once with ovn-northd, once
with ovn-northd-ddlog. To do that, we add a macro OVN_FOR_EACH_NORTHD
and bracket each test that uses ovn-northd in it.
Signed-off-by: Ben Pfaff
---
tests/ovn-ic.at | 11 +-
tests/ovn-macros.at | 96 ++--
tests/ovn
Bleep bloop. Greetings Ben Pfaff, 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: Unexpected sign-offs from developers who are not authors or co-authors
or committers: Ben
Bleep bloop. Greetings Ben Pfaff, 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: Comment with 'xxx' marker
#4597 FILE: northd/ovn-northd-ddlog.c:1273:
* XXX If the tr
From: Numan Siddique
This patch series optimizes the load balancer hairpin logical flows.
Presently, ovn-northd generates a lot of these logical flows.
Suppose there are 'm' load balancers associated to a logical switch and each
load balancer
has 'n' VIPs and each VIP has 'p' backends then ovn-
From: Numan Siddique
This patch adds a new table 'Load_Balancer' in SB DB and syncs the
Load_Balancer table rows
from NB DB to SB DB. An upcoming patch will make use of this table for handling
the
load balancer hairpin traffic.
Signed-off-by: Numan Siddique
---
northd/ovn-northd.c | 146 ++
From: Numan Siddique
Presently to handle the load balancer hairpin traffic (the traffic destined to
the
load balancer VIP is dnatted to the backend which originated the traffic),
ovn-northd
adds a lot of logical flows to check this scenario. This patch attempts to
reduce the
these logical flow
From: Numan Siddique
Parsing of the load balancer VIPs is moved to a separate file - lib/lb.c.
ovn-northd makes use of these functions. Upcoming patch will make use of these
util functions for parsing SB Load_Balancers.
Signed-off-by: Numan Siddique
---
lib/automake.mk | 4 +-
lib/lb.c
From: Numan Siddique
The action - chk_lb_hairpin checks if the packet destined to a load balancer VIP
is to be hairpinned back to the same destination and if so, sets the
destination register
bit to 1.
The action - chk_lb_hairpin_reply checks if the packet is a reply of the
hairpinned
packet.
From: Numan Siddique
Signed-off-by: Numan Siddique
---
utilities/ovn-detrace.in | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/utilities/ovn-detrace.in b/utilities/ovn-detrace.in
index 1dd98df0aa..af42b5fc4e 100755
--- a/utilities/ovn-detrace.in
+++ b/utilities/
From: Numan Siddique
Signed-off-by: Numan Siddique
---
utilities/ovn-sbctl.c | 56 +++
1 file changed, 56 insertions(+)
diff --git a/utilities/ovn-sbctl.c b/utilities/ovn-sbctl.c
index 00c112c7e5..d3c37369d2 100644
--- a/utilities/ovn-sbctl.c
+++ b/utili
From: Numan Siddique
This patch makes use of the new hairpin OVN actions - chk_lb_hairpin,
chk_lb_hairpin_reply
and ct_snat_to_vip.
Suppose there are 'm' load balancers associated to a logical switch and each
load balancer
has 'n' VIPs and each VIP has 'p' backends then ovn-northd adds (m * ((
57 matches
Mail list logo