New issue
Advanced search Search tips

Issue 728798 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 710981
Owner: ----
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: ----
Type: ----



Sign in to add a comment

Error uploading patch?

Project Member Reported by sky@chromium.org, Jun 1 2017

Issue description

I was uploading a new revision of a patch ( https://chromium-review.googlesource.com/c/519464 ) and git cl upload -t 'foo' produced weird output. I *think* it succeeded, but the output was concerning:

Here's the output:

> git cl upload -t merge
Running presubmit upload checks ...
Done processing /secondary/builds/build1/src/chrome/browser/ui/views/desktop_capture/desktop_media_picker_views.cc
Done processing /secondary/builds/build1/src/chrome/browser/ui/views/extensions/extension_install_dialog_view.cc
Done processing /secondary/builds/build1/src/chrome/browser/ui/views/extensions/extension_install_dialog_view.h
Done processing /secondary/builds/build1/src/chrome/browser/ui/views/intent_picker_bubble_view.cc
Done processing /secondary/builds/build1/src/chrome/browser/ui/views/payments/payment_request_sheet_controller.cc
Done processing /secondary/builds/build1/src/chrome/browser/ui/views/toolbar/extension_toolbar_menu_view.cc

Presubmit checks took 5.2s to calculate.

Presubmit checks passed.
 ash/system/tray/tray_details_view.cc                                  |   7 ++---
 chrome/browser/ui/views/desktop_capture/desktop_media_picker_views.cc |  10 ------
 chrome/browser/ui/views/extensions/extension_install_dialog_view.cc   |   6 ----
 chrome/browser/ui/views/extensions/extension_install_dialog_view.h    |   1 -
 chrome/browser/ui/views/intent_picker_bubble_view.cc                  |   2 +-
 chrome/browser/ui/views/payments/payment_request_sheet_controller.cc  |   2 +-
 chrome/browser/ui/views/toolbar/extension_toolbar_menu_view.cc        |   3 ++
 ui/message_center/views/message_center_view.cc                        |   7 +----
 ui/views/controls/scroll_view.cc                                      | 148 +++++++++++++++++++++++++++++++++++++++++++++++++++++---------------------------------
 ui/views/controls/scroll_view.h                                       |  53 +++++++++++++++++--------------
 ui/views/controls/scroll_view_unittest.cc                             |  48 ++++++++++++++++++++++++++++
 ui/views/view_unittest.cc                                             |  25 ---------------
 12 files changed, 177 insertions(+), 135 deletions(-)
remote: Processing changes: (\)
To https://chromium.googlesource.com/chromium/src.git
 ! [remote rejected]           5a805c8a563bf25b014896092a7f38034358e4eb -> refs/for/refs/heads/master%m=merge,notify=NONE (error in Gerrit backend)
error: failed to push some refs to 'https://chromium.googlesource.com/chromium/src.git'

Error after CL description prompt -- saving description to /usr/local/google/home/sky/.git_cl_description_backup

Failed to create a change. Please examine output above for the reason of the failure.
Hint: run command below to diagnose common Git/Gerrit credential problems:
  git cl creds-check


 
Mergedinto: 710981
Status: Duplicate (was: Unconfirmed)
Yeah, I've got the gerrit folks digging into why the system very rarely returns a 500 even though the operation succeeded. Thanks for providing another reproduction.

Sign in to add a comment