mbox series

[v3,0/2] Reduced checks API usage

Message ID 20240122234034.3883647-1-aconole@redhat.com (mailing list archive)
Headers
Series Reduced checks API usage |

Message

Aaron Conole Jan. 22, 2024, 11:40 p.m. UTC 
  As Jeremy Kerr reports, we aren't being as nice as we should be w.r.t.
API usage - especially around the checks API reporting.  While the
initial version *worked* for what we wanted, we are heavily using the
checks API, especially when we don't need to be.

This series should reduce the number of API calls we make into
patchwork with respect to the checks API specifically.

Additionally, we add a simple token to identify that the requests
coming in are from a patchwork robot project (so at least a site
admin can start identifying who is actually sending the requests).

Aaron Conole (2):
  treewide: Add a User Agent for CURL requests
  post_pw: Store submitted checks locally as well

 ci_mon             |  2 +-
 github_get_logs.sh |  8 ++++----
 github_mon         |  2 +-
 github_restart     |  6 ++++--
 jenkins_lib.sh     |  5 +++--
 post_pw.sh         | 43 ++++++++++++++++++++++++++++++++++++++-----
 pw_mon             | 14 +++++++-------
 series_db_lib.sh   | 25 +++++++++++++++++++++++++
 8 files changed, 83 insertions(+), 22 deletions(-)

Comments

Aaron Conole Jan. 23, 2024, 1:46 p.m. UTC | #1 
Aaron Conole <aconole@redhat.com> writes:

> As Jeremy Kerr reports, we aren't being as nice as we should be w.r.t.
> API usage - especially around the checks API reporting.  While the
> initial version *worked* for what we wanted, we are heavily using the
> checks API, especially when we don't need to be.
>
> This series should reduce the number of API calls we make into
> patchwork with respect to the checks API specifically.
>
> Additionally, we add a simple token to identify that the requests
> coming in are from a patchwork robot project (so at least a site
> admin can start identifying who is actually sending the requests).
>
> Aaron Conole (2):
>   treewide: Add a User Agent for CURL requests
>   post_pw: Store submitted checks locally as well

Series applied.

Thanks Ilya and Michael for quick turnaround for reviews.