Why not something like:
Recheck-request: [attribute-list],[test-list]...
For example, then we can do:
Recheck-request: rebase=[identifier],....
where identifier is a branch specifier (or the word 'latest')?
I hadn't thought about the option of allowing branch specifiers. Agree that allowing a human correction process for the pw_maintainer_cli.py script choosing the wrong branch sounds helpful.
My original idea was offering 2 options (test original artifact, or re-apply on latest). Do we want to support for checking out to a specific commit and re-applying there? I figured that would not be worth it (too much of a niche case), but your comments are making me reconsider.
That lets us fixup if the branch picker script guessed a wrong branch.
Just spit-balling on syntax.
That said, I agree - if a rebase has been requested, all tests need to
be rerun. Maybe we should consider that the test labels should be added
with a run number or something? Or we could also include that the run
is a rerun. That way for labs that don't currently support the recheck
request framework, we can easily tell that they weren't re-tried.