If a gerrit hook blocks none of the other hooks queued behind it run.
Give each hook subprocess 10 minutes to run otherwise timeout and kill
that process.
Change-Id: Ic5e1243e08088dbf0c5bff49e7fc05bcb867819b
Reviewed-on: https://review.openstack.org/11475
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
Change the text color to black to increase the contrast and improve
legibility.
Change-Id: I641c30904184711c239741076a549e0d2b4761c7
Reviewed-on: https://review.openstack.org/10285
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: linuxjedi <andrew@linuxjedi.co.uk>
Approved: Monty Taylor <mordred@inaugust.com>
Tested-by: Jenkins
Additionally, the file installation of the scripts wasn't working,
so the sync script wasn't actually getting installed. This moves
the underlying scripts to be installed by the gerrit module,
because it owns /usr/local/gerrit/scripts, and then manages the
gerrit hooks which call those scripts in the openstack_project
class, since that's where the config choice to enable those
functions really should live.
Change-Id: I54fb9edd9fb0c634d8d9de4e57f9ddad6af63a99
Launchpad integration and the OpenStack branding files are really
more about the OpenStack specific install of Gerrit than they are
about any installation of gerrit. Both of these are moved to the
openstack_project module.
Change-Id: I8b281aa5cb751a8023c2101c45146a3aca5f90f3