tree 1d8f6c70d0d163366bc3780fe464317a77439b01
parent f2f2feb438586bbfb157e627b34a11103e3c9cc5
author Andrew Bonventre <andybons@golang.org> 1518571890 -0500
committer Andrew Bonventre <andybons@golang.org> 1518631310 +0000

cmd/gerritbot: use conditional requests for PRs from GitHub

GerritBot retrieves all open PRs each time the maintner corpus
is updated since maintner doesn’t support PRs yet and some
additional info is needed from GitHub (the PR branch’s HEAD
SHA value in particular).

Currently the GitHub client doesn’t take advantage of conditional
requests, which causes us to needlessly take a hit in our API
quota, counting against all maintner users.

Use the returned Etag response header to make conditional requests
for PRs so that they don’t count against our rate limit unless
a change to the PR has been observed.

Change-Id: Ie984e94cc7cc4d254cbd7c7f4e92ca3f5bf688ba
Reviewed-on: https://go-review.googlesource.com/93835
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>