11 Commits

Author SHA1 Message Date
Monty Taylor
47a361d596 Manage gerritbot ssh key with puppet
I'm not sure this really needs a longer commit message.

Change-Id: If0a7db1b4a988fd181753748d5fc9d26cf1bb906
2014-04-28 12:18:42 -07:00
Matthew Wagoner
4f433a021f Clean up of minor puppet-lint warnings.
Mostly documentation and parameterised class parameter complaints.

Change-Id: I5b3ffa4ad3f707f385165c2d86c891c6bd4c1ae0
Reviewed-on: https://review.openstack.org/16901
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Tested-by: Jenkins
2012-11-26 20:55:57 +00:00
Paul Belanger
57d13bacfd Additional puppet-lint reformatting
Change-Id: I49e63755b8b5bf60673cf37b62371b3f299e3bbe
Signed-off-by: Paul Belanger <paul.belanger@polybeacon.com>
Reviewed-on: https://review.openstack.org/14971
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Tested-by: Jenkins
2012-10-29 19:21:24 +00:00
Paul Belanger
bd74f69d00 Style Guide updates for puppet-lint
More fixes in preparation for our gate-ci-puppet-lint job.

Change-Id: I1cd3499e6cd958fbbacd01f3daef581a7b4d3217
Signed-off-by: Paul Belanger <paul.belanger@polybeacon.com>
Reviewed-on: https://review.openstack.org/13764
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Approved: James E. Blair <corvus@inaugust.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
2012-09-28 21:24:49 +00:00
Clark Boylan
00623af0c4 Ensure gerritbot is present and not latest.
Pip doesn't appear to be able to do a clean upgrade of gerritbot on
oneiric. It should remove the old versions of gerritbot after installing
the new versions but it does not. This confuses Puppet into thinking
that it need to reinstall gerritbot 0.1.2 on each puppet run which
restarts the IRC bot. To prevent this behavior ensure the package is
present and not latest.

Change-Id: I7729d126a7811b7e08541ed65ca48f4dc302ff6b
Reviewed-on: https://review.openstack.org/11955
Approved: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Tested-by: Jenkins
2012-08-24 22:04:08 +00:00
Clark Boylan
7398b4b0d0 Enable gerritbot logging to /var/log/gerritbot.
Configure gerritbot to log into /var/log/gerritbot.

Change-Id: I24f8602eaa0f779de9d455cda8ad692f932ebd39
Reviewed-on: https://review.openstack.org/11939
Approved: James E. Blair <corvus@inaugust.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
2012-08-24 20:05:23 +00:00
Clark Boylan
ce9cb08928 Correct typo in gerritbot manifests/init.pp.
gerritbot.confg should be gerritbot.config

Change-Id: I9532226a690a986ae8480c1cfbd89e23cf1f3ac1
Reviewed-on: https://review.openstack.org/11905
Approved: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Tested-by: Jenkins
2012-08-23 23:08:19 +00:00
Clark Boylan
97933d1cd2 Use upstream gerritbot.
Stop using the gerritbot script within the puppet tree. Use the
upstream gerritbot implementation from pypi instead.

Remove old gerritbot script, use pip to install upstream gerritbot,
and update init script to find new location, of the gerritbot script.

Change-Id: I62afe404b1c4d130901e54ca4e7857bb71b4cdd8
Reviewed-on: https://review.openstack.org/10676
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Tested-by: Jenkins
2012-08-23 22:48:34 +00:00
Monty Taylor
7d8c838038 Align all web server usage on apache module.
Change-Id: Idd712a8ee5ec81c6b88b7d3e2270dce4da254927
Reviewed-on: https://review.openstack.org/10838
Approved: Monty Taylor <mordred@inaugust.com>
Reviewed-by: Monty Taylor <mordred@inaugust.com>
Tested-by: Jenkins
2012-08-12 17:03:46 +00:00
Monty Taylor
dbd4da09a7 Fix all of the gerrit module problems.
Change-Id: I46001fd677bc9a3634c9860ec07438c326e908e3
2012-07-24 00:28:04 -05:00
Monty Taylor
e3e9aaba10 Split gerritbot into its own module.
Change-Id: Ia9034d87321624006ccf447b058ee828a97fcc7a
2012-07-23 10:33:59 -05:00