[dpdk-ci] Failing pf_smoke test

[Thomas Monjalon]

22/09/2021 11:52, David Marchand:
> On Wed, Sep 22, 2021 at 9:23 AM Thomas Monjalon <thomas at monjalon.net> wrote:
> >
> > 22/09/2021 08:17, Chen, Zhaoyan:
> > > Thanks Lincoln and David to reminder.
> > >
> > > Actually, this is a real issue on main branch.
> > >
> > > The pf_smoke failed due to dpdk bad commit id: b3d95f1817288ca228f09b9164d6d3ff6249b175(has been merged into main branch).
> >
> > Why a patch failing CI has been merged?
> > Is it because we get used to have CI failing and don't pay attention?
> 
> The CI did report this failure on the original series.
> http://mails.dpdk.org/archives/test-report/2021-August/210297.html
> This series should not have been merged but it was on 08/29:
> https://inbox.dpdk.org/dev/0935a75ef4334a4c9af65ecb6adac33f@intel.com/
> 
> I am not sure when it got into next-net.
> 
> A revert was proposed before next-net pull in main on 09/07:
> http://patches.dpdk.org/project/dpdk/patch/20210907064537.74596-1-wenjun1.wu@intel.com/
> 
> Note, at this point, I had not noticed the original patch had
> triggered a regression.
> This revert was then deemed not appropriate and in the end the
> original series hit the main branch.

Thanks for the history.
We must improve to avoid such failure in future.