Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(262)

Issue 23093011: Create tryserver appengine app. (Closed)

Created:
7 years, 4 months ago by agable
Modified:
7 years, 4 months ago
CC:
chromium-reviews, cmp-cc_chromium.org
Visibility:
Public.

Description

Create tryserver appengine app. This appengine app is a step towards decentralized job control. It accepts POSTs of json blobs describing buildbot jobs to its 'push' endpoint. Then, requests to the 'pull' endpoint will be served not-yet-picked-up jobs. The poller can then POST to the 'accept' endpoint to indicate that it has successfully picked up and started a job. The 'peek' endpoint serves to inspect the job queue without marking any jobs as potentially taken. This is intended to interact with 'git try' on the client side to push blobs to the app, and a new buildbot poller and scheduler to consume blobs and start jobs. R=cmp@google.com Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=219250

Patch Set 1 #

Total comments: 12

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+189 lines, -0 lines) Patch
A chromium-jobqueue/.gitignore View 1 chunk +2 lines, -0 lines 0 comments Download
A chromium-jobqueue/LICENSE View 1 1 chunk +27 lines, -0 lines 0 comments Download
A chromium-jobqueue/app.py View 1 1 chunk +130 lines, -0 lines 0 comments Download
A chromium-jobqueue/app.yaml View 1 1 chunk +30 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
agable
PTAL. Now in the new-and-improved location. cmp@ on review because he started taking a look ...
7 years, 4 months ago (2013-08-21 13:43:06 UTC) #1
cmp
https://codereview.chromium.org/23093011/diff/1/chromium-jobqueue/app.py File chromium-jobqueue/app.py (right): https://codereview.chromium.org/23093011/diff/1/chromium-jobqueue/app.py#newcode1 chromium-jobqueue/app.py:1: # Copyright (c) 2012 The Chromium Authors. All rights ...
7 years, 4 months ago (2013-08-22 04:44:12 UTC) #2
agable
https://codereview.chromium.org/23093011/diff/1/chromium-jobqueue/app.py File chromium-jobqueue/app.py (right): https://codereview.chromium.org/23093011/diff/1/chromium-jobqueue/app.py#newcode1 chromium-jobqueue/app.py:1: # Copyright (c) 2012 The Chromium Authors. All rights ...
7 years, 4 months ago (2013-08-22 13:43:42 UTC) #3
cmp_google
lgtm
7 years, 4 months ago (2013-08-23 03:10:45 UTC) #4
agable
7 years, 4 months ago (2013-08-23 07:43:35 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 manually as r219250 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698