Last modified: 2013-02-20 21:46:12 UTC
The Gerrit init script look for /etc/default/gerritcodereview. Both our debian package and the puppet manifests install a /etc/default/gerrit which is thus never loaded rendering the init script useless. Both our Gerrit package and the puppet manifest provide the same content. To avoid changing upstream init script, we would want to update our Gerrit debian package to isntall the default as gerritcodereview and then to update our puppet manifest to use the same name /etc/default/gerritcodereview. Or maybe strip the puppet part since it is redundant with the package.
Yeah, this was changed from gerrit to gerritcodereview at some point. I updated the init script when we upgraded, but the puppet manifest was never changed. Have done that now in Gerrit change #50016.
This is fixed now.