Re: [EXTERNAL] Re: regression test failures.

2018-06-05 Thread Rushford, John
Thanks On 6/5/18, 7:02 AM, "Leif Hedstrom" wrote: > On Jun 4, 2018, at 9:37 PM, John Rushford wrote: > > I've started seeing this failure today when running regression tests, > > traffic_server -R 1 > > Regression test(SDK_API_TSHttpConnectIntercept) st

Re: [EXTERNAL] Re: [PROPOSAL] Remove the tools/traffic_shell.pl script

2018-05-07 Thread Rushford, John
+1 I've never used it. On 5/4/18, 3:07 PM, "Bryan Call" wrote: +1 - I don’t use it and it looks like it is broken on master -Bryan > On May 3, 2018, at 8:38 AM, Leif Hedstrom wrote: > > I don’t think this script has a purpose any more, and it’s not maintained. D

Re: Replacing TSAssert(!"error message")

2018-02-16 Thread Rushford, John
+1 TSFatal() On 2/16/18, 2:25 PM, "Bryan Call" wrote: +1 on TSFatal(). We already have Fatal() in the code. -Bryan > On Feb 16, 2018, at 7:33 AM, Alan Carroll wrote: > > PR 3129 was a fix to the ICC builds that disables a "expression is > constant" err

Re: parent.config documentation questions

2016-11-03 Thread Rushford, John
Hi Eric, I can't think of any reason to use .990 over 1.0. John From: Eric Friedrich (efriedri) Sent: Wednesday, November 2, 2016 6:46 PM To: dev@trafficserver.apache.org Subject: Re: parent.config documentation questions Thanks John- In Traffic Contro

Re: [GitHub] trafficserver pull request: TS-4287: Add a simple and unavailable ...

2016-04-10 Thread Rushford, John
Leif, Let me know if this looks okay, I’d like to land it for 6.2.0. Thanks -- John J. Rushford IPCDN Engineering 1400 Wewatta Street, Denver Colorado 80202 john_rushf...@cable.comcast.com On 4/10/16, 5:27 PM, "zwoop" wrote: >Github user zwoop commented on the pull request:

Re: ATS byte-range requests

2016-03-14 Thread Rushford, John
ient_send_response(): added range header: bytes=0-20 > > >Thanks for your help. >Nuno > > > >Nuno Baeta >nuno.ba...@nos.pt > >Direção de Plataformas Core e Serviços >F: +351 210 1045 32 M:+351 931 019 532 >Av. D. João II, Lote 1

Re: ATS byte-range requests

2016-03-11 Thread Rushford, John
Nuno, You can enable debugging in records.config: CONFIG proxy.config.diags.debug.enabled INT 1 CONFIG proxy.config.diags.debug.tags STRING cache_range_requests Re-run your testing to see if the Range Request header is recognized. Thanks -- John J. Rushford IPCDN Engineering 1400 Wewatta Street

Re: [GitHub] trafficserver pull request: TS-4115: add multi origin feature to p...

2016-03-01 Thread Rushford, John
I¹ve closed the original pull request and submitted another for just the multi-origin changes. See https://github.com/apache/trafficserver/pull/504 Thanks -- John J. Rushford On 2/23/16, 11:08 AM, "bryancall" wrote: >Github user bryancall commented on the pull request: > > >https://git

Re: [trafficserver] TS-3418: Second hash ring for consistently hashed parent selection (#359)

2015-12-11 Thread Rushford, John
James, I squashed in a final with these changes, you might wish to clone the branch again. 1. I fixed the ownership concern. It helps having someone else review your code, thanks very much for that! 2. I kept the round_robin flag. As the implementation differences between no round robi

Re: [GitHub] trafficserver pull request: Modified the plugin to add back the Ra...

2015-09-18 Thread Rushford, John
I¹ll look into this and some other changes jpeach suggested. Thanks -- John J. Rushford IPCDN Engineering 1400 Wewatta Street, Denver Colorado 80202 john_rushf...@cable.comcast.com On 9/18/15, 12:30 PM, "sudheerv" wrote: >Github user sudheerv commented on the pull request: > >

range request plugin.

2015-06-03 Thread Rushford, John
Greetings, I’ve submitted a plugin, TS-3662, I wrote that transforms range requests so that each byte range request may be stored in the ATS cache as an individual object. We have a this plugin in production where it is used to cache range requests for delivery of 60gb + game files. This plug