Re: [dpdk-ci] Failures reported by Intel CI for series 10551

["Chen, Zhaoyan" <zhaoyan.chen@intel.com>]

Hi all,

I have updated the info for this issue in bugzilla.


We have re-built the patchset. It was passed. The root cause is that patches in the  series are disorder by patchid. (patch 7/9 and patch 8/9). Usually, we apply patches by the order of patch id in a series. 

Why the issue is exposed this time?

Meet 2 conditions,
- the patches are disorder in the series
- the disordered patches are modified same file

Solution
- Change applying patch order by patch date, rather than patch id in patchwork.
But we don't know if patch date is unique and ordered for each patch in the series. 
We need patchwork document to confirm. So far, its good.


For Thomas' suggestion, "sending the series report only on the last patch of the series", currently, I find all reports iol-* are sent to the first patch in the series. Shall we align? and shall we get feedback from all maintainers or developers in the community?



Regards,
Zhaoyan Chen

> -----Original Message-----
> From: Thomas Monjalon <thomas@monjalon.net>
> Sent: Friday, June 26, 2020 6:24 PM
> To: Chen, Zhaoyan <zhaoyan.chen@intel.com>
> Cc: ci@dpdk.org; sys_stv <sys_stv@intel.com>; David Marchand
> <david.marchand@redhat.com>
> Subject: Re: [dpdk-ci] Failures reported by Intel CI for series 10551
> 
> 26/06/2020 09:43, David Marchand:
> > On Fri, Jun 26, 2020 at 5:03 AM Chen, Zhaoyan
> <zhaoyan.chen@intel.com> wrote:
> > >
> > > Hi, David,
> > >
> > > For your question, "Is it normal to see all patches with the exact same
> test report?"
> > > Yes, we always test a series, rather than a single patch. You can
> > > see the exact same report on any patch in a series. (it's
> > > convenient, you don't need backward to search the header of the
> > > series, then check result)
> >
> > Convenience is subject to interpretation :-).
> > Other CI systems send a single report which is more sane for me.
> 
> I think these tests have 2 purposes:
> 	- sending quick error feedback to the author
> 	- check that all is green before merging In both cases we don't
> need to have the same report duplicated, because we check for failures in
> all patches anyway.
> 
> I suggest sending the series report only on the last patch of the series.
>