[ 
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 <notificati...@github.com> 
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)

Reply via email to