New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 0 users
Status: WontFix
Closed: Jun 2014
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Sign in to add a comment
"Revert Patchset" button on does not add TBR and NOTRY to the revert patch
Project Member Reported by, Jun 12 2014 Back to list
Steps to reproduce:
1) Find a patch you want to revert on
2) Click on the "Revert Patchset" button.

The reverting patch has TBR=<original-patch-owner> and NOTRY=true. The CQ commits the reverting patch.

The reverting patch does not have TBR and NOTRY. The CQ attempts to commit the reverting patch, but cannot, because of no LGTM from an owner.

See for example. This started happening today (Jun 12, 2014) and happened for both patches that I attempted to revert.

Workaround: Manually edit the description of the reverting patch to add the NOTRY and TBR.
+jrobbins who is in charge of
Comment 2 by, Jun 13 2014
Thank you for letting us know? Is there a reason to revert a revision using the CQ? I've been using drover to revert it, for some reason CQ had never been instantaneous in the past, even when revert patch set worked. 
Seems to be fixed now. Not sure what fixed it.
Comment 4 by, Jun 13 2014
Status: WontFix
There is actually logic in Rietveld that conditionally adds the TBR no NOTRY only for recent patches.  The reason is that a concern was raised about people being able to revert only changes on the sly.  So, what you saw was working as intended, but I can see how it could be confusing.
Sign in to add a comment