My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 2502: Committer replaced with “Gerrit Code Review”
1 person starred this issue and may be notified of changes. Back to list
Status:  WontFix
Owner:  ----
Closed:  Mar 2014


Sign in to add a comment
 
Reported by jhabib1...@gmail.com, Feb 26, 2014
************************************************************
***** NOTE: THIS BUG TRACKER IS FOR GERRIT CODE REVIEW *****
***** DO NOT SUBMIT BUGS FOR CHROME, ANDROID, INTERNAL *****
***** ISSUES WITH YOUR COMPANY'S GERRIT SETUP, ETC.    *****
***** THOSE ISSUE BELONG IN DIFFERENT ISSUE TRACKERS!  *****
************************************************************

Affected Version: 2.8.1

What steps will reproduce the problem?
1. submit type Rebase if necessary
2. Gerrit decides to rebase (instead of merging)
3. the committer is replaced with “Gerrit Code Review” 

What is the expected output? What do you see instead?
I expect the committer name instead we have "Gerrit Code Review"

Please provide any additional information below.

Mar 3, 2014
Project Member #1 bklarson@gmail.com
This is by design.  If Gerrit is rebasing a change, we set the committer to Gerrit to make it clear who did the rebase.  Just like if Developer B rebased and reuploaded Developer A's change, the committer is Developer B.  This information is especially important if there were any issues caused by the rebase.
Status: WontFix
Mar 3, 2015
#2 paf...@gmail.com
Sorry, but this is not acceptable. I do not mind gerrit to make changes into committer e-mail field, as long as the new value is a valid e-mail within organisation (not current case), like gerrit administrator. 
Actually this is request of our IT tech department, If we cannot meet their request, we might be forced to shut down the gerrit.
Mar 3, 2015
#3 paf...@gmail.com
Agggh ... Ignore my previous comment.
Sign in to add a comment

Powered by Google Project Hosting