Dzahn has uploaded a new change for review. https://gerrit.wikimedia.org/r/170273
Change subject: phab: move defines into autoload layout ...................................................................... phab: move defines into autoload layout defines should also be in their own files like classes are ERROR: phabricator::extension not in autoload module layout ERROR: phabricator::redirector not in autoload module layout http://puppet-lint.com/checks/autoloader_layout/ also fixes some lint warnins about aligned errors Change-Id: I4745cc5e02ef8bbb270fae22f50f690206d743ff --- M modules/phabricator/manifests/init.pp M modules/phabricator/manifests/monitoring.pp M modules/phabricator/manifests/phd.pp 3 files changed, 6 insertions(+), 23 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/operations/puppet refs/changes/73/170273/1 diff --git a/modules/phabricator/manifests/init.pp b/modules/phabricator/manifests/init.pp index 3613591..5e67401 100644 --- a/modules/phabricator/manifests/init.pp +++ b/modules/phabricator/manifests/init.pp @@ -259,7 +259,7 @@ class { 'phabricator::phd': settings => $phab_settings, - before => Service['phd'], + before => Service['phd'], } # This needs to become Upstart managed @@ -276,20 +276,3 @@ } } -define phabricator::extension($rootdir='/') { - file { "${rootdir}/phabricator/src/extensions/${name}": - ensure => link, - target => "${rootdir}/extensions/${name}", - } -} - -define phabricator::redirector($mysql_user, $mysql_pass, $mysql_host, $rootdir='/') { - file { "${rootdir}/phabricator/support/preamble.php": - source => 'puppet:///modules/phabricator/preamble.php', - require => File["${rootdir}/phabricator/support/redirect_config.json"] - } - - file { "${rootdir}/phabricator/support/redirect_config.json": - content => template('phabricator/redirect_config.json.erb') - } -} diff --git a/modules/phabricator/manifests/monitoring.pp b/modules/phabricator/manifests/monitoring.pp index 605459b..3ff2c1b 100644 --- a/modules/phabricator/manifests/monitoring.pp +++ b/modules/phabricator/manifests/monitoring.pp @@ -2,7 +2,7 @@ nrpe::monitor_service { 'check_phab_taskmaster': description => 'check if phabricator taskmaster is running', - nrpe_command => "/usr/lib/nagios/plugins/check_procs -w 10:30 -c 10:30 --ereg-argument-array 'PhabricatorTaskmasterDaemon'", + nrpe_command => '/usr/lib/nagios/plugins/check_procs -w 10:30 -c 10:30 --ereg-argument-array PhabricatorTaskmasterDaemon', } monitor_service { 'phabricator-https': diff --git a/modules/phabricator/manifests/phd.pp b/modules/phabricator/manifests/phd.pp index 0576162..18e1893 100644 --- a/modules/phabricator/manifests/phd.pp +++ b/modules/phabricator/manifests/phd.pp @@ -4,14 +4,14 @@ file { $settings['phd.pid-directory']: ensure => 'directory', - owner => $settings['phd.user'], - group => 'phd', + owner => $settings['phd.user'], + group => 'phd', } file { $settings['phd.log-directory']: ensure => 'directory', - owner => $settings['phd.user'], - group => 'phd', + owner => $settings['phd.user'], + group => 'phd', } group { 'phd': -- To view, visit https://gerrit.wikimedia.org/r/170273 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I4745cc5e02ef8bbb270fae22f50f690206d743ff Gerrit-PatchSet: 1 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Dzahn <dz...@wikimedia.org> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits