system-config/modules/openstack_project/templates/gerrit_patchset-created.erb

#!/bin/sh

# Use timeout to kill any process running longer than 10 minutes.
timeout -k 2m 10m /usr/local/bin/update-blueprint patchset-created "$@"
timeout -k 2m 10m /usr/local/bin/update-bug patchset-created "$@"
timeout -k 2m 10m /usr/local/bin/notify-impact patchset-created "$@" --impact SecurityImpact --dest-address 'openstack-security@lists.openstack.org'
<% if @trivial_rebase_role_id != "" -%>
timeout -k 2m 10m /usr/local/bin/trivial-rebase \
    patchset-created \
    --whitespace \
    --private-key-path=<%= @ssh_host_key %> \
    --role-user=<%= @trivial_rebase_role_id %> "$@"
<% end -%>
<% if @ssh_welcome_rsa_key_contents != "" -%>
timeout -k 2m 10m /usr/local/bin/welcome-message patchset-created \
    --verbose --ssh-user=welcome-message \
    --ssh-key=/home/gerrit2/review_site/etc/ssh_welcome_rsa_key "$@"
<% end -%>
Put blueprint updates into production. Call the blueprint update script from the patchset created hook. Add blueprint comment links. Also, add changeid comment links. This change depends on I62f965ca7f14f589e3b299ea46729efb68abd06f in openstack-ci. Change-Id: I62b0644ffea4b4d7f5a3407fc9e776b28e200278 Reviewed-on: https://review.openstack.org/254 Reviewed-by: Monty Taylor <mordred@inaugust.com> Tested-by: Jenkins 2011-08-16 22:05:14 +00:00			`#!/bin/sh`

Use timeout(1) to timeout and kill gerrit hooks. 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 2012-08-16 09:09:00 -07:00			`# Use timeout to kill any process running longer than 10 minutes.`
Consume jeepyb. Instead of keeping many of these files directly in the tree, use them from the out-of-tree jeepyb project, which makes them easier to consume for other people who are not us. Change-Id: Id704f2e17dd80709ef63cbbf2c5475a08a835f91 Reviewed-on: https://review.openstack.org/16777 Reviewed-by: Clark Boylan <clark.boylan@gmail.com> Reviewed-by: James E. Blair <corvus@inaugust.com> Approved: James E. Blair <corvus@inaugust.com> Tested-by: Jenkins 2012-11-22 10:45:10 -08:00			`timeout -k 2m 10m /usr/local/bin/update-blueprint patchset-created "$@"`
			`timeout -k 2m 10m /usr/local/bin/update-bug patchset-created "$@"`
Change SecurityImpact destination address. Change the destination address for SecurityImpact email notifications to openstack-security@lists.openstack.org. This is being done at the request of bdpayne. Change-Id: If548c9ad74bb5e4f53f42ac737150a32ff54006e Reviewed-on: https://review.openstack.org/27304 Reviewed-by: bdpayne <bdpayne@acm.org> Reviewed-by: Jeremy Stanley <fungi@yuggoth.org> Reviewed-by: Spencer Krum <krum.spencer@gmail.com> Reviewed-by: Elizabeth Krumbach <lyz@princessleia.com> Approved: James E. Blair <corvus@inaugust.com> Reviewed-by: James E. Blair <corvus@inaugust.com> Tested-by: Jenkins 2013-04-22 16:06:09 -07:00			`timeout -k 2m 10m /usr/local/bin/notify-impact patchset-created "$@" --impact SecurityImpact --dest-address 'openstack-security@lists.openstack.org'`
Actually disable trivial_rebase_hook The test in the erb file was bogus. Change-Id: I9790f9f7bb25f105d9875a34f0451ef373f61dca 2014-04-28 16:57:51 -07:00			`<% if @trivial_rebase_role_id != "" -%>`
Consume jeepyb. Instead of keeping many of these files directly in the tree, use them from the out-of-tree jeepyb project, which makes them easier to consume for other people who are not us. Change-Id: Id704f2e17dd80709ef63cbbf2c5475a08a835f91 Reviewed-on: https://review.openstack.org/16777 Reviewed-by: Clark Boylan <clark.boylan@gmail.com> Reviewed-by: James E. Blair <corvus@inaugust.com> Approved: James E. Blair <corvus@inaugust.com> Tested-by: Jenkins 2012-11-22 10:45:10 -08:00			`timeout -k 2m 10m /usr/local/bin/trivial-rebase \`
Gerrit Trivial Rebase Detection Adds trivial_rebase.py for bug 881184. * modules/openstack_project/files/gerrit/scripts/trivial_rebase.py: A modified version of the contrib/trivial_rebase.py included in upstream Gerrit's git tree (we'll try to get individual patches submitted upstream soon): git clone https://gerrit.googlesource.com/gerrit * modules/openstack_project/files/gerrit/patchset-created: Moved to modules/openstack_project/templates/gerrit_patchset-created.erb and added an entry for the trivial_rebase.py script with variables for its command-line options sourced from gerrit.pp and review{,_dev}.pp. * modules/openstack_project/manifests/gerrit.pp: Changed the file definition for /home/gerrit2/review_site/hooks/patchset-created to use a template, adding the ssh_host_key and trivial_rebase_role_id variables it requires. Added a file entry for trivial_rebase.py as well, since we're putting it in modules/openstack_project instead of modules/gerrit where the existing scripts reside (we'll eventually want to move all of modules/gerrit/files/scripts out of there at some point, but not now). * modules/openstack_project/manifests/review_dev.pp: Override the trivial_rebase_role_id variable to trivial-rebase@review-dev.o.o on review-dev. * modules/openstack_project/manifests/review.pp: Override the trivial_rebase_role_id variable to trivial-rebase@review.o.o on review. Change-Id: I941f15525c72b84708ae1de6832834eb53ab6863 Reviewed-on: https://review.openstack.org/12373 Reviewed-by: Clark Boylan <clark.boylan@gmail.com> Reviewed-by: James E. Blair <corvus@inaugust.com> Approved: James E. Blair <corvus@inaugust.com> Tested-by: Jenkins 2012-09-05 17:17:53 +00:00			`patchset-created \`
Gerrit Whitespace Change Detection Consider whitespace changes significant enough not to reapply code reviews, but still comment if that's all which changed between patchsets. This addresses bug 1057506. * modules/openstack_project/files/gerrit/scripts/trivial_rebase.py (GetPatchId): Add a flag called consider_whitespace, but defaulting to False so as to preserve default behavior of the module. Add conditional behavior to replace all spaces and tabs with percent signs before calculating the patch-id hash. (Main): Add a --whitespace command-line option to turn on whitespace change checking. If enabled and if normal GetPatchId calls return a match, re-run with consider_whitespace set to True and apply a comment to the new patchset in Gerrit if the result is non-matching. * modules/openstack_project/templates/gerrit_patchset-created.erb: Add --whitespace to the trivial_rebase.py invocation, enabling whitespace checking. Change-Id: I89c479614a637717cf515a5d3f6d03b5f7830581 Reviewed-on: https://review.openstack.org/13775 Reviewed-by: James E. Blair <corvus@inaugust.com> Reviewed-by: Clark Boylan <clark.boylan@gmail.com> Reviewed-by: Paul Belanger <paul.belanger@polybeacon.com> Approved: Monty Taylor <mordred@inaugust.com> Reviewed-by: Monty Taylor <mordred@inaugust.com> Tested-by: Jenkins 2012-09-27 18:15:49 +00:00			`--whitespace \`
Clean up some variable references Change-Id: Id8b9f3acb742f8ebedb3818e283082432b83359c 2014-04-17 17:51:50 -07:00			`--private-key-path=<%= @ssh_host_key %> \`
			`--role-user=<%= @trivial_rebase_role_id %> "$@"`
disable the jeepyb trivial rebase hook on review-dev.o.o This change is for the pending gerrit upgrade. Gerrit 2.8 has built-in trivial rebase detection[1]. This change disables the jeepyb trivial rebase hook on review-dev.o.o so so we can use and test gerrit's built-in rebase detector instead. [1] https://gerrit-review.googlesource.com/Documentation/config-labels.html#label_copyAllScoresOnTrivialRebase and https://gerrit-review.googlesource.com/Documentation/config-labels.html#label_copyAllScoresIfNoCodeChange Closes-Bug: #1273959 Partial-Bug: #1082781 Change-Id: Ia14d8ee86b9d59b6f79ee873231bb0d13b7f43d2 2014-01-28 22:23:38 -08:00			`<% end -%>`
Clean up some variable references Change-Id: Id8b9f3acb742f8ebedb3818e283082432b83359c 2014-04-17 17:51:50 -07:00			`<% if @ssh_welcome_rsa_key_contents != "" -%>`
Enable Welcome Message to post review comments After extensive fixing from Jeremey and the Infrastructure team, welcome message now detects newbies correctly. The code has been run for a week in dry-run mode, and has been verified to produce no false positives. This patch removes the dry run option from the hook, to enable welcome message to post welcoming messages on reviews from first- time contributors. Change-Id: I5dd072d36480294eefc23aa6376ae0a686621b08 2014-03-13 09:58:13 +11:00			`timeout -k 2m 10m /usr/local/bin/welcome-message patchset-created \`
Update patchset-created hook for Gerrit welcome * .../templates/gerrit_patchset-created.erb: The SSH user and key for the welcome-message hook script are now options instead of positional parameters. Also, the user is a short name not a full E-mail address. Change-Id: Ibcb70068db89ec75a4b284340206718fd9d10e96 2014-03-18 02:32:08 +00:00			`--verbose --ssh-user=welcome-message \`
			`--ssh-key=/home/gerrit2/review_site/etc/ssh_welcome_rsa_key "$@"`
Install and use the Gerrit welcome-message keypair Change-Id: I7c23112664e43713f8dfc4bd0e4b9aec46b71500 2014-01-31 21:49:09 +00:00			`<% end -%>`