So I was looking at our git log. I can fit two, maybe three commits if I'm lucky. It's terrible. We have all this redundant information. We have both "Reviewed-on" and "Change-Id" lines. Those only point to the same resource. The "Tested-by" is useless. It's always Jenkins who tests changes. Then you have a "Reviewed-by" for every person who ever bikeshed on said patch. Since Paul has to say something on every single patch, he always has his own line in almost every commit.
"Reviewed-by" is not the same as "Acked-by". I've seen acked-by used to let linux folks know that some maintainer/big guy with more credentials than the author already likes the patch. Though Acked-by is not necessary, even for linux. Come to think of it, the only piece of information needed to find a commit on our gerrit is the hash. All the other lines, except for the "Signed- off-by" are redundant. "Reviewed-on" and "Change-Id" will take you to the same place as the commit hash. All the crap added by gerrit is redundant.
Now add a patch that comes from our Chromium friends, and you've doubled the size of said pork. Is it convenient to have all that information in the commit message? Maybe. Is it necessary? Definitely not. Those lines are useful for tracking a patch, but once it's +2'd and submitted, gerrit should strip out all that pork. We might not even use gerrit anymore a few years down the road. Adding all this crap is pointless.
It's time to drop the crap.
Alex