New issue
Advanced search Search tips

Issue 696706 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Task

Blocking:
issue 491339
issue 762051
issue 819863



Sign in to add a comment

Bugdroid should be a gerrit plugin

Project Member Reported by aga...@chromium.org, Feb 27 2017

Issue description

Bugdroid today works like this:
* Download bitmaps keeping track of every CL it has ever seen
* Poll gerrit for changes that have been Submitted recently
* Update bitmap to include those changes
* Upload bitmap
* Restart

Instead, it should work like this:
* Configure a plugin like git-numberer
* Listen to RefOperationValidationLister.onRefOperation and OnSubmitValidationListener.preBranchUpdate events

This will also let us do cool things like
* POST to monorail when a change is uploaded as well as when it is landed
* Not have to store any state ever at all thank god
* Delete bugdroid, delete bugdroid's crazy appengine app, delete bugdroid's luci config, never have to worry about configuring bugdroid to watch a new repo ever again
* Expose bug state directly in the PolyGerrit UI if we really want to
 
Cc: -andyb...@chromium.org
Labels: Pri-2
Owner: andyb...@chromium.org
Status: Assigned (was: Untriaged)
Owner: ----
Status: Available (was: Assigned)

Comment 3 by aga...@chromium.org, Sep 11 2017

Blocking: 762051

Comment 4 by aga...@chromium.org, Sep 11 2017

 Issue 750950  has been merged into this issue.
Labels: -Milestone-Afterglow
Removing Milestone-Afterglow, as it has ceased to have meaning. More refined milestones may be added back in the near future.
Blocking: 491339
Labels: -Proj-Gerrit-Migration Type-Task

Comment 8 by aga...@chromium.org, Mar 19 2018

For posterity, here's code that enables authentication from googlesource gerrit plugins: http://shortn/_2ONVDJg3ea

Comment 9 by aga...@chromium.org, Mar 27 2018

Blocking: 819863
Update: Gerrit team told us to not write this as a plugin, but instead use Cloud PubSub on the chromium host.

We started some work for this as a hackathon project: https://chromium.googlesource.com/infra/experimental/+/master/opstoberfest2018-hackathon/bugdroid2
Cc: j...@chromium.org
Cc: zhangtiff@chromium.org
Cc: -zhangtiff@chromium.org
(Sorry for the noise, trying to replicate a bug described in monorail:4467)
Cc: zhangtiff@chromium.org
Cc: ajp@chromium.org

Sign in to add a comment