* 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@dpdk.org'
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@dpdk.org'
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
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).