On Mon, Dec 12, 2022 at 3:45 PM Han Zhou wrote:
>
> Signed-off-by: Han Zhou
Thanks for improving the documentation.
There are a couple of small typos. PSB
Acked-by: Numan Siddique
Numan
> ---
> lib/inc-proc-eng.h | 127 +
> 1 fi
On Thu, Dec 15, 2022 at 4:38 PM Han Zhou wrote:
>
> On Mon, Dec 12, 2022 at 1:28 AM venu iyer wrote:
> >
> > Currently, even stateless flows are subject to connection tracking when
> there are
> > LB rules (for DNAT). However, if a flow needs to be subjected to LB, then
> it shouldn't
> > be conf
ic: prevent advertising/learning multiple same routes
> ic: minor code improvements
> ic-sb schema: add index for routes table & document upgrade path
For the entire series:
Acked-by: Numan Siddique
Numan
>
> Documentation/intro/install/ovn-upg
On Tue, Nov 22, 2022 at 12:54 PM Lorenzo Bianconi
wrote:
>
> Add Similar to IPv4 counterpart, introduce IPv6 load-balancer health
> check support.
>
> Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2136094
> Signed-off-by: Lorenzo Bianconi
Hi Lorenzo,
Thanks for adding this missing fe
On Tue, Dec 13, 2022 at 10:07 AM Xavier Simonart wrote:
>
> Hi Mark, Ales
>
> Thanks for your review and feedback.
> I'll send v2 for patch 4 and 7, and comments on patch 10.
>
> Thanks
> Xavier
>
> On Mon, Dec 12, 2022 at 10:33 PM Mark Michelson wrote:
>
> > Hi Xavier. I did a review of this ser
On Tue, Dec 13, 2022 at 1:20 PM Lorenzo Bianconi
wrote:
>
> > On Tue, Dec 13, 2022 at 5:03 AM Dumitru Ceara wrote:
> > >
> > > On 12/13/22 07:17, Han Zhou wrote:
> > > > On Mon, Dec 12, 2022 at 8:52 AM Numan Siddique wrote:
> > > >>
>
On Wed, Dec 14, 2022 at 7:48 PM Ihar Hrachyshka wrote:
>
> There's one code branch left for the now-killed "both" filter. (See
> below.) With this branch cleaned up,
>
> Acked-By: Ihar Hrachyshka
I applied this patch to the main and branch-22,12 with a few changes.
Please see below
>
> On Tue
On Wed, Dec 14, 2022 at 7:40 PM Ihar Hrachyshka wrote:
>
> Acked-By: Ihar Hrachyshka
Thanks Ihar for the reviews.
I applied this patch with the below small changes
-
diff --git a/ovn-nb.xml b/ovn-nb.xml
index c2d0
On Tue, Dec 13, 2022 at 2:33 PM Abhiram R N wrote:
>
> This patch set adds a new feature in OVN.
> i.e Remote Port Mirroring
> Already in OVS this support is present. We are leveraging that
> and trying to add support in OVN.
> So that from OVN APIs we can create/delete mirrors and
> attach and de
On Tue, Dec 13, 2022 at 3:33 PM Ihar Hrachyshka wrote:
>
> Abhiram, these style violations should be fixed.
Hi Ihar,
These warnings are ok and can be ignored. Perhaps we need to fix it
in the checkpatch to address these warnings.
Numan
>
> (I will review the actual code changes tomorrow.)
>
>
On Mon, Dec 12, 2022 at 2:17 PM Ihar Hrachyshka wrote:
>
> I appreciate the split of the code into pieces, since it became hard
> to track the changes and focus discussion on parts that are indeed
> controversial / require additional discussion, compared to rather
> static pieces.
>
> I'm not COMP
On Tue, Nov 29, 2022 at 7:50 AM Abhiram R N wrote:
>
> Hi Mark,
>
> Thanks for your review.
> Please see replies inline below
>
> On Tue, Nov 29, 2022 at 3:24 AM Mark Michelson wrote:
>
> > Hi, since this patch series doesn't have a cover letter, I figure I
> > would make some top-level comments
On Thu, Dec 8, 2022 at 12:15 PM Lorenzo Bianconi
wrote:
>
> In the current codebase ct_commit {} action clears ct_state metadata of
> the incoming packet. This behaviour introduces an issue if we need to
> check the connection tracking state in the subsequent pipeline stages,
> e.g. for hairpin tr
On Sun, Dec 11, 2022 at 8:54 AM Lorenzo Bianconi
wrote:
>
> > On Fri, Dec 9, 2022 at 9:43 AM Lorenzo Bianconi
> > wrote:
> > >
> > > > On Sun, Dec 4, 2022 at 5:06 PM Lorenzo Bianconi
> > > > wrote:
> > > > >
> > > > > Introduce the capability to apply QoS rules for logical switch ports
> > > > >
On Mon, Dec 12, 2022 at 5:41 AM Dumitru Ceara wrote:
>
> On 12/9/22 19:42, Vladislav Odintsov wrote:
> > Hi Dumitru,
>
> Hi Vladislav,
>
> >
> > please see answers inline.
> >
> > Regards,
> > Vladislav Odintsov
> >
> >> On 9 Dec 2022, at 17:37, Dumitru Ceara wrote:
> >>
> >> On 12/6/22 11:20, Vl
On Fri, Dec 9, 2022 at 9:43 AM Lorenzo Bianconi
wrote:
>
> > On Sun, Dec 4, 2022 at 5:06 PM Lorenzo Bianconi
> > wrote:
> > >
> > > Introduce the capability to apply QoS rules for logical switch ports
> > > claimed by ovn-controller. Rely on shash instead of sset for
> > > egress_ifaces.
> > >
>
On Sun, Dec 4, 2022 at 5:06 PM Lorenzo Bianconi
wrote:
>
> Introduce the capability to apply QoS rules for logical switch ports
> claimed by ovn-controller. Rely on shash instead of sset for
> egress_ifaces.
>
> Acked-by: Mark Michelson
> Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2
On Mon, Nov 28, 2022 at 9:26 AM Ales Musil wrote:
>
> Update the build script, so it is easier to
> define opts/flags per compiler, but also per architecture.
>
> Signed-off-by: Ales Musil
Thanks Ales.
I applied both the patches to the main.
Numan
> ---
> .ci/linux-build.sh | 48
On Tue, Dec 6, 2022 at 10:21 AM Ales Musil wrote:
>
> In order to keep backward compatibility with northd we need
> to check if MAC binding table actually has the timestamp column.
>
> Reported-at: https://bugzilla.redhat.com/2151066
> Signed-off-by: Ales Musil
Acked-by: Numa
On Tue, Dec 6, 2022 at 9:56 AM Dumitru Ceara wrote:
>
> On 12/6/22 15:52, Frode Nordahl wrote:
> > If a OVN deployment has OVN RBAC enabled for the southbound
> > database, enabling BFD would lead to permission errors.
> >
> > The data in the entries in the BFD table do not belong to any
> > given
On Tue, Dec 6, 2022 at 7:47 AM Dumitru Ceara wrote:
>
> On 12/2/22 21:36, Frode Nordahl wrote:
> > When creating a route with BFD, ovn-nbctl creates an entry both
> > in the Logical_Router_Static_Route and BFD tables.
> >
> > However, before this patch, removing a route does not remove the
> > cor
On Mon, Dec 5, 2022 at 11:37 AM Dumitru Ceara wrote:
>
> On 12/2/22 18:31, Vladislav Odintsov wrote:
> > This change will be useful in next commit.
> >
> > Signed-off-by: Vladislav Odintsov
> > ---
>
> Hi Vladislav,
>
> This looks OK to me but I think I'd squash it in the patch that actually
> us
he patches as RFC.
Numan
>
> Regards,
> Vladislav Odintsov
>
> > On 2 Dec 2022, at 00:20, Numan Siddique wrote:
> >
> > On Thu, Dec 1, 2022 at 3:58 PM Vladislav Odintsov > <mailto:odiv...@gmail.com>> wrote:
> >>
> >> Hi,
> >>
> &g
On Thu, Dec 1, 2022 at 10:11 AM Frode Nordahl
wrote:
>
> Hello all,
>
> We would like to request a new tag in OVN branch-22.03 for a couple of
> important issues [0][1], and of course all the other goodness that has
> arrived on that branch since the .1 point release in June.
>
> I guess these day
On Thu, Dec 1, 2022 at 3:58 PM Vladislav Odintsov wrote:
>
> Hi,
>
> is it possible to consider any of the problems written below and here [0] for
> the possible fixes to be included in upcoming OVN/OVS releases?
Hi,
I didn't get a chance to look at the patches. But if some of them are
fixing
On Thu, Nov 24, 2022 at 4:33 PM Lorenzo Bianconi
wrote:
>
> Introduce the capability to apply QoS rules for logical switch ports
> claimed by ovn-controller. Rely on shash instead of sset for
> egress_ifaces.
>
> Acked-by: Mark Michelson
> Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=
On Tue, Oct 25, 2022 at 5:17 PM Ilya Maximets wrote:
>
> On 9/26/22 19:52, num...@ovn.org wrote:
> > From: Numan Siddique
> >
> > ovsdb-server allows the OVSDB clients to specify the uuid for
> > the row inserts [1]. The C IDL client library is missing this
>
On Fri, Nov 25, 2022 at 4:53 AM Dumitru Ceara wrote:
>
> On 11/22/22 16:03, Ales Musil wrote:
> > The related traffic wasn't correctly forwarded
> > through the LB, the main issue was that the
> > traffic was not NATted. This series allows
> > the NAT to be applied and the traffic should
> > arriv
On Fri, Nov 25, 2022 at 6:18 AM Dumitru Ceara wrote:
>
> A few logical pipeline stages have been added. The internal minor
> version must be bumped every time that happens.
>
> Fixes: d3926b433e44 ("northd: rely on new actions for lb affinity")
> Signed-off-by: Dum
On Thu, Nov 24, 2022 at 9:14 AM Abhiram R N wrote:
>
> Mirror creation just creates the mirror. The lsp-attach-mirror
> triggers the sequence to create Mirror in OVS DB on compute node.
> OVS already supports Port Mirroring.
>
> Note: This is targeted to mirror to destinations anywhere outside the
On Tue, Oct 25, 2022 at 4:39 PM Lorenzo Bianconi
wrote:
>
> In the current codebase ct_commit {} action clears ct_state metadata of
> the incoming packet. This behaviour introduces an issue if we need to
> check the connection tracking state in the subsequent pipeline stages,
> e.g. for hairpin tr
On Tue, Nov 22, 2022 at 5:31 PM Lorenzo Bianconi
wrote:
>
> > Thanks Lorenzo!
> >
> > Acked-by: Mark Michelson
> >
> > The only question I have is why you converted to a shash instead of a smap?
>
> Hi Mark,
>
> Thx for the review. Do you mean shash instead of sset?
>
> Regards,
> Lorenzo
>
Than
On Thu, Nov 17, 2022 at 7:38 AM Dumitru Ceara wrote:
>
> Move the submodule to the tip of OVS branch3.0. This picks up:
> 6690ed80c netdev-linux: Fix inability to apply QoS on ports with custom
> qdiscs.
>
> Signed-off-by: Dumitru Ceara
Acked-by: Numan Siddique
Numan
On Fri, Nov 18, 2022 at 10:11 AM Dumitru Ceara wrote:
>
> Signed-off-by: Dumitru Ceara
Acked-by: Numan Siddique
Numan
> ---
> .github/workflows/test.yml | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/.github/workflows/test.yml b/.github/workflows/test.y
Global options.
> - 24 least significant bits: 0.
> - obs_point_id: The openflow table number.
>
> Adding this configuration is not enough to make OVS sample drops. The
> apropriate configuration IPFIX needs to be added to those chassis that
> you wish to sample from. See man(5)
al flows and as default
> openflow flows (e.g: for physical tables). The only exceptions are
> physical tables 68 and 70 that are used to implement chk_lb_hairpin and
> ct_snat_to_vip actions and don't drop traffic.
>
> Signed-off-by: Adrian Moreno
Acked-by: Numan Siddique
ple_Collector_Set table in order to configure IPFIX sampling.
>
> Signed-off-by: Adrian Moreno
Acked-by: Numan Siddique
This patch series needs a rebase. Can you please spin up v7 ?
Thanks
Numan
> ---
> controller/lflow.c| 1 +
> include/ovn/actions.h | 16
On Tue, Nov 22, 2022 at 2:36 AM Han Zhou wrote:
>
> On Mon, Nov 21, 2022 at 4:40 AM Xavier Simonart wrote:
> >
> > In the following scenario:
> > - interface "old" is created and external_ids:iface-id is set (to lp)
> > - interface "new" is created and external_ids:iface-id is set (to same lp)
>
On Tue, Nov 22, 2022 at 1:20 AM Han Zhou wrote:
>
> On Mon, Nov 21, 2022 at 7:50 PM wrote:
> >
> > From: Numan Siddique
> >
> > Updates to NB address sets and NB port groups are handled
> > incrementally for syncing the SB address sets. This patch
> &
On Sat, Nov 19, 2022 at 8:02 PM Han Zhou wrote:
>
> On Sat, Nov 19, 2022 at 3:35 PM Numan Siddique wrote:
> >
> > On Sat, Nov 19, 2022 at 6:24 PM Numan Siddique
> wrote:
> > >
> > > On Fri, Nov 18, 2022 at 7:38 PM Han Zhou wrote:
> > > >
>
On Wed, Nov 16, 2022 at 5:30 AM Ales Musil wrote:
>
> In order to allow related traffic use the
> new action ct_commit_nat, which ensures that
> the traffic is commited and NATted. In combination
> with match on ct.rel it allows the related traffic
> to go through with correct NAT being applied.
>
On Fri, Nov 4, 2022 at 10:14 AM Ales Musil wrote:
>
> On Fri, Nov 4, 2022 at 2:57 PM Mark Michelson wrote:
>
> > Hi Ales,
> >
>
> Hi Mark,
> thank you for the review.
>
>
> >
> > On 10/20/22 10:14, Ales Musil wrote:
> > > To avoid make it easier to add flow to this stage refactor
> > > the functi
On Fri, Nov 4, 2022 at 4:52 AM Ales Musil wrote:
>
> The new timestamp column in MAC_Binding is
> populated with current time whenever the row is
> created or the MAC address is updated.
> This can be utilized by MAC binding aging mechanism,
> when we can check if enough time has passed since the
On Sat, Nov 19, 2022 at 6:24 PM Numan Siddique wrote:
>
> On Fri, Nov 18, 2022 at 7:38 PM Han Zhou wrote:
> >
> > On Fri, Nov 18, 2022 at 3:53 PM Han Zhou wrote:
> > >
> > >
> > >
> > > On Fri, Nov 18, 2022 at 2:58 PM Numan Siddique wrote:
On Fri, Nov 18, 2022 at 7:38 PM Han Zhou wrote:
>
> On Fri, Nov 18, 2022 at 3:53 PM Han Zhou wrote:
> >
> >
> >
> > On Fri, Nov 18, 2022 at 2:58 PM Numan Siddique wrote:
> > >
> > > '
> > >
> > > On Fri, Nov 18, 2022 at 5:
'
On Fri, Nov 18, 2022 at 5:44 PM Numan Siddique wrote:
>
>
>
>
> On Fri, Nov 18, 2022, 4:55 PM Han Zhou wrote:
>>
>> On Fri, Nov 18, 2022 at 1:47 PM Numan Siddique wrote:
>> >
>> > On Fri, Nov 18, 2022 at 3:46 PM Han Zhou wrote:
>> &g
On Fri, Nov 18, 2022, 4:55 PM Han Zhou wrote:
> On Fri, Nov 18, 2022 at 1:47 PM Numan Siddique wrote:
> >
> > On Fri, Nov 18, 2022 at 3:46 PM Han Zhou wrote:
> > >
> > > On Fri, Nov 18, 2022 at 11:23 AM Numan Siddique
> wrote:
> > > >
> &g
On Fri, Nov 18, 2022 at 3:46 PM Han Zhou wrote:
>
> On Fri, Nov 18, 2022 at 11:23 AM Numan Siddique wrote:
> >
> > On Fri, Nov 18, 2022 at 11:27 AM Numan Siddique wrote:
> > >
> > > '
> > >
> > > On Fri, Nov 18, 2022 at 2:14 AM Han Z
On Fri, Nov 18, 2022 at 11:27 AM Numan Siddique wrote:
>
> '
>
> On Fri, Nov 18, 2022 at 2:14 AM Han Zhou wrote:
> >
> > On Tue, Nov 15, 2022 at 7:19 AM wrote:
> > >
> > > From: Numan Siddique
> > >
> > > Updates to NB address se
'
On Fri, Nov 18, 2022 at 2:14 AM Han Zhou wrote:
>
> On Tue, Nov 15, 2022 at 7:19 AM wrote:
> >
> > From: Numan Siddique
> >
> > Updates to NB address sets and NB port groups are handled
> > incrementally for syncing the SB address sets. This patch
&
On Fri, Nov 18, 2022 at 1:37 AM Han Zhou wrote:
>
> On Tue, Nov 15, 2022 at 7:19 AM wrote:
> >
> > From: Numan Siddique
> >
> > A sub-engine node 'en_address_set_sync' is added with-in the
> > 'en_sb_sync' node to sync the Address_Set tab
On Fri, Nov 18, 2022 at 9:56 AM Dumitru Ceara wrote:
>
> On 11/15/22 10:49, Ales Musil wrote:
> > On Fri, Nov 4, 2022 at 4:50 PM Adrian Moreno wrote:
> >
> >> Very often when troubleshooting networking issues in an OVN cluster one
> >> would like to know if any packet (or a specific one) is being
se checks that there is no
conntrack entry for the stateless traffic.
Existing system tests should give you a good reference. Let me know
if you have any questions.
Numan
> regards,
>
> -venu
>
> ____
> From: Numan Siddique
> Sent: Monday,
On Tue, Nov 15, 2022 at 11:10 AM Mark Michelson wrote:
>
> On 11/14/22 16:34, Numan Siddique wrote:
> > On Mon, Nov 14, 2022 at 3:23 PM Mark Michelson wrote:
> >>
> >> Hi Numan, I have just one minor suggestion below.
> >>
> >> On 11/14/22 11:48,
On Mon, Nov 14, 2022 at 3:23 PM Mark Michelson wrote:
>
> Hi Numan, I have just one minor suggestion below.
>
> On 11/14/22 11:48, num...@ovn.org wrote:
> > From: Numan Siddique
> >
> > Updates to NB address sets and NB port groups are handled
> > increment
On Wed, Nov 9, 2022 at 4:15 PM Numan Siddique wrote:
>
> On Wed, Nov 9, 2022 at 4:11 PM Han Zhou wrote:
> >
> > On Tue, Nov 8, 2022 at 7:51 AM venu.iyer wrote:
> > >
> > > Currently, even stateless flows are subject to connection tracking when
> > th
On Wed, Nov 9, 2022 at 4:11 PM Han Zhou wrote:
>
> On Tue, Nov 8, 2022 at 7:51 AM venu.iyer wrote:
> >
> > Currently, even stateless flows are subject to connection tracking when
> there are
> > LB rules (for DNAT). However, if a flow needs to be subjected to LB, then
> it shouldn't
> > be config
On Fri, Nov 4, 2022 at 5:00 AM Felix Hüttner via dev
wrote:
>
> previously garps/rarps where only sent for "external" lsp's if these
> had an ipv4 address attached. For lsp's on gateway routers that do
> not have an ipv4 address assigned (e.g. if they are ipv6 only) no
> rarps were send out.
>
> T
On Thu, Nov 3, 2022 at 7:10 AM Xavier Simonart wrote:
>
> On Wed, Nov 2, 2022 at 3:31 PM Dumitru Ceara wrote:
>
> > On 11/2/22 15:21, Numan Siddique wrote:
> > > On Wed, Nov 2, 2022 at 7:39 AM Xavier Simonart
> > wrote:
> > >>
> > >> Hi N
On Wed, Nov 2, 2022 at 7:39 AM Xavier Simonart wrote:
>
> Hi Numan, Dumitru
>
> Thanks for looking at this patch and for your feedback.
>
>
> On Tue, Nov 1, 2022 at 10:41 PM Numan Siddique wrote:
>
> > On Fri, Oct 28, 2022 at 11:36 AM Dumitru Ceara wrote:
> &g
get_zone_num "$ct_zones" ls0-hv1)
> +port2_zone=$(get_zone_num "$ct_zones" ls0-hv2)
> +snat_zone=$(get_zone_num "$ct_zones" ${lr_uuid}_snat)
> +
> +check test "$snat_zone" -eq "$snat_req_zone"
> +check test "$port1_zone" -ne
On Mon, Oct 24, 2022 at 6:47 AM Felix Hüttner via dev
wrote:
>
> previously garps/rarps where only sent for "external" lsp's if these
> had an ipv4 address attached. For lsp's on gateway routers that do
> not have an ipv4 address assigned (e.g. if they are ipv6 only) no
> rarps were send out.
>
>
t;
> This change has been tested in an environment with 600 logical routers
> on a single ipv6 external network.
>
> Signed-off-by: Felix Huettner
Acked-by: Numan Siddique
Numan
> ---
> controller/pinctrl.c | 23 +
> tests/ovn.at | 80 +
On Mon, Oct 24, 2022 at 6:48 AM Felix Hüttner via dev
wrote:
>
> in order to easily add future ipv6 test cases the common `ovn_attach`
> function should also support ipv6 addresses.
>
> Signed-off-by: Felix Huettner
Acked-by: Numan Siddique
Numan
> ---
> tes
hanges done, you can add my Acked-by tag in the next version
Acked-by: Numan Siddique
Numan
> ---
> northd/northd.c | 9 +
> northd/ovn_northd.dl | 2 +-
> tests/ovn-northd.at | 18 +-
> tests/ovn.at | 8 +++-
> 4 files changed, 22
On Mon, Oct 24, 2022 at 6:47 AM Felix Hüttner via dev
wrote:
>
> We need to be able to handle rarp fields in order to ensure we can
> handle rarp messages we send ourselves.
> This will be used by the next patch in the series.
>
> Signed-off-by: Felix Huettner
Acked-by: Numa
On Fri, Oct 28, 2022 at 11:36 AM Dumitru Ceara wrote:
>
> Hi Xavier,
>
> On 10/24/22 17:21, Xavier Simonart wrote:
> > In the following scenario:
> > - interface "old" is created and external_ids:iface-id is set (to lp)
> > - interface "new" is created and external_ids:iface-id is set (to same lp)
On Mon, Oct 17, 2022 at 9:15 AM Adrian Moreno wrote:
>
> By default, traffic that doesn't match any configured flow will be dropped.
> But having that behavior implicit makes those drops more difficult to
> visualize.
>
> Make default drops explicit both as default logical flows and as default
> o
On Mon, Oct 17, 2022 at 9:14 AM Adrian Moreno wrote:
>
> sample ovn action encodes into the OFPACT_SAMPLE ovs action.
>
> OVN action allows the following parameters:
>
> - obs_domain_id: 8-bit integer that identifies the sampling application.
> This value will be combined with the datapath's tun
rectly from "sockaddr_storage".
>
> Signed-off-by: Ales Musil
Thanks for the patch.
LGTM. Can you please also update the function comments in
lib/ovn-util.c now that the function
also sets 'struct in6_addr *ip'.
Acked-by: Numan Siddique
Numan
> ---
> lib/lb.
On Wed, Oct 12, 2022 at 3:32 PM Mark Michelson wrote:
>
> Wow, this shash has been unused since branch-21.06 . Good find!
>
> Acked-by: Mark Michelson
Thanks. Applied to the main branch.
Numan
>
> On 10/12/22 04:07, Ales Musil wrote:
> > Signed-off-by: Ales Musil
> > ---
> > controller/ovn
fa:16:3e:4c:e6:1f,dst=fa:16:3e:c9:a7:6b),eth_type(0x0800),ipv4(src=192.168.210.103,dst=1.1.1.254,proto=6,tos=0/0,ttl=64,frag=no),tcp(src=0/0,dst=0/0),
> packets:2, bytes:148, used:3.230s, offloaded:yes, dp:tc,
> actions:set(eth(src=fa:16:3e:ae:b5:e5,dst=8c:1f:64:30:61:43)),set(ipv4(t
On Tue, Oct 4, 2022 at 5:41 AM Dumitru Ceara wrote:
>
> On 9/21/22 15:36, Vladislav Odintsov wrote:
> > Signed-off-by: Vladislav Odintsov
> > ---
>
> Hi Vladislav,
>
> > rhel/ovn-fedora.spec.in | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/rhel/ovn-fedora.spec.in b/rhel/ovn-fedo
tage is reached, a
> separate CT zone is used for SNAT.
>
> Signed-off-by: Mark Michelson
Acked-by: Numan Siddique
Numan
> ---
> v2 -> v3:
> * Rebased on top of current main branch
> * Fixed checkpatch issues from v2.
> * Accounted for the ct_label -> ct_ma
On Sun, Sep 11, 2022 at 10:26 AM Abhiram R N wrote:
>
> Added changes in ovn-nbctl, ovn-sbctl, northd and in ovn-controller.
> While Mirror creation just creates the mirror, the lsp-attach-mirror
> triggers the sequence to create Mirror in OVS DB on compute node.
> OVS already supports Port Mirror
On Wed, Sep 21, 2022 at 11:33 AM Dumitru Ceara wrote:
>
> On 9/21/22 16:03, Mark Michelson wrote:
> > Acked-by: Mark Michelson
> >
> > Congrats, Dumitru!
Thanks. Applied to main.
Congrats, Dumitru !
Numan
> >
> > On 9/21/22 07:20, num...@ovn.
On Mon, Sep 19, 2022 at 2:06 AM Ales Musil wrote:
>
> On Fri, Aug 26, 2022 at 2:35 PM Dumitru Ceara wrote:
>
> > Signed-off-by: Dumitru Ceara
> > ---
> > lib/lb.c | 5 +
> > lib/lb.h | 1 -
> > 2 files changed, 1 insertion(+), 5 deletions(-)
> >
> > diff --git a/lib/lb.c b/lib/lb.c
> > inde
On Fri, Sep 16, 2022 at 11:59 AM Numan Siddique wrote:
>
> On Tue, Sep 13, 2022 at 7:34 AM Ilya Maximets wrote:
> >
> > Every time a new logical flow is created, ovn-northd creates an
> > exclusive hash map to collect all the datapaths this logical flow is
> > appl
On Fri, Sep 16, 2022 at 11:38 AM Mark Michelson wrote:
>
> Signed-off-by: Mark Michelson
Acked-by: Numan Siddique
Numan
> ---
> NEWS | 3 +++
> configure.ac | 2 +-
> debian/changelog | 6 ++
> 3 files changed, 10 insertions(+), 1 deletion(-)
>
&g
On Fri, Sep 16, 2022 at 11:38 AM Mark Michelson wrote:
>
> Signed-off-by: Mark Michelson
Acked-by: Numan Siddique
I'm sure you're already aware of it (but just as a reminder) that this
patch should also be merged to the main branch.
Thanks
Numan
> ---
> NEWS
l DHCP option but a
> special use case in OVN.
>
> [0]
> https://github.com/openvswitch/ovs/blob/9dd3031d2e0e9597449e95428320ccaaff7d8b3d/lib/dhcp.h#L42
>
> Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2083629
> Reported-at:
> https://mail.openvswitch.org/pipermail/ovs-discuss/
ces. However, 'n_datapaths' is used practically
> everywhere, including ovn_lflow_add(), so we'll have to pass it around
> in most of the functions in northd as an argument. Taking that into
> account, I decided to make them both global to avoid touching half of
> th
h/20220908140644.2647859-1-odiv...@gmail.com/
>
> Regards,
> Vladislav Odintsov
>
> > On 16 Sep 2022, at 02:25, Vladislav Odintsov wrote:
> >
> > Ok,
> > I’ll correct patch and resend v2.
> >
> > Regards,
> > Vladislav Odintsov
> &
On Tue, Sep 13, 2022 at 1:19 PM Han Zhou wrote:
>
> On Mon, Sep 12, 2022 at 3:25 AM Dumitru Ceara wrote:
> >
> > On 9/9/22 23:32, Ilya Maximets wrote:
> > > Re-compute of 'northd' node in ovn-northd may take almost half of the
> > > total processing time in case there is a big number of load bala
,nw_frag=no
> Datapath actions: drop
>
> [root@dev1 ~]# ovn-nbctl show public
> switch 633ad6c1-f3e5-4f31-b2fe-215203cf5272 (public)
> port p-port1
> addresses: ["50:54:00:00:00:03 172.16.0.100"]
> port ln-public
> type: localnet
>
On Tue, Sep 13, 2022 at 6:41 PM venugopal iyer via dev
wrote:
>
> Hi, Han, Numan:
> While testing a use case in our ovn-k8s cluster we ran into an issue where
> wecouldn't effectively use stateless ACL on the OVN interface. Turns out we
> will track
> all the packets here, since there will be a
On Mon, Sep 12, 2022 at 6:11 PM Vladislav Odintsov wrote:
>
> If one has a UDP load balancer with backend IP which is located under
> disabled LSP, such backend would be threated as alive and marked as
> 'online' on Service_Monitor table and added to load balancing as well.
> Though such LSP can't
On Fri, Sep 9, 2022 at 4:00 AM Ilya Maximets wrote:
>
> On 8/25/22 11:03, Dumitru Ceara wrote:
> > On 8/24/22 18:32, Ilya Maximets wrote:
> >> On 8/24/22 17:57, Dumitru Ceara wrote:
> >>> On 8/24/22 17:49, Ilya Maximets wrote:
> On 8/24/22 17:43, Ilya Maximets wrote:
> > On 8/23/22 16:42,
livered to p-public.
Thanks
Numan
>
> Regards,
> Vladislav Odintsov
>
> > On 15 Sep 2022, at 15:42, Numan Siddique wrote:
> >
> > On Thu, Sep 8, 2022 at 7:41 AM Vladislav Odintsov > <mailto:odiv...@gmail.com>> wrote:
> >>
> >> Prior to thi
On Thu, Sep 8, 2022 at 7:41 AM Vladislav Odintsov wrote:
>
> Prior to this patch traffic to LSPs, which are disabled with
> `ovn-nbctl lsp-set-enabled disabled` was dropped in the end of
> lswitch egress pipeline. This means that traffic is processed in vain:
> - traffic, which should be dropped
ovnnb_cond_seqno = new_ovnnb_cond_seqno;
> > }
> >
> > - struct ovsdb_idl_txn *ovnsb_txn =
> > -ovsdb_idl_loop_run(&ovnsb_idl_loop);
> > +struct ovsdb_idl_txn *ovnsb_txn =
> run_i
On Fri, Sep 9, 2022 at 2:11 AM Ales Musil wrote:
>
> On Fri, Sep 9, 2022 at 7:43 AM Han Zhou wrote:
>
> >
> >
> > On Tue, Sep 6, 2022 at 6:02 AM Ales Musil wrote:
> > >
> > > The current behavior is that load balancers
> > > default to tcp protocol if it is not specified.
> > > However, this is
On Wed, Aug 31, 2022 at 9:17 AM Vladislav Odintsov wrote:
>
> Please, add this tag before applying the patch:
>
> Reported-at:
> https://mail.openvswitch.org/pipermail/ovs-discuss/2022-August/052021.html
I'm sorry. I missed adding this tag before applying the patch.
I also backported to branch
On Tue, Sep 6, 2022 at 2:40 PM Vladislav Odintsov wrote:
>
> Hi,
>
> we’ve noticed that after port binding is released, the conntrack entries in
> the associated to deleted port-binding zone are left in conntrack.
> I’ve ran through the code, it seems that conntrack zone flushing after port
> bi
, 10 insertions(+), 1 deletion(-)
>
> LGTM,
> Acked-by: Ilya Maximets
Looks like the branch-22.09 is not yet created.
For the entire series :
Acked-by: Numan Siddique
Numan
> ___
> dev mailing list
> d...@openvswitch
On Fri, Aug 19, 2022 at 10:42 AM Han Zhou wrote:
>
> On Thu, Aug 18, 2022 at 5:21 PM Han Zhou wrote:
> >
> > When handling port_binding changes, it is possible that new
> > local_datapaths are added, and the fields of the local_datapath, such as
> > localnet_port, external_ports, etc. need to be
it is risky if the OVSDB IDL implementation changes.
>
> Fixes: 3103487e087b ("Split out code to handle port binding db updates")
> Signed-off-by: Han Zhou
Acked-by: Numan Siddique
Numan
> ---
> controller/binding.c | 10 +-
> 1 file changed, 5 insertions(+),
ps://github.com/numansiddique/ovn/runs/7891302586?check_suite_focus=true
Please also add documentation in ovn-northd.8.xml about the new
logical flows you'd be adding, fix any existing test case failures and
also add a system test if possible.
Thanks
Numan
> Wentao Jia
>
>
0.102"
> type: "dnat_and_snat"
> nat 88b9371b-a058-46b5-91f9-6741edf3391a
> external ip: "1.1.1.124"
> logical ip: "192.168.210.0/24"
> type: "snat"
> nat f9843d73-341a-4557-b996-ced76b4fa
On Thu, Aug 18, 2022 at 3:47 AM Lorenzo Bianconi
wrote:
>
> Introduce system test for the following fix:
> 'd9f69b2d7 ("controller: physical: fix regression for container ports")'
Thanks for the follow up patch to add the test case. I applied this
to the main branch.
Numan
>
> Signed-off-by: L
401 - 500 of 2412 matches
Mail list logo