[dpdk-ci] [EXT] Re: Wrong Checkpatch failure in Intel CI

[Jerin Jacob Kollanukkaran]

> -----Original Message-----
> From: Thomas Monjalon <thomas at monjalon.net>
> Sent: Wednesday, April 1, 2020 1:33 AM
> To: zhaoyan.chen at intel.com; ci at dpdk.org; Jerin Jacob Kollanukkaran
> <jerinj at marvell.com>
> Cc: sys_stv at intel.com; Nithin Kumar Dabilpuram
> <ndabilpuram at marvell.com>; david.marchand at redhat.com
> Subject: [EXT] Re: Wrong Checkpatch failure in Intel CI
> 
> External Email
> 
> ----------------------------------------------------------------------
> 31/03/2020 21:51, Jerin Jacob Kollanukkaran:
> > Hello,
> >
> > It looks like "devtools/check-symbol-change.sh" cached in Intel Patchwork CI.
> > The commit[2] is fixed the issue reported by [1]. The commit[2]
> > already part of Master but some reason some this cached in CI system and it
> pick OLD checkpatch infra.
> >
> > Could someone look into this issue from Intel side?
> >
> >
> > [1]
> > https://urldefense.proofpoint.com/v2/url?u=http-3A__mails.dpdk.org_arc
> > hives_test-2Dreport_2020-
> 2DMarch_123095.html&d=DwICAg&c=nKjWec2b6R0mOy
> >
> Paz7xtfQ&r=1DGob4H4rxz6H8uITozGOCa0s5f4wCNtTa4UUKvcsvI&m=RErROLH
> hI94iK
> > fA-Ee95iCZBQDqP8JFGsyTT9Nq5e8w&s=iL1-
> HL1ttarMJ5bCAL0aIxUjp49eoa3xUMldw
> > ma-CTw&e=
> 
> This email comes from checkpatch at dpdk.org This is not Intel infrastructure.

OK. I was not aware of it. Sorry guys.

> I will update checkpatch on dpdk.org.

The actual change is in devtools/check-symbol-change.sh. I am not sure why it is not picking up.
Hope updating the checkpatch will fix it. Just for the record, I am testing with checkpatch.pl from 5.6.0-rc5 kernel + todays dpdk.org master.

> 
> Side note: Marvell is allowed to run some tests and send reports as part of the
> distributed CI.

Good info.

>