* 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).