Issue 180: Don't send "failed to submit" messages to everybody watching for submitted changes
Status:  Released
Owner:
Closed:  Oct 2012
Reported by code-rev...@gtempaccount.com, Sep 24, 2009
Reported by David Brown <dab@android.com> on Fri May 15 13:14:17 PDT 2009
Source: JIRA GERRIT-180

In my Gerrit settings I'm signed up to get email notification for
"Submitted Changes" in a bunch of projects.

But in addition to actual submissions, I also get the corresponding
"FAILED to submit" messages when there was a conflict, like this one:

   Date: Fri, 15 May 2009 10:48:54 -0700 (PDT)
   From: Mike LeBeau <mlebeau@android.com>
   To: Mike LeBeau <mlebeau@android.com>
   Cc: Bjorn Bringert <bringert@android.com>
   Cc: Android Search ML <android-search@google.com>
   Subject: [donut] Change 1697: (platform/vendor/google) Get the color value
    for the suggestion url color from a them...

   Change 1697 by Mike LeBeau FAILED to submit to donut.

   Error message:
   ....................................................
   Your change could not be merged due to a path conflict.

   Please merge (or rebase) the change locally and upload the resolution for
review.

   --
   To view visit https://android-git.corp.google.com/g/1697
   To unsubscribe, visit https://android-git.corp.google.com/g/settings

Could these please be sent only to the submitter (or maybe submitter +
reviewers), rather than everybody who's watching for submitted changes?
Sep 24, 2009
#1 code-rev...@gtempaccount.com
Update by Shawn Pearce <sop@google.com> on Wed May 20 18:41:18 PDT 2009
Sep 24, 2009
#2 code-rev...@gtempaccount.com
Comment by Shawn Pearce <sop@google.com> on Thu May 21 11:39:11 PDT 2009

Fixed by https://review.source.android.com/10033
Sep 24, 2009
#3 code-rev...@gtempaccount.com
Update by Shawn Pearce <sop@google.com> on Thu May 21 11:39:11 PDT 2009

Fixed in version 2.0.13.
Status: Fixed
Sep 25, 2009
#4 code-rev...@gtempaccount.com
(No comment was entered for this change.)
Labels: FixedIn-2.0.13
Oct 25, 2012
#5 sop@google.com
(No comment was entered for this change.)
Status: Released