Faidon Liambotis has uploaded a new change for review.
https://gerrit.wikimedia.org/r/282357
Change subject: Remove statsdlb, unreferenced now
......................................................................
Remove statsdlb, unreferenced now
We've replaced it with statsd-proxy.
Change-Id: I782ce4cb9d312f5d279aad279ee786f51f871d2e
---
M manifests/role/statsd.pp
D modules/statsdlb/manifests/init.pp
2 files changed, 0 insertions(+), 63 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/operations/puppet
refs/changes/57/282357/1
diff --git a/manifests/role/statsd.pp b/manifests/role/statsd.pp
index 319728d..1fd0a91 100644
--- a/manifests/role/statsd.pp
+++ b/manifests/role/statsd.pp
@@ -6,13 +6,6 @@
#
class role::statsd {
- class { '::statsdlb':
- ensure => absent
- server_port => 8125,
- backend_ports => range(8126, 8131),
- before => Class['::statsd_proxy'],
- }
-
class { '::statsd_proxy':
server_port => 8125,
backend_ports => range(8126, 8131),
diff --git a/modules/statsdlb/manifests/init.pp
b/modules/statsdlb/manifests/init.pp
deleted file mode 100644
index 8c4e885..0000000
--- a/modules/statsdlb/manifests/init.pp
+++ /dev/null
@@ -1,56 +0,0 @@
-# == Class: statsdlb
-#
-# statsdlb is a StatsD-compatible proxy which routes metrics to multiple local
-# UDP ports. It chooses the backend port for each metric by hashing the metric
-# name, to ensure aggregation works properly. Its purpose is to allow scaling
-# StatsD implementations which cannot exploit multiple CPU cores.
-#
-# === Parameters
-#
-# [*server_port*]
-# UDP port to listen on. Example: 9001.
-#
-# [*backend_ports*]
-# Array of local UDP ports to which incoming metrics should be routed.
-# Example: [ 9002, 9003, 9004 ].
-#
-# === Examples
-#
-# # Listen on UDP port 9001 and route to ports 9002-9004:
-# class { 'statsdlb':
-# server_port => 9001,
-# backend_ports => [ 9002, 9003, 9004 ],
-# }
-#
-class statsdlb(
- $ensure = present,
- $server_port,
- $backend_ports,
-) {
- validate_ensure($ensure)
- validate_array($backend_ports)
- validate_re(join($backend_ports, ' '), '^\d+( \d+)*$', '$backend_ports
must be an array of port numbers')
- validate_re($server_port, '^\d+$', '$server_port must be a port number')
-
- package { 'statsdlb':
- ensure => $ensure,
- }
-
- file { '/etc/default/statsdlb':
- ensure => $ensure,
- content => sprintf("DAEMON_ARGS=\"%s %s\"\n", $server_port,
join($backend_ports, ' ')),
- owner => 'root',
- group => 'root',
- mode => '0444',
- }
-
- service { 'statsdlb':
- ensure => ensure_service($ensure),
- enable => $ensure == 'present',
- provider => $::initsystem,
- subscribe => [
- Package['statsdlb'],
- File['/etc/default/statsdlb'],
- ],
- }
-}
--
To view, visit https://gerrit.wikimedia.org/r/282357
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I782ce4cb9d312f5d279aad279ee786f51f871d2e
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Faidon Liambotis <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
