Mobrovac has uploaded a new change for review. https://gerrit.wikimedia.org/r/196229
Change subject: Hit Parsoid directly ...................................................................... Hit Parsoid directly Until now we have been calling Parsoid by passing through Varnish. However, given that we use exclusively Parsoid's v2, which isn't cached at all, it's better to hit the LVS directly. Change-Id: Idce3d86ee6a9de1d029058e020467f9edf178f98 --- M hieradata/role/common/restbase.yaml M modules/restbase/manifests/init.pp M modules/restbase/templates/config.yaml.erb 3 files changed, 5 insertions(+), 1 deletion(-) git pull ssh://gerrit.wikimedia.org:29418/operations/puppet refs/changes/29/196229/1 diff --git a/hieradata/role/common/restbase.yaml b/hieradata/role/common/restbase.yaml index 0cb2926..a9dca47 100644 --- a/hieradata/role/common/restbase.yaml +++ b/hieradata/role/common/restbase.yaml @@ -23,6 +23,7 @@ restbase::cassandra_defaultConsistency: localQuorum restbase::cassandra_localDc: "%{::site}" restbase::statsd_host: statsd.eqiad.wmnet +restbase::parsoid_uri: http://10.2.2.28:8000 # parsoid.svc.eqiad.wmnet lvs::realserver::realserver_ips: - '10.2.2.17' # restbase.svc.eqiad.wmnet diff --git a/modules/restbase/manifests/init.pp b/modules/restbase/manifests/init.pp index 456b430..86be047 100644 --- a/modules/restbase/manifests/init.pp +++ b/modules/restbase/manifests/init.pp @@ -19,6 +19,8 @@ # Which DC should be considered local. Default: 'datacenter1'. # [*port*] # Port where to run the restbase service. Default: 7231 +# [*parsoid_uri*] +# URI to reach Parsoid. Default: http://parsoid-lb.eqiad.wikimedia.org # [*logstash_host*] # GELF logging host. Default: localhost # [*logstash_port*] @@ -38,6 +40,7 @@ $cassandra_defaultConsistency = 'localQuorum', $cassandra_localDc = 'datacenter1', $port = 7231, + $parsoid_uri = 'http://parsoid-lb.eqiad.wikimedia.org', $logstash_host = 'localhost', $logstash_port = 12201, $logging_level = 'warn', diff --git a/modules/restbase/templates/config.yaml.erb b/modules/restbase/templates/config.yaml.erb index 9892e5f..8298e72 100644 --- a/modules/restbase/templates/config.yaml.erb +++ b/modules/restbase/templates/config.yaml.erb @@ -115,7 +115,7 @@ version: 1.0.0 type: file options: - parsoidHost: http://parsoid-lb.eqiad.wikimedia.org + parsoidHost: <%= @parsoid_uri %> /{module:action}: x-modules: -- To view, visit https://gerrit.wikimedia.org/r/196229 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Idce3d86ee6a9de1d029058e020467f9edf178f98 Gerrit-PatchSet: 1 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Mobrovac <mobro...@wikimedia.org> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits