<div dir="ltr">Sorry to dig that one up, but I stumble upon it again, and it got me thinking: can't we just put the point to req->err_code in priv? That's not super clean, but from what I re-read in the code, it should still be there when it's time to clean the priv.</div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div>-- <br></div>Guillaume Quintard<br></div></div></div>
<br><div class="gmail_quote">On Tue, May 23, 2017 at 11:15 PM, Ryan Burn <span dir="ltr"><<a href="mailto:rnickb731@gmail.com" target="_blank">rnickb731@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">> When it comes to header manipulations (including method, status etc)<br>
> they are already logged so a VUT can already pick that up and save<br>
> some work from the VMOD.<br>
<br>
</span>It doesn't need to just view the headers; it needs to add headers to encode the<br>
active span context. See<br>
<a href="http://opentracing.io/documentation/pages/api/cross-process-tracing.html" rel="noreferrer" target="_blank">http://opentracing.io/<wbr>documentation/pages/api/cross-<wbr>process-tracing.html</a><br>
And the active span needs to be started before those headers can be added.<br>
<span class=""><br>
> I'm not familiar with OT but what I described is how Zipnish does its<br>
> tracing. Except that Zipnish relies on the X-Varnish header to get a<br>
> unique-ish id, so no blocking call needs to be made. So maybe they are<br>
> highly different systems, I chimed in because I saw a Zipkin example<br>
> while briefly skimming through the docs.<br>
<br>
</span>Zipkin is a distributed tracing system that provides OpenTracing<br>
implementations, but Zipnish is just using it as a monitor for<br>
varnish. It's not doing context propagation. If you're only interested<br>
in monitoring varnish, that's fine; but if you want to see how a<br>
request is processed in an entire distributed system (i.e. not just<br>
varnish, but the backend servers it's sitting in front of or any other<br>
service that might be in front of varnish), then you need to do<br>
context propagation.<br>
</blockquote></div><br></div>