[GitHub] trafficserver issue #1218: TS-5050: The background_fetch plugin fails to che...

2016-11-16 Thread jrushford
Github user jrushford commented on the issue: https://github.com/apache/trafficserver/pull/1218 whew wipes sweat from furrowed brow :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have thi

[GitHub] trafficserver issue #1218: TS-5050: The background_fetch plugin fails to che...

2016-11-15 Thread jrushford
Github user jrushford commented on the issue: https://github.com/apache/trafficserver/pull/1218 see the changes @jpeach. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled a

[GitHub] trafficserver issue #1218: TS-5050: The background_fetch plugin fails to che...

2016-11-15 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1218 Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/1096/ for details. --- If your project is set up for it, you can reply to this email and have your rep

[GitHub] trafficserver issue #1218: TS-5050: The background_fetch plugin fails to che...

2016-11-15 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1218 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1203/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1218: TS-5050: The background_fetch plugin fails to che...

2016-11-14 Thread jrushford
Github user jrushford commented on the issue: https://github.com/apache/trafficserver/pull/1218 @jpeach - tested and the fix consumes absolute paths, relative paths to install directory, and it now consumes config files relative to the config directory --- If your project is set up

[GitHub] trafficserver issue #1218: TS-5050: The background_fetch plugin fails to che...

2016-11-11 Thread jpeach
Github user jpeach commented on the issue: https://github.com/apache/trafficserver/pull/1218 @jrushford Since you are fixing this, please make sure that the new behavior is consistent with other plugins that load files. IIRC absolute paths are consumes as is, relative paths are relati

[GitHub] trafficserver issue #1218: TS-5050: The background_fetch plugin fails to che...

2016-11-11 Thread jrushford
Github user jrushford commented on the issue: https://github.com/apache/trafficserver/pull/1218 @zwoop I just put back the code used previous to TS-2682 in background_fetch.cc. Didn't spend too much time on it, just wanted our configs to be read wen we deploy 6.2. I can go back and

[GitHub] trafficserver issue #1218: TS-5050: The background_fetch plugin fails to che...

2016-11-11 Thread jpeach
Github user jpeach commented on the issue: https://github.com/apache/trafficserver/pull/1218 Is this change consistent with other plugins are doing? > On Nov 11, 2016, at 1:07 PM, Leif Hedstrom wrote: > > @zwoop approved this pull request. > > I'm ok with th

[GitHub] trafficserver issue #1218: TS-5050: The background_fetch plugin fails to che...

2016-11-11 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1218 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1201/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1218: TS-5050: The background_fetch plugin fails to che...

2016-11-11 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1218 Please add Milestones, Labels etc... --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature e

[GitHub] trafficserver issue #1218: TS-5050: The background_fetch plugin fails to che...

2016-11-11 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1218 Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/1094/ for details. --- If your project is set up for it, you can reply to this email and have your rep