My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 547: Sanity check ChangeIDs on push
19 people starred this issue and may be notified of changes. Back to list
Status:  Accepted
Owner:  ----


Sign in to add a comment
 
Project Member Reported by jaysoff...@gmail.com, Apr 27, 2010
Affected Version: 2.1.2.2

What steps will reproduce the problem?
Gerrit doesn't currently do any sanity checking of the ChangeID on pushes to a branch. A 
developer tried to submit a patch through gerrit, got a conflict, and then tried to deal with the 
conflict in git via merge. Somehow they ended up pushing two separate commits with the same 
ChangeID

What is the expected output? What do you see instead?
When a commit is being pushed to a branch (not to the special refs/for space) and the commit 
has a ChangeID footer:

Gerrit should block the push unless it knows of the ChangeID AND the change is open and 
approved/valid.

This should be an optional permission. I can envision a scenario were you might want to merge 
in a commit from an upstream source (also using gerrit) that has ChangeID unknown to the local 
gerrit installation.


Apr 27, 2010
#1 sop@google.com
(No comment was entered for this change.)
Status: Accepted
Apr 27, 2010
Project Member #2 jaysoff...@gmail.com
I got as far as I could before I ran into a wall https://review.source.android.com/14447
Feb 17, 2012
Project Member #3 nas...@codeaurora.org
Is this still an issue in 2.2.2 and/or tip?
Sign in to add a comment

Powered by Google Project Hosting