Bug 9311 - patchtest: track last poll series' event inside patchtest so we do not depend on patchwork git
Summary: patchtest: track last poll series' event inside patchtest so we do not depend...
Status: RESOLVED FIXED
Alias: None
Product: Patchwork/Patchtest
Classification: Yocto Project Subprojects
Component: Patchtest (show other bugs)
Version: 2.2
Hardware: x86 Multiple
: High enhancement
Target Milestone: 2.2 M1
Assignee: Leonardo Sandoval Gonzalez
QA Contact: David Rodriguez
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2016-03-18 17:25 UTC by Leonardo Sandoval Gonzalez
Modified: 2016-06-20 13:35 UTC (History)
3 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-03-18 17:25:35 UTC
Currently, the system polls events using the patchwork git command. The latter, uses a file .git-pw.oe-core.poll.timestamp to store last polled series event (including the timestamp) and patchtest uses this timestamp to poll new events. This is fine if the .git-pw.oe-core.poll.timestamp is always present, but it may be the case that it is removed and next patchtest will poll ALL series events, doing multiple POST requests. The system should have a logic to avoid the latter.
Comment 1 Leonardo Sandoval Gonzalez 2016-04-06 20:31:51 UTC
This is an important feature that we should be done before release.
Comment 2 Leonardo Sandoval Gonzalez 2016-04-22 22:00:53 UTC
The following logic has been added:

if the timestamp file is not present:
    - poll all events, and take last event's timestamp and store it on the
      timestamp file
    - do not trigger patchtest
else
    - poll latest events, take last event's timestamp and update the 
      timestamp file
    - trigger patchtest with latest events