On Thu, Apr 13, 2023 at 12:44 PM Han Zhou wrote:
>
>
>
> On Thu, Apr 13, 2023 at 12:35 PM Dumitru Ceara wrote:
> >
> > On 4/13/23 21:21, Han Zhou wrote:
> > > On Thu, Apr 13, 2023 at 8:37 AM Dumitru Ceara
wrote:
> > >>
> > >> On 4/13/23 17:34, Han Zhou wrote:
> > >>> On Thu, Apr 13, 2023 at
Scapy allows to define packets in descriptive form that is easier to
digest and debug.
Signed-off-by: Ihar Hrachyshka
---
v1: initial version
v2: use .decode() instead of sed to truncate b'...'
v2: remove unnecessary backspaces in example of use
v2: use $PYTHON3
---
tests/ovn-macros.at | 23
When setting flows for LS, OVN distinguishes between two states: where
there’s a stateful ACL present in its list (has_stateful == true *)
and when it’s missing (all ACLs are stateless).
When has_stateful == true, the following is done (among other things):
- ct handling flows are set;
- they are
I'm not sure but maybe it's something to do with the difference
between python3 paths in github CI and locally in my env? I see
python3 in github is
/opt/hostedtoolcache/Python/3.11.2/x64/bin/python3. I will try
$PYTHON3 in the function definition and see if it works better.
On Thu, Apr 13, 2023
On 4/13/23 21:42, Han Zhou wrote:
> On Thu, Apr 13, 2023 at 10:01 AM Dumitru Ceara wrote:
>>
>> On 4/13/23 18:26, Han Zhou wrote:
>>
>> I suggest a different approach if we want to go ahead and propagate
>>> such
>> information to the NB: can't we store the "active chassis"
>
On Thu, Apr 13, 2023 at 12:35 PM Dumitru Ceara wrote:
>
> On 4/13/23 21:21, Han Zhou wrote:
> > On Thu, Apr 13, 2023 at 8:37 AM Dumitru Ceara wrote:
> >>
> >> On 4/13/23 17:34, Han Zhou wrote:
> >>> On Thu, Apr 13, 2023 at 12:54 AM Dumitru Ceara
> > wrote:
>
> On 4/13/23 07:07, Han
On Thu, Apr 13, 2023 at 10:01 AM Dumitru Ceara wrote:
>
> On 4/13/23 18:26, Han Zhou wrote:
>
> I suggest a different approach if we want to go ahead and propagate
> > such
> information to the NB: can't we store the "active chassis"
information
> per
On 4/13/23 21:21, Han Zhou wrote:
> On Thu, Apr 13, 2023 at 8:37 AM Dumitru Ceara wrote:
>>
>> On 4/13/23 17:34, Han Zhou wrote:
>>> On Thu, Apr 13, 2023 at 12:54 AM Dumitru Ceara
> wrote:
On 4/13/23 07:07, Han Zhou wrote:
> In RFC1812 section 5.3.1, it is mentioned that:
>
On Thu, Apr 13, 2023 at 8:37 AM Dumitru Ceara wrote:
>
> On 4/13/23 17:34, Han Zhou wrote:
> > On Thu, Apr 13, 2023 at 12:54 AM Dumitru Ceara
wrote:
> >>
> >> On 4/13/23 07:07, Han Zhou wrote:
> >>> In RFC1812 section 5.3.1, it is mentioned that:
> >>>
> >>>If the TTL is reduced to zero (or
Hi all,
I just caught up on this discussion and wanted to complicate things
further by suggesting another idea. I think the Red Hat folks have heard
this before, but I'm not sure if it has been brought up on this list before.
Aside from this issue, there is also this high-priority issue from
Hello:
This patch was applied to netdev/net.git (main)
by Jakub Kicinski :
On Wed, 12 Apr 2023 07:58:28 -0400 you wrote:
> The netlink message for creating a new datapath takes an array
> of ports for the PID creation. This shouldn't cause much issue
> but correct it for future cases where we
On 4/13/23 18:26, Han Zhou wrote:
I suggest a different approach if we want to go ahead and propagate
> such
information to the NB: can't we store the "active chassis" information
per Gateway_chassis/HA_Chassis_group instead? That's
O(number-of-chassis) records that we
On Thu, Apr 13, 2023 at 6:33 AM Lucas Martins wrote:
>
> Hi Han, Dumitru and Luis,
>
> Thanks for the discussion and ideas so far. My reply is inline:
>
> On Thu, Apr 13, 2023 at 10:45 AM Luis Tomas Bolivar
wrote:
> >
> >
> >
> > On Thu, Apr 13, 2023 at 9:33 AM Dumitru Ceara wrote:
> >>
> >> On
On 4/13/23 17:19, Ales Musil wrote:
> On Thu, Apr 13, 2023 at 5:16 PM Dumitru Ceara wrote:
>
>> The package is not available for ARM:
>> E: Package 'gcc-multilib' has no installation candidate.
>>
>> It's needed for the 32-bit tests and will be addressed separately when
>> CI starts using the
On 4/13/23 17:34, Han Zhou wrote:
> On Thu, Apr 13, 2023 at 12:54 AM Dumitru Ceara wrote:
>>
>> On 4/13/23 07:07, Han Zhou wrote:
>>> In RFC1812 section 5.3.1, it is mentioned that:
>>>
>>>If the TTL is reduced to zero (or less), the packet MUST be
>>>discarded, and if the destination is
On Thu, Apr 13, 2023 at 12:54 AM Dumitru Ceara wrote:
>
> On 4/13/23 07:07, Han Zhou wrote:
> > In RFC1812 section 5.3.1, it is mentioned that:
> >
> >If the TTL is reduced to zero (or less), the packet MUST be
> >discarded, and if the destination is not a multicast address the
> >
On Thu, Apr 13, 2023 at 5:16 PM Dumitru Ceara wrote:
> The package is not available for ARM:
> E: Package 'gcc-multilib' has no installation candidate.
>
> It's needed for the 32-bit tests and will be addressed separately when
> CI starts using the pre-built container images.
>
> Fixes:
On 4/13/23 17:03, Ales Musil wrote:
> On Thu, Apr 13, 2023 at 4:53 PM Dumitru Ceara wrote:
>
>> On 3/15/23 07:29, Ales Musil wrote:
>>> The containers will bring some benefits
>>> to the overall CI proccess:
>>>
>>> 1) Reproducibility, there is high chance
>>> that if the test is failing
The package is not available for ARM:
E: Package 'gcc-multilib' has no installation candidate.
It's needed for the 32-bit tests and will be addressed separately when
CI starts using the pre-built container images.
Fixes: f89f5664259b ("ci: Add Ubuntu based Dockerfile")
Signed-off-by: Dumitru
On Thu, Apr 13, 2023 at 4:53 PM Dumitru Ceara wrote:
> On 3/15/23 07:29, Ales Musil wrote:
> > The containers will bring some benefits
> > to the overall CI proccess:
> >
> > 1) Reproducibility, there is high chance
> > that if the test is failing constantly on CI
> > users can reproduce it by
On 3/15/23 07:29, Ales Musil wrote:
> The containers will bring some benefits
> to the overall CI proccess:
>
> 1) Reproducibility, there is high chance
> that if the test is failing constantly on CI
> users can reproduce it by getting the same image
> and running the affect test.
>
> 2) Ease of
This patch allow to configure max/min rate greater than 4Gbps
https://bugzilla.redhat.com/show_bug.cgi?id=2129742
Tested-by: Rodolfo Alonso
Signed-off-by: Lorenzo Bianconi
---
controller/binding.c | 30 ++
1 file changed, 18 insertions(+), 12 deletions(-)
diff
Remove ovn-egress-iface paramter since it is no longer used.
https://bugzilla.redhat.com/show_bug.cgi?id=2129742
Tested-by: Rodolfo Alonso
Signed-off-by: Lorenzo Bianconi
---
controller/binding.c | 23 +--
northd/ovn-northd.8.xml | 6 --
tests/ovn-performance.at |
This patch allows to apply QoS rules on the localnet port related to
logical switch ports running on the same datapath. Considering the
following netowrk configuration:
LSP{0,1} -- LogicalSwitch -- Localnet0
It is possible to apply the following QoS rules on Localnet0 on egress traffic
entering
Introduce ovsport_lookup_by_name and ovsport_lookup_by_qos routines in
order to speed-up ovs port lookup based on port name or qos.
https://bugzilla.redhat.com/show_bug.cgi?id=2129742
Tested-by: Rodolfo Alonso
Signed-off-by: Lorenzo Bianconi
---
controller/binding.c| 45
Rework OVN QoS implementation in order to configure it through OVS QoS
table instead of running tc command directly bypassing OVS.
https://bugzilla.redhat.com/show_bug.cgi?id=2129742
Tested-by: Rodolfo Alonso
Signed-off-by: Lorenzo Bianconi
---
controller/binding.c| 421
Remove tunnel interfaces from egress list in order to not shape them.
Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2129742
Tested-by: Rodolfo Alonso
Signed-off-by: Lorenzo Bianconi
---
controller/binding.c | 9 -
1 file changed, 9 deletions(-)
diff --git
This is a preliminary patch to allow OVN to configure QoS through OvS
db instead of running tc directly.
https://bugzilla.redhat.com/show_bug.cgi?id=2129742
Tested-by: Rodolfo Alonso
Signed-off-by: Lorenzo Bianconi
---
controller/binding.c| 56 ++---
Rework OVN QoS implementation in order to configure it through OVS QoS
table instead of running tc command directly bypassing OVS.
This series allows to apply QoS rules on the localnet port related to
logical switch ports running on the same datapath. Considering the
following netowrk
Hi Han, Dumitru and Luis,
Thanks for the discussion and ideas so far. My reply is inline:
On Thu, Apr 13, 2023 at 10:45 AM Luis Tomas Bolivar wrote:
>
>
>
> On Thu, Apr 13, 2023 at 9:33 AM Dumitru Ceara wrote:
>>
>> On 4/12/23 23:07, Han Zhou wrote:
>> > On Wed, Apr 12, 2023 at 8:01 AM wrote:
On Mon, Apr 10, 2023 at 7:26 PM Mark Michelson wrote:
> This allows for evaluating ACLs at the current tier to stop, and to
> start evaluating ACLs at the next tier. If not using tiers, or if we
> match on the final ACL tier, then a "pass" verdict results in the
> default ACL action being
On Mon, Apr 10, 2023 at 7:26 PM Mark Michelson wrote:
> This modifies the acl-add and acl-del commands so that an ACL
> tier can be specified when adding or deleting ACLs.
>
> For acl-add, if the tier is specified, then the ACL created by the
> command will have that tier set.
>
> For acl-del,
On Mon, Apr 10, 2023 at 7:26 PM Mark Michelson wrote:
> With this commit, ACLs can now be arranged in hierarchical tiers. A tier
> number can be assigned to an ACL. When evaluating ACLs, we first will
> consider ACLs at tier 0. If no matching ACL is found, then we move to
> tier 1. This
On Thu, Apr 13, 2023 at 9:33 AM Dumitru Ceara wrote:
> On 4/12/23 23:07, Han Zhou wrote:
> > On Wed, Apr 12, 2023 at 8:01 AM wrote:
> >>
> >> From: Lucas Alvares Gomes
> >>
> >> In order for the CMS to know which Chassis a distributed gateway port
> >> is bond to, this patch updates the
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 of one-command run of ovn-sbctl
>>> (non-daemon mode), it doesn’t make sense to have client -> server
>>> inactivity
On Thu, Apr 13, 2023 at 10:30 AM Dumitru Ceara wrote:
> On 4/12/23 03:29, Ihar Hrachyshka wrote:
> > Scapy allows to define packets in descriptive form that is easier to
> > digest and debug.
> >
> > Signed-off-by: Ihar Hrachyshka
> > ---
> > tests/ovn-macros.at | 23 +++
>
On 4/13/23 10:28, Dumitru Ceara wrote:
> One is because the tests fail to run scapy:
>
> Traceback (most recent call last):
> File "", line 1, in
> ModuleNotFoundError: No module named 'scapy'
> "netdev-dummy/receive" command requires at least 2 arguments
> ovs-appctl: ovs-vswitchd: server
On 4/12/23 03:29, Ihar Hrachyshka wrote:
> Scapy allows to define packets in descriptive form that is easier to
> digest and debug.
>
> Signed-off-by: Ihar Hrachyshka
> ---
> tests/ovn-macros.at | 23 +++
> 1 file changed, 23 insertions(+)
>
> diff --git
On 4/12/23 03:29, Ihar Hrachyshka wrote:
> When setting flows for LS, OVN distinguishes between two states: where
> there’s a stateful ACL present in its list (has_stateful == true *)
> and when it’s missing (all ACLs are stateless).
>
> When has_stateful == true, the following is done (among
Let's separate DPDK compilation from the rest of OVS build:
- this avoids multiple jobs building DPDK in parallel, which especially
affects builds in the dpdk-latest branch,
- we separate concerns about DPDK build requirements from OVS build
requirements, like python dependencies,
- building
On 4/13/23 07:07, Han Zhou wrote:
> In RFC1812 section 5.3.1, it is mentioned that:
>
>If the TTL is reduced to zero (or less), the packet MUST be
>discarded, and if the destination is not a multicast address the
>router MUST send an ICMP Time Exceeded message ...
>
The code itself
On 4/13/23 07:07, Han Zhou wrote:
> Remove 'ip4' from the match condition for this flow.
>
> Signed-off-by: Han Zhou
> ---
> northd/northd.c | 2 +-
> tests/ovn-northd.at | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/northd/northd.c b/northd/northd.c
> index
On 4/12/23 23:07, Han Zhou wrote:
> On Wed, Apr 12, 2023 at 8:01 AM wrote:
>>
>> From: Lucas Alvares Gomes
>>
>> In order for the CMS to know which Chassis a distributed gateway port
>> is bond to, this patch updates the ovn-northd daemon to populate the
>> Logical_Router_Port table with that
43 matches
Mail list logo
