DPDK CI discussions
 help / color / mirror / Atom feed
* Re: [dpdk-ci] Understanding results of patchwork
       [not found]   ` <CAJFAV8xmLOdY3uoi1wD8jpdMRTYtAbgq8zXxhXqWETYBNA+7qw@mail.gmail.com>
@ 2020-04-15 16:45     ` Ferruh Yigit
  2020-04-15 17:11       ` Aaron Conole
  0 siblings, 1 reply; 3+ messages in thread
From: Ferruh Yigit @ 2020-04-15 16:45 UTC (permalink / raw)
  To: David Marchand
  Cc: Ajit Khaparde, Aaron Conole, sys_stv, Chen, Zhaoyan, Lincoln Lavoie, ci

On 4/15/2020 5:42 PM, David Marchand wrote:
> On Wed, Apr 15, 2020 at 6:34 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
>>
>> On 4/15/2020 3:40 PM, Ajit Khaparde wrote:
>>> Ferruh,
>>> I am trying to apply this patchset.
>>> https://patchwork.dpdk.org/project/dpdk/list/?series=9386
>>>
>>> So I am trying to check if the patches passed all the checks.
>>> I can see only the checkpatch for individual patches and the
>>> travis result - presumably for the complete set.
>>>
>>> https://travis-ci.com/ovsrobot/dpdk/builds/160349438
>>>
>>> For previous versions, I had seen results for individual patches and performance
>>> results also. Is it enough to apply the patches based on just the travis and
>>> checkpatch results?
>>>
>>
>> Hi Ajit,
>>
>> We expect other checks, cc'ed David too.
> 
> Better to Cc: ci@dpdk.org from my pov.

Agree, cc'ed now.

> 
> 
>>
>> Intel-compilation seems not sending the reports for a day, last one is from
>> yesterday, it may be stuck in some set, etc.. @Zhaoyan, can you please check it?
>>
>> The community lab one is still on pending state, it may be because it is a big
>> patchset.
>> https://lab.dpdk.org/results/dashboard/patchsets/10445/
>>
>>
>> And @David, @Aaron, does travis checks patchset only or does it checks each
>> patch in set?
> 
> Travis checks the whole patchset at the last patch.

Thanks, so we are missing patch by patch checks in our CI checks, since as far
as I know Intel check is also for patchset.

> The dpdk.org server sends checkpatch reports for each patch.
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-ci] Understanding results of patchwork
  2020-04-15 16:45     ` [dpdk-ci] Understanding results of patchwork Ferruh Yigit
@ 2020-04-15 17:11       ` Aaron Conole
  2020-04-15 17:24         ` Ajit Khaparde
  0 siblings, 1 reply; 3+ messages in thread
From: Aaron Conole @ 2020-04-15 17:11 UTC (permalink / raw)
  To: Ferruh Yigit
  Cc: David Marchand, Ajit Khaparde, sys_stv, Chen\,
	Zhaoyan, Lincoln Lavoie, ci\

Ferruh Yigit <ferruh.yigit@intel.com> writes:

> On 4/15/2020 5:42 PM, David Marchand wrote:
>> On Wed, Apr 15, 2020 at 6:34 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
>>>
>>> On 4/15/2020 3:40 PM, Ajit Khaparde wrote:
>>>> Ferruh,
>>>> I am trying to apply this patchset.
>>>> https://patchwork.dpdk.org/project/dpdk/list/?series=9386
>>>>
>>>> So I am trying to check if the patches passed all the checks.
>>>> I can see only the checkpatch for individual patches and the
>>>> travis result - presumably for the complete set.
>>>>
>>>> https://travis-ci.com/ovsrobot/dpdk/builds/160349438
>>>>
>>>> For previous versions, I had seen results for individual patches and performance
>>>> results also. Is it enough to apply the patches based on just the travis and
>>>> checkpatch results?
>>>>
>>>
>>> Hi Ajit,
>>>
>>> We expect other checks, cc'ed David too.
>> 
>> Better to Cc: ci@dpdk.org from my pov.
>
> Agree, cc'ed now.
>
>> 
>> 
>>>
>>> Intel-compilation seems not sending the reports for a day, last one is from
>>> yesterday, it may be stuck in some set, etc.. @Zhaoyan, can you please check it?
>>>
>>> The community lab one is still on pending state, it may be because it is a big
>>> patchset.
>>> https://lab.dpdk.org/results/dashboard/patchsets/10445/
>>>
>>>
>>> And @David, @Aaron, does travis checks patchset only or does it checks each
>>> patch in set?
>> 
>> Travis checks the whole patchset at the last patch.
>
> Thanks, so we are missing patch by patch checks in our CI checks, since as far
> as I know Intel check is also for patchset.

For now, we only check the series as a whole in travis to save on
time.  It's expected that someone / something else would do
patch-at-a-time for the series, to preserve the ability to bisect
through series.

>> The dpdk.org server sends checkpatch reports for each patch.
>> 


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-ci] Understanding results of patchwork
  2020-04-15 17:11       ` Aaron Conole
@ 2020-04-15 17:24         ` Ajit Khaparde
  0 siblings, 0 replies; 3+ messages in thread
From: Ajit Khaparde @ 2020-04-15 17:24 UTC (permalink / raw)
  To: Aaron Conole
  Cc: Ferruh Yigit, David Marchand, sys_stv, Chen, Zhaoyan, Lincoln Lavoie, ci

[-- Attachment #1: Type: text/plain, Size: 2311 bytes --]

On Wed, Apr 15, 2020 at 10:11 AM Aaron Conole <aconole@redhat.com> wrote:

> Ferruh Yigit <ferruh.yigit@intel.com> writes:
>
> > On 4/15/2020 5:42 PM, David Marchand wrote:
> >> On Wed, Apr 15, 2020 at 6:34 PM Ferruh Yigit <ferruh.yigit@intel.com>
> wrote:
> >>>
> >>> On 4/15/2020 3:40 PM, Ajit Khaparde wrote:
> >>>> Ferruh,
> >>>> I am trying to apply this patchset.
> >>>> https://patchwork.dpdk.org/project/dpdk/list/?series=9386
> >>>>
> >>>> So I am trying to check if the patches passed all the checks.
> >>>> I can see only the checkpatch for individual patches and the
> >>>> travis result - presumably for the complete set.
> >>>>
> >>>> https://travis-ci.com/ovsrobot/dpdk/builds/160349438
> >>>>
> >>>> For previous versions, I had seen results for individual patches and
> performance
> >>>> results also. Is it enough to apply the patches based on just the
> travis and
> >>>> checkpatch results?
> >>>>
> >>>
> >>> Hi Ajit,
> >>>
> >>> We expect other checks, cc'ed David too.
> >>
> >> Better to Cc: ci@dpdk.org from my pov.
> >
> > Agree, cc'ed now.
> >
> >>
> >>
> >>>
> >>> Intel-compilation seems not sending the reports for a day, last one is
> from
> >>> yesterday, it may be stuck in some set, etc.. @Zhaoyan, can you please
> check it?
> >>>
> >>> The community lab one is still on pending state, it may be because it
> is a big
> >>> patchset.
> >>> https://lab.dpdk.org/results/dashboard/patchsets/10445/
> >>>
> >>>
> >>> And @David, @Aaron, does travis checks patchset only or does it checks
> each
> >>> patch in set?
> >>
> >> Travis checks the whole patchset at the last patch.
> >
> > Thanks, so we are missing patch by patch checks in our CI checks, since
> as far
> > as I know Intel check is also for patchset.
>
> For now, we only check the series as a whole in travis to save on
> time.  It's expected that someone / something else would do
> patch-at-a-time for the series, to preserve the ability to bisect
> through series.
>
Ok, thanks for the clarification.
We went through the exercise. Since this was a big patchset,
I was trying to be extra careful. So I wanted to make sure reports for all
automated checks are in. That's when I started digging into the reports
and the process.

Thanks


> >> The dpdk.org server sends checkpatch reports for each patch.
> >>
>
>

[-- Attachment #2: Type: text/html, Size: 4454 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-04-15 17:24 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <CACZ4nhvvNPB71Gs3DnpLmUDBPmkyOCmBL8qf9-_ynKRXc9itcA@mail.gmail.com>
     [not found] ` <d463e85b-83aa-300c-b89e-7f9138d18ff3@intel.com>
     [not found]   ` <CAJFAV8xmLOdY3uoi1wD8jpdMRTYtAbgq8zXxhXqWETYBNA+7qw@mail.gmail.com>
2020-04-15 16:45     ` [dpdk-ci] Understanding results of patchwork Ferruh Yigit
2020-04-15 17:11       ` Aaron Conole
2020-04-15 17:24         ` Ajit Khaparde

DPDK CI discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/ci/0 ci/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ci ci/ https://inbox.dpdk.org/ci \
		ci@dpdk.org
	public-inbox-index ci

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.ci


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git