Sometimes it’s just a failure in the testing system and needs to retrigger the tests.
Regards,
Maayan Kashani
From: Yogev Chaimovich <yogev@cgstowernetworks.com>
Sent: Monday, 9 June 2025 16:11
To: Maayan Kashani <mkashani@nvidia.com>; Stephen Hemminger <stephen@networkplumber.org>
Cc: Dariusz Sosnowski <dsosnowski@nvidia.com>; Slava Ovsiienko <viacheslavo@nvidia.com>; Bing Zhao <bingz@nvidia.com>; Ori Kam <orika@nvidia.com>; Suanming Mou <suanmingm@nvidia.com>; Matan Azrad <matan@nvidia.com>; dev@dpdk.org
Subject: Re: [PATCH] net/mlx5: fix transceiver warning when not exist
External email: Use caution opening links or attachments
|
Hi,
Actually looks like we're going with another workaround, so I might not be needing my original patch.
Also, I do not understand how my patch (not printing a log) makes a test fail - I couldn't find that test in DPDK to verify. I'm not that familiar though..
Yogev
From: Maayan Kashani <mkashani@nvidia.com>
Sent: Wednesday, June 4, 2025 10:02
To: Stephen Hemminger <stephen@networkplumber.org>; Yogev Chaimovich <yogev@cgstowernetworks.com>
Cc: Dariusz Sosnowski <dsosnowski@nvidia.com>; Slava Ovsiienko <viacheslavo@nvidia.com>; Bing Zhao <bingz@nvidia.com>; Ori Kam
<orika@nvidia.com>; Suanming Mou <suanmingm@nvidia.com>; Matan Azrad <matan@nvidia.com>;
dev@dpdk.org <dev@dpdk.org>
Subject: RE: [PATCH] net/mlx5: fix transceiver warning when not exist
Hi, Yogev,
Thank you for contributing,
Can you fix the failing CI TEST?
https://patches.dpdk.org/project/dpdk/patch/20250506133107.4432-1-yogev@cgstowernetworks.com/
Regards,
Maayan Kashani
> -----Original Message-----
> From: Stephen Hemminger <stephen@networkplumber.org>
> Sent: Tuesday, 20 May 2025 21:49
> To: Yogev Chaimovich <yogev@cgstowernetworks.com>
> Cc: Dariusz Sosnowski <dsosnowski@nvidia.com>; Slava Ovsiienko
> <viacheslavo@nvidia.com>; Bing Zhao <bingz@nvidia.com>; Ori Kam
> <orika@nvidia.com>; Suanming Mou <suanmingm@nvidia.com>; Matan
> Azrad <matan@nvidia.com>;
dev@dpdk.org
> Subject: Re: [PATCH] net/mlx5: fix transceiver warning when not exist
>
> External email: Use caution opening links or attachments
>
>
> On Mon, 5 May 2025 12:14:51 +0000
> Yogev Chaimovich <yogev@cgstowernetworks.com> wrote:
>
> > From ffe666291bc531b89620cc28a7b45fa209700f35 Mon Sep 17 00:00:00
> 2001
> > From: yogev <yogev@cgstowernetworks.com>
> > Date: Sun, 4 May 2025 23:11:46 +0300
> > Subject: [PATCH] net/mlx5: fix transceiver warning when not exist
> >
> > In case no transceiver exist when trying to get the transceiver's info
> > There should not be an error message from the library.
> > The caller may print an appropriate error message using the relevant
> rte_errno if required.
> >
> > Signed-off-by: yogev <yogev@cgstowernetworks.com>
>
> Signed off by is a legal requirement and must use your full name.
> As in:
> Signed-off-by: Yogev Chaimovich <yogev@cgstowernetworks.com>