Faidon Liambotis has submitted this change and it was merged.
Change subject: diamond: don't disable IPVS by default
......................................................................
diamond: don't disable IPVS by default
I searched for quite a while but couldn't find evidence that IPVS is
enabled by default, as the comments suggest. On a trusty system,
diamond-setup -p | grep IPVS
says 'enabled': False, among other things I checked.
Change-Id: I9e11849a0b6d1fe3eb827610a517e3e7750ad579
---
M manifests/role/diamond.pp
1 file changed, 0 insertions(+), 14 deletions(-)
Approvals:
Filippo Giunchedi: Looks good to me, but someone else must approve
Faidon Liambotis: Looks good to me, approved
jenkins-bot: Verified
diff --git a/manifests/role/diamond.pp b/manifests/role/diamond.pp
index ae18d78..85d3446 100644
--- a/manifests/role/diamond.pp
+++ b/manifests/role/diamond.pp
@@ -34,18 +34,4 @@
port => '8125',
},
}
-
- #IPVS collector seems to be enabled by default on trusty
- #causes non LVS hosts to spam with sudo violations for
- #stats collection among other things. Explicit disable
- #for now this needs to dealt with upstream.
- case $::operatingsystemrelease {
- '14.04': {
- diamond::collector { 'IPVS':
- settings => {
- enabled => 'false',
- },
- }
- }
- }
}
--
To view, visit https://gerrit.wikimedia.org/r/183785
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9e11849a0b6d1fe3eb827610a517e3e7750ad579
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Faidon Liambotis <[email protected]>
Gerrit-Reviewer: Faidon Liambotis <[email protected]>
Gerrit-Reviewer: Filippo Giunchedi <[email protected]>
Gerrit-Reviewer: Rush <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
