Issue 547: Sanity check ChangeIDs on push
Status:  Accepted
Owner: ----
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?