[
https://issues.apache.org/jira/browse/TS-5050?focusedWorklogId=31997&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-31997
]
ASF GitHub Bot logged work on TS-5050:
--------------------------------------
Author: ASF GitHub Bot
Created on: 11/Nov/16 21:13
Start Date: 11/Nov/16 21:13
Worklog Time Spent: 10m
Work Description: 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 <[email protected]>
wrote:
>
> @zwoop approved this pull request.
>
> I'm ok with this, but it feels a little clunky that we duplicate the
codes / error checking nested like this. Since it's C++, couldn't we iterate
over a vector of candidate directories?
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub, or mute the thread.
>
Issue Time Tracking
-------------------
Worklog Id: (was: 31997)
Time Spent: 50m (was: 40m)
> The background_fetch plugin fails to check for config files relative to the
> config dir etc/trafficserver
> --------------------------------------------------------------------------------------------------------
>
> Key: TS-5050
> URL: https://issues.apache.org/jira/browse/TS-5050
> Project: Traffic Server
> Issue Type: Bug
> Components: Plugins
> Affects Versions: 6.2.0, 7.0.0, 7.1.0
> Reporter: John Rushford
> Assignee: John Rushford
> Fix For: 7.1.0
>
> Time Spent: 50m
> Remaining Estimate: 0h
>
> We are currently using the background_fetch plugin with ATS 5.3.2 per remap.
> We configure the background_fetch plugin in remap.config using the relative
> path to a config file in etc/trafficserver. While testing ATS 6.2.1 with our
> configs, I found that the background_fetch plugin no-longer looks for config
> files relative to the etc/trafficserver configuration directory.
> I think that this is a regression from 5.3.2 to 6.2.x and master. I found
> that in configs.cc, TSConfigDirGet() is no-longer used to check for a config
> file relative to etc/trafficserver.
> I'm testing a patch to fix this.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
