New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 700630 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocked on:
issue 700628

Blocking:
issue 700627
issue 700639



Sign in to add a comment

Create a reverting CL for a given commit

Project Member Reported by st...@chromium.org, Mar 11 2017

Issue description

For a given commit, create a reverting CL with a reason.

API: /api/<issue>/<patchset>/revert

Request: A POST request with the following parameters:
* xsrf_token
* revert_reason
* revert_cq: set it to 1 will automatically commit the CL through CQ

Response: maybe the issue id/url of the reverting CL
https://cs.chromium.org/chromium/infra/appengine/chromium_rietveld/codereview/revert_patchset.py?l=359

 

Comment 1 by st...@chromium.org, Mar 11 2017

Blockedon: 700628
Labels: Milestone-PreAutoRevert

Comment 2 by st...@chromium.org, Mar 11 2017

Blocking: 700639

Comment 3 by st...@chromium.org, Mar 13 2017

Owner: robert...@chromium.org
Status: Assigned (was: Available)

Comment 4 by st...@chromium.org, Mar 13 2017

Owner: st...@chromium.org

Comment 5 by st...@chromium.org, Mar 14 2017

Description: Show this description
Project Member

Comment 6 by bugdroid1@chromium.org, Mar 14 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/34f7cae7798e0fa82273eb2d8cd14bc1e71727d2

commit 34f7cae7798e0fa82273eb2d8cd14bc1e71727d2
Author: Shuotao Gao <stgao@chromium.org>
Date: Tue Mar 14 18:28:04 2017

[Findit] Refactor Rietveld and CodeReview APIs.

BUG= 700630 

Change-Id: I8c8151ebf611f744f62d57794b2f2646602707ec
Reviewed-on: https://chromium-review.googlesource.com/454896
Commit-Queue: Shuotao Gao <stgao@chromium.org>
Reviewed-by: Roberto Carrillo <robertocn@chromium.org>
Reviewed-by: Jeffrey Li <lijeffrey@chromium.org>

[modify] https://crrev.com/34f7cae7798e0fa82273eb2d8cd14bc1e71727d2/appengine/findit/waterfall/send_notification_for_culprit_pipeline.py
[modify] https://crrev.com/34f7cae7798e0fa82273eb2d8cd14bc1e71727d2/appengine/findit/waterfall/test/send_notification_for_culprit_pipeline_test.py
[modify] https://crrev.com/34f7cae7798e0fa82273eb2d8cd14bc1e71727d2/appengine/findit/infra_api_clients/codereview/rietveld.py
[add] https://crrev.com/34f7cae7798e0fa82273eb2d8cd14bc1e71727d2/appengine/findit/infra_api_clients/codereview/codereview_util.py
[add] https://crrev.com/34f7cae7798e0fa82273eb2d8cd14bc1e71727d2/appengine/findit/infra_api_clients/codereview/test/codereview_util_test.py
[modify] https://crrev.com/34f7cae7798e0fa82273eb2d8cd14bc1e71727d2/appengine/findit/infra_api_clients/codereview/codereview.py
[modify] https://crrev.com/34f7cae7798e0fa82273eb2d8cd14bc1e71727d2/appengine/findit/infra_api_clients/codereview/test/rietveld_test.py

Project Member

Comment 7 by bugdroid1@chromium.org, Mar 14 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/bbf96c5b9b444ee903129312211c6bf1fc424e1b

commit bbf96c5b9b444ee903129312211c6bf1fc424e1b
Author: Shuotao Gao <stgao@chromium.org>
Date: Tue Mar 14 19:29:10 2017

[Findit] Add the API definition to revert a CL on Rietveld.

BUG= 700630 

Change-Id: I33694d6950db45eb10280da77d03aa300eb5283e
Reviewed-on: https://chromium-review.googlesource.com/454126
Commit-Queue: Shuotao Gao <stgao@chromium.org>
Reviewed-by: Jeffrey Li <lijeffrey@chromium.org>

[modify] https://crrev.com/bbf96c5b9b444ee903129312211c6bf1fc424e1b/appengine/findit/infra_api_clients/codereview/codereview.py
[modify] https://crrev.com/bbf96c5b9b444ee903129312211c6bf1fc424e1b/appengine/findit/infra_api_clients/codereview/rietveld.py

Comment 9 by st...@chromium.org, Mar 15 2017

Status: Fixed (was: Assigned)

Sign in to add a comment