Ottomata has uploaded a new change for review. https://gerrit.wikimedia.org/r/322668
Change subject: Puppetization for statsv args added in https://gerrit.wikimedia.org/r/#/c/321911/ ...................................................................... Puppetization for statsv args added in https://gerrit.wikimedia.org/r/#/c/321911/ Previous attempt accidentally included mariadb submodule change Bug: T150765 Change-Id: I4e6bd74b3d42ac5dd7fd8181f0d06743667a3814 --- M modules/webperf/manifests/statsv.pp R modules/webperf/templates/statsv.service.erb 2 files changed, 12 insertions(+), 7 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/operations/puppet refs/changes/68/322668/1 diff --git a/modules/webperf/manifests/statsv.pp b/modules/webperf/manifests/statsv.pp index 1a09bff..5f49f95 100644 --- a/modules/webperf/manifests/statsv.pp +++ b/modules/webperf/manifests/statsv.pp @@ -5,7 +5,12 @@ class webperf::statsv { include ::webperf - require_package('python-pykafka') + require_package('python-kafka') + + # These are rendered in statsv.service + $kafka_config = kafka_config('analytics') + $kafka_brokers = $kafka_config['brokers']['string'] + $statsd = hiera('statsd') package { 'statsv': ensure => present, @@ -13,14 +18,14 @@ } file { '/lib/systemd/system/statsv.service': - ensure => present, - source => 'puppet:///modules/webperf/statsv.service', + ensure => 'present', + content => template('webperf/statsv.service.erb'), require => Package['statsv'], - notify => Service['statsv'], } service { 'statsv': - ensure => running, - provider => 'systemd', + ensure => 'running', + provider => 'systemd', + subscribe => File['/lib/systemd/system/statsv.service'], } } diff --git a/modules/webperf/files/statsv.service b/modules/webperf/templates/statsv.service.erb similarity index 80% rename from modules/webperf/files/statsv.service rename to modules/webperf/templates/statsv.service.erb index 20300e0..617f79f 100644 --- a/modules/webperf/files/statsv.service +++ b/modules/webperf/templates/statsv.service.erb @@ -4,7 +4,7 @@ [Service] WorkingDirectory=/srv/deployment/statsv/statsv -ExecStart=/usr/bin/python /srv/deployment/statsv/statsv/statsv.py +ExecStart=/usr/bin/python /srv/deployment/statsv/statsv/statsv.py --brokers <%= @kafka_brokers %> --statsd <%= @statsd %> User=nobody Restart=always WatchdogSec=5 -- To view, visit https://gerrit.wikimedia.org/r/322668 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I4e6bd74b3d42ac5dd7fd8181f0d06743667a3814 Gerrit-PatchSet: 1 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Ottomata <[email protected]> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
