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