Ori.livneh has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/177634

Change subject: stats bits endpoint: don't require a trailing slash
......................................................................

stats bits endpoint: don't require a trailing slash

Makes for nicer URLs, and URL length is at a premium when you're encoding
payload data as query parameters.

Change-Id: I6fd90b33a368149c8c73f567522777faaca41789
---
M manifests/role/cache.pp
M templates/varnish/bits.inc.vcl.erb
2 files changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/34/177634/1

diff --git a/manifests/role/cache.pp b/manifests/role/cache.pp
index 9e2e564..24f682e 100644
--- a/manifests/role/cache.pp
+++ b/manifests/role/cache.pp
@@ -488,7 +488,7 @@
             format_type       => 'json',
             topic             => 'statsv',
             varnish_name      => $varnish_name,
-            varnish_opts      => { 'm' => 'RxURL:^/statsv/', },
+            varnish_opts      => { 'm' => 'RxURL:^/statsv', },
             # By requiring 2 ACKs per message batch, we survive a
             # single broker dropping out of its leader role,
             # without seeing lost messages.
diff --git a/templates/varnish/bits.inc.vcl.erb 
b/templates/varnish/bits.inc.vcl.erb
index 6f4e20e..f32f889 100644
--- a/templates/varnish/bits.inc.vcl.erb
+++ b/templates/varnish/bits.inc.vcl.erb
@@ -23,7 +23,7 @@
 <% end -%>
 
 sub vcl_recv {
-       if (req.url ~ "^/(event\.gif|statsv/)") {
+       if (req.url ~ "^/(event\.gif|statsv)") {
                // URLs that start with 'event.gif' or 'statsv' are logging 
beacon endpoints.
                // They are handled by log tailers (varnishkafka and 
varnishncsa) that filter
                // the Varnish shm log for reqs to these endpoints and forward 
them to log

-- 
To view, visit https://gerrit.wikimedia.org/r/177634
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I6fd90b33a368149c8c73f567522777faaca41789
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Ori.livneh <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to