Bug 9945 - patchtest-oe: check pull request series against git repository
Summary: patchtest-oe: check pull request series against git repository
Status: RESOLVED OBSOLETE
Alias: None
Product: Patchwork/Patchtest
Classification: Yocto Project Subprojects
Component: Patchtest Testsuite (show other bugs)
Version: 2.3
Hardware: x86 Multiple
: Medium enhancement
Target Milestone: 2.5
Assignee: Leonardo Sandoval Gonzalez
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2016-07-14 21:12 UTC by Leonardo Sandoval Gonzalez
Modified: 2017-12-06 17:03 UTC (History)
2 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Leonardo Sandoval Gonzalez 2016-07-14 21:12:13 UTC
The check needs to go to each patch on the (patchwork) series and compare with the corresponding one on the git repository. If payload differ (or checksums), user should be notified and a v2 series must be sent.

This task was originally discussed on https://bugzilla.yoctoproject.org/show_bug.cgi?id=8648#c5
Comment 1 Paul Eggleton 2016-08-03 23:35:27 UTC
We'll probably need to have a discussion with the community about how to handle this issue before we go ahead. Right now we don't officially place any restrictions on people updating the branch and it's relatively common practice, so we should establish whether we want to change that before we add this check.
Comment 2 Leonardo Sandoval Gonzalez 2016-10-27 21:43:25 UTC
As a first approach, lets see how often this practice is done by the community (sending a pull-requests and a bit later the branch is modified and no v2 is sent). Once we have this info, we can talk to the community about it.
Comment 3 Leonardo Sandoval Gonzalez 2017-06-15 19:26:52 UTC
At 2.4, the focus is on the python 2 to python 3 migration, tinfoil usage and stabilization of these two. So moving it to 2.5
Comment 4 Leonardo Sandoval Gonzalez 2017-11-13 18:20:43 UTC
patchtest in production executes in an isolated space (no network), thus there is no way we can do this check.