Ryan Lane has submitted this change and it was merged. Change subject: Overhauling gerrit manifest to be a role class ......................................................................
Overhauling gerrit manifest to be a role class * gerrit::instance a function like the varnish::instance * made gerrit::proxy to accept parameters 'no_apache' and 'apache_ssl' * made gerrit::jetty with a ton of parameters for various config * new roles that document our Gerrit instances in a concise form: - gerrit::production : the main instance we all know about - gerrit::production::slave : unknown usage - gerrit::labs : similar to production, but with no SSL * database should already be created and user should have all rights * Removed a bunch of the globals from the gerrit templates, eliminated gerrit_config class Co-Authored-By: Antoine Musso <[email protected]> Change-Id: I3d8a9fd26e52b6efb36b55ec69062b5af7f149a5 --- M files/apache/sites/gerrit.wikimedia.org M manifests/gerrit.pp A manifests/role/gerrit.pp M manifests/site.pp D templates/gerrit/gerrit-user.cnf.erb D templates/gerrit/gerrit-user.sql.erb M templates/gerrit/gerrit.config.erb M templates/gerrit/hookconfig.py.erb M templates/gerrit/secure.config.erb 9 files changed, 178 insertions(+), 130 deletions(-) Approvals: Ryan Lane: Looks good to me, approved gerrit2: Verified -- To view, visit https://gerrit.wikimedia.org/r/13484 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3d8a9fd26e52b6efb36b55ec69062b5af7f149a5 Gerrit-PatchSet: 40 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Demon <[email protected]> Gerrit-Reviewer: Andrew Bogott <[email protected]> Gerrit-Reviewer: Demon <[email protected]> Gerrit-Reviewer: Ryan Lane <[email protected]> Gerrit-Reviewer: gerrit2 <[email protected]> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
