From: Patrick Robb <probb@iol.unh.edu>
To: zhoumin <zhoumin@loongson.cn>
Cc: ci@dpdk.org, Aaron Conole <aconole@redhat.com>,
David Marchand <david.marchand@redhat.com>,
"Brandes, Shai" <shaibran@amazon.com>
Subject: Re: Email based retests for the Loongarch lab
Date: Wed, 11 Jun 2025 17:05:39 -0400 [thread overview]
Message-ID: <CAJvnSUB79G2-p3yjdqbtFzHrmFCQ87QkizHDJr1JbxCJro8u8g@mail.gmail.com> (raw)
In-Reply-To: <24d143d3-4739-457d-bf15-c6224ca21bb0@loongson.cn>
[-- Attachment #1: Type: text/plain, Size: 1888 bytes --]
Hi Min Zhou,
Thanks for the response and for getting some retest support online! I agree
the next reasonable steps are to continue to align out processes, in part
so that the lab behavior is predictable according to what is documented on
dpdk.org.
On Sat, Jun 7, 2025 at 8:50 PM zhoumin <zhoumin@loongson.cn> wrote:
> Yes, we don't support the rebase argument. Meanwhile, for the sake of
> simplicity, the rechecking behavior at Loongson lab is a little different
> from the default behavior. we recheck the patches on the latest HEAD of the
> branch selected by pw_maintainers_cli.py script. This behavior is same to
> that of the first testing for the commit at Loongson lab. It's ok to be
> consistent with the defualt behavior. Maybe we can support both it and
> rebase argument after you have synced some infrastructures.
>
Good point, we need to make sure the support for the rebase={branch}
argument is integrated into the get_reruns.py script in dpdk-ci. The UNH
team made that addition and it is available for patch review:
https://patchwork.dpdk.org/project/ci/patch/20250611205849.72165-1-probb@iol.unh.edu/
Maybe you can apply it, give it a run and add a tested by tag to the patch
if it is working for you? In order to make sure it was working I submitted
a placeholder loongson recheck request with the rebase argument (requesting
the main branch) included on the DPDK mailing list. Right now if I run the
script with the following args it produces this json:
python3 tools/get_reruns.py -ts 2025-06-10 --contexts loongarch-compilation
output json:
{
"retests": {
"34851": {
"contexts": [
"loongarch-compilation"
],
"arguments": {
"rebase": "main"
}
}
},
"last_comment_timestamp": "2025-06-11T20:29:39.327426"
}
[-- Attachment #2: Type: text/html, Size: 2745 bytes --]
prev parent reply other threads:[~2025-06-11 21:10 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-22 5:55 Patrick Robb
2024-02-22 13:54 ` Patrick Robb
2024-02-28 5:34 ` zhoumin
2024-02-29 6:09 ` Patrick Robb
2024-03-01 10:19 ` zhoumin
2025-05-28 19:36 ` Patrick Robb
2025-06-08 0:49 ` zhoumin
2025-06-11 21:05 ` Patrick Robb [this message]
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=CAJvnSUB79G2-p3yjdqbtFzHrmFCQ87QkizHDJr1JbxCJro8u8g@mail.gmail.com \
--to=probb@iol.unh.edu \
--cc=aconole@redhat.com \
--cc=ci@dpdk.org \
--cc=david.marchand@redhat.com \
--cc=shaibran@amazon.com \
--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).