[dpdk-ci] |SUCCESS| [PATCH 2/3] maintainers: update virtio section name

Thomas Monjalon thomas.monjalon at 6wind.com
Thu Dec 1 10:06:08 CET 2016


2016-12-01 08:44, Xu, Qian Q:
> > On Thu, Dec 01, 2016 at 08:06:41AM +0100, checkpatch at dpdk.org wrote:
> > > Test-Label: checkpatch
> > > Test-Status: SUCCESS
> > > http://dpdk.org/patch/17348
> > >
> > > _coding style OK_
> > 
> > Great!  This kind of emails is something I'd like to see for a while.
> > But I don't like to receive SUCCESS notice. Normally, I'm positive my patches are
> > well qualified. The SUCCESS notice only makes my inbox fat-er but nothing else.

It is a bug.
If the email has the list-id dev.dpdk.org, success should not be reported to the author.

> > Of course, I'd like to receive failure notices if any.

> Thomas is the owner for the check patch, I wonder if Thomas can add one button or provide options for the patch submitters to receive the mails or not. 
> Generally developer may not want to receive the SUCCESS mail, but some may want to receive the mail to make sure the test is done and all pass. We may 
> not simply disable all SUCCESS mails, instead we'd better provide choice for people. 

If the patch is directly sent to checkpatch at dpdk.org, the author receive
the report (success or failure).
If the patch is sent to the mailing list, he receives only the failure
and can check success on patchwork.
I don't see the point of receiving success reports if it is public.


More information about the ci mailing list