DPDK CI discussions
 help / color / mirror / Atom feed
From: Ferruh Yigit <ferruh.yigit@amd.com>
To: Patrick Robb <probb@iol.unh.edu>
Cc: ci@dpdk.org, "Tu, Lijuan" <lijuan.tu@intel.com>,
	Aaron Conole <aconole@redhat.com>, zhoumin <zhoumin@loongson.cn>,
	Michael Santana <maicolgabriel@hotmail.com>,
	Lincoln Lavoie <lylavoie@iol.unh.edu>
Subject: Re: Email Based Re-Testing Framework
Date: Tue, 6 Jun 2023 22:40:14 +0100	[thread overview]
Message-ID: <7dee4c8f-8dbd-b9bd-c2ac-ba986de5b0f1@amd.com> (raw)
In-Reply-To: <CAJvnSUAbhwRqSw5jHRuP4Dwpa5eC5wCKMP+kFfCJE5NqePGVCw@mail.gmail.com>

On 6/6/2023 8:27 PM, Patrick Robb wrote:
>     Also it can be useful to run daily sub-tree testing by request, if
>     possible.
> 
> 
> That wouldn't be too difficult. I'll make a ticket for this. Although,
> for testing on the daily sub-trees, since that's UNH-IOL specific, that
> wouldn't necessarily have to be done via an email based testing request
> framework. We could also just add a button to our dashboard which
> triggers a sub-tree ci run. That would help keep narrow the scope of
> what the email based retesting framework is for. But, both email or a
> dashboard button would both work. 
> 

Thanks, agree that for sub-trees a button on dashboard is sufficient.


> On Tue, Jun 6, 2023 at 1:53 PM Ferruh Yigit <ferruh.yigit@amd.com
> <mailto:ferruh.yigit@amd.com>> wrote:
> 
>     On 6/6/2023 5:56 PM, Patrick Robb wrote:
>     > Hello all,
>     >
>     > I'd like to revive the conversation about a request from the community
>     > for an email based re-testing framework. The idea is that using one
>     > standardized format, dpdk developers could email the test-report
>     mailing
>     > list, requesting a rerun on their patch series for "X" set of tests at
>     > "Y" lab. I think that since patchwork testing labels (ie.
>     > iol-broadcom-Performance, github-robot: build, loongarch-compilation)
>     > are already visible on patch pages on patchwork, those labels are the
>     > most reasonable ones to expect developers to use when requesting a
>     > re-test. We probably wouldn't want to get any more general than that,
>     > like, say, rerunning all CI testing for a specific patch series at a
>     > specific lab, since it would result in a significant amount of
>     "wasted"
>     > testing capacity.
>     >
>     > The standard email format those of us at the Community Lab are
>     thinking
>     > of is like below. Developers would request retests by emailing the
>     > test-report mailing list with email bodies like:
>     >
>     > [RETEST UNH-IOL]
>     > iol-abi-testing
>     > iol-broadcom-Performance
>     >
>     > [RETEST Intel]
>     > intel-Functional
>     >
>     > [RETEST Loongson]
>     > loongarch-compilation
>     >
>     > [RETEST GHA]
>     > github-robot: build
>     >
>     > From there, it would be up to the various labs to poll the test-report
>     > mailing list archive (or use a similar method) to check for such
>     > requests, and trigger a CI testing rerun based on the labels
>     provided in
>     > the re-test email. If there is interest from other labs, UNH might
>     also
>     > be able to host the entire set of re-test requests, allowing other
>     labs
>     > to poll a curated list hosted by UNH. One simple approach would be for
>     > labs to download all emails sent to test-report and parse with
>     regex to
>     > determine the re-test list for their specific lab. But, if anyone has
>     > any better ideas for aggregating the emails to be parsed, suggestions
>     > are welcome! If this approach sounds reasonable to everyone, we could
>     > determine a timeline by which labs would implement the functionality
>     > needed to trigger re-tests. Or, we can just add re-testing for various
>     > labs if/when they add this functionality - whatever is better.
>     Happy to
>     > discuss at the CI meeting on Thursday.
>     >
> 
>     +1 to re-testing framework.
> 
> 
>     Also it can be useful to run daily sub-tree testing by request, if
>     possible.
> 
> 
> 
> -- 
> 
> Patrick Robb
> 
> Technical Service Manager
> 
> UNH InterOperability Laboratory
> 
> 21 Madbury Rd, Suite 100, Durham, NH 03824
> 
> www.iol.unh.edu <http://www.iol.unh.edu/>
> 
> 


  reply	other threads:[~2023-06-06 21:40 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-06-06 16:56 Patrick Robb
2023-06-06 17:53 ` Ferruh Yigit
2023-06-06 19:27   ` Patrick Robb
2023-06-06 21:40     ` Ferruh Yigit [this message]
2023-06-07 12:53     ` Aaron Conole
2023-06-08  1:14       ` Patrick Robb
2023-06-08  1:47       ` Patrick Robb
2023-06-12 15:01         ` Aaron Conole
2023-06-13 13:28           ` Patrick Robb
2023-06-20 14:01             ` Aaron Conole
2023-06-07  7:04 ` Thomas Monjalon
2023-06-21 16:21 ` Ali Alnubani
2023-07-10 21:16   ` Jeremy Spewock

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=7dee4c8f-8dbd-b9bd-c2ac-ba986de5b0f1@amd.com \
    --to=ferruh.yigit@amd.com \
    --cc=aconole@redhat.com \
    --cc=ci@dpdk.org \
    --cc=lijuan.tu@intel.com \
    --cc=lylavoie@iol.unh.edu \
    --cc=maicolgabriel@hotmail.com \
    --cc=probb@iol.unh.edu \
    --cc=zhoumin@loongson.cn \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).