<div dir="ltr"><div><font size="2" style="color:rgb(0,0,0)"><span style="font-family:times new roman,serif">Hi All,<br><br></span></font></div><div><font size="2" style="color:rgb(0,0,0)"><span style="font-family:times new roman,serif">Below is our final draft of our announcement to the DPDK developers informing them of our new recheck request system. We would like your feedback and suggestions before we send it out.<br><br></span></font></div><div><font size="2" style="color:rgb(0,0,0)"><span style="font-family:times new roman,serif">Thanks,<br></span></font></div><div><font size="2" style="color:rgb(0,0,0)"><span style="font-family:times new roman,serif">Adam<br><br>--<br><br></span></font><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt" id="gmail-docs-internal-guid-f6f1d1f1-7fff-378f-271a-0982eca7c8a7"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:transparent;font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">Hello DPDK Developers,</span></font></p><font size="2" style="color:rgb(0,0,0)"><span style="font-family:times new roman,serif"><br></span></font><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:transparent;font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">Currently, various testing labs perform CI testing on new patch series sent to </span><a href="mailto:dev@dpdk.org" style="text-decoration:none;font-family:times new roman,serif"><span style="background-color:transparent;font-weight:400;font-style:normal;font-variant:normal;text-decoration:underline;vertical-align:baseline;white-space:pre-wrap">dev@dpdk.org</span></a><span style="background-color:transparent;font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif"> and report their results to </span><a href="https://patchwork.dpdk.org/project/dpdk/list/" style="text-decoration:none;font-family:times new roman,serif"><span style="background-color:transparent;font-weight:400;font-style:normal;font-variant:normal;text-decoration:underline;vertical-align:baseline;white-space:pre-wrap">https://patchwork.dpdk.org/project/dpdk/list/</span></a><span style="background-color:transparent;font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">. On each series on the patch list, the results appear in the test category contexts for IOL (community lab), GitHub, LoongSon, and Intel.</span></font></p><font size="2" style="color:rgb(0,0,0)"><span style="font-family:times new roman,serif"><br></span></font><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:transparent;font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">If a reported failure on a series seems suspect to the patch submitter or maintainer, then there is an interest in requesting a retest on the series for the failing label(s) in order to verify the failure is not spurious or false positive. This retest demonstrates to the submitter or maintainer that the failure can be reliably reproduced. Unfortunately, at present, the best way to accomplish this is to reach out to lab maintainers via email or slack. This is not ideal for developers in need of quick test results.</span></font></p><font size="2" style="color:rgb(0,0,0)"><span style="font-family:times new roman,serif"><br></span></font><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:transparent;font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">Going forward, CI testing labs will be implementing the option to request retest for their respective test labels on patchwork via emails sent to the dev mailing list. This feature is ready today for labels reported by the UNH-IOL Community Lab, and will soon also be an option for the Github Robot at least.</span></font></p><font size="2" style="color:rgb(0,0,0)"><span style="font-family:times new roman,serif"><br></span></font><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:transparent;font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">In order to request a retest on your patch series, send an email reply to one of your series’s patch or cover letter emails with email content of the format used below: </span></font></p><font size="2" style="color:rgb(0,0,0)"><span style="font-family:times new roman,serif"><br></span></font><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:rgb(255,255,255);font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">Recheck-request: <test names></span></font></p><font size="2" style="color:rgb(0,0,0)"><span style="font-family:times new roman,serif"><br></span></font><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:rgb(255,255,255);font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">The valid delimiter is a comma optionally followed by a space or a newline character: “,” “, “ “,\n”</span></font></p><font size="2" style="color:rgb(0,0,0)"><span style="font-family:times new roman,serif"><br></span></font><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:rgb(255,255,255);font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">Valid examples:</span></font></p><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:rgb(255,255,255);font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">Recheck-request: iol-compile-amd64-testing, iol-broadcom-Performance, iol-compile-arm64-testing,</span></font></p><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:rgb(255,255,255);font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">Recheck-request: iol-compile-amd64-testing,iol-broadcom-Performance, iol-compile-arm64-testing,</span></font></p><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:rgb(255,255,255);font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">Recheck-request: iol-compile-amd64-testing, iol-broadcom-Performance, iol-compile-arm64-testing</span></font></p><font size="2" style="color:rgb(0,0,0)"><span style="font-family:times new roman,serif"><br></span></font><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:rgb(255,255,255);font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">Invalid examples:</span></font></p><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:rgb(255,255,255);font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">Recheck-request: iol-compile-amd64-testing,  iol-broadcom-Performance</span></font></p><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:rgb(255,255,255);font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">Recheck-request: iol-compile-amd64-testing iol-broadcom-Performance,iol-compile-arm64-testing,</span></font></p><font size="2" style="color:rgb(0,0,0)"><span style="font-family:times new roman,serif"><br></span></font><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:rgb(255,255,255);font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">Some important notes:</span></font></p><ol style="margin-top:0px;margin-bottom:0px"><li dir="ltr" style="list-style-type:decimal;background-color:transparent;font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre"><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt" role="presentation"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:rgb(255,255,255);font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">At present, there is only support for retesting the series as it existed when the lab received it. As in, if the lab applied the series on DPDK mainline when the head was commit X, and a retest is requested, then retests will be run using those same sources applied on top of commit X. This is important to note because this means retest requests will not provide a solution to your patch being submitted when the tree is in a “bad state.”</span></font></p></li><li dir="ltr" style="list-style-type:decimal;background-color:transparent;font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre"><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt" role="presentation"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:rgb(255,255,255);font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">For any series submitted earlier than August 2023, you must submit a retest request in reply to a patch email, NOT in reply to a cover letter email.</span></font></p></li><li dir="ltr" style="list-style-type:decimal;background-color:transparent;font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre"><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt" role="presentation"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:rgb(255,255,255);font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">The initial policy is to accept no more than one retest request per patch per lab.</span></font></p></li><li dir="ltr" style="list-style-type:decimal;background-color:transparent;font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre"><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt" role="presentation"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:rgb(255,255,255);font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">Your patch should begin to retest within 30 minutes of your request, but wait time is subject to the testing queue just like any other series. As a result, retesting will be slower during peak submission time.</span></font></p></li></ol><font size="2" style="color:rgb(0,0,0)"><span style="font-family:times new roman,serif"><br></span></font><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:rgb(255,255,255);font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">Improvements we are considering for v2 of the email retesting framework:</span></font></p><ol style="margin-top:0px;margin-bottom:0px"><li dir="ltr" style="list-style-type:decimal;background-color:transparent;font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre"><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt" role="presentation"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:rgb(255,255,255);font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">Add in an option to re-apply on the latest commit on DPDK mainline. So, if your patch was originally applied on commit X, and you want to retest, but have it be applied to commit Y (latest), you could specify that. Under these circumstances, we would have to do a retest of all labels, since it would be inappropriate to mix reports for results from different commits.</span></font></p></li><li dir="ltr" style="list-style-type:decimal;background-color:transparent;font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre"><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt" role="presentation"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:rgb(255,255,255);font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">Add a policy for vetting retest requesters - so maybe only maintainers, or maybe only maintainers and the submitter, or another set of people. </span></font></p></li><li dir="ltr" style="list-style-type:decimal;background-color:transparent;font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre"><p dir="ltr" style="line-height:1.38;margin-top:0pt;margin-bottom:0pt" role="presentation"><font size="2" style="color:rgb(0,0,0)"><span style="background-color:rgb(255,255,255);font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap;font-family:times new roman,serif">Add in an option to request a retest for next-* branches and/or LTS branches.</span></font></p></li></ol></div><div><div><font size="2" style="color:rgb(0,0,0)"><span style="font-family:times new roman,serif"><br><br></span></font></div></div></div>