[jira] [Work logged] (TS-5050) The background_fetch plugin fails to check for config files relative to the config dir etc/trafficserver

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/TS-5050?focusedWorklogId=31993&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-31993 ]

ASF GitHub Bot logged work on TS-5050:
--------------------------------------

                Author: ASF GitHub Bot
            Created on: 11/Nov/16 20:46
            Start Date: 11/Nov/16 20:46
    Worklog Time Spent: 10m 
      Work Description: GitHub user jrushford opened a pull request:

    https://github.com/apache/trafficserver/pull/1218

    TS-5050: The background_fetch plugin fails to check for config files …

    This may be from a regression with TS-2682.  Anyway, with the following remap config that we've been using with ATS 5.3.2, the background_fetch config file could not be found and read.  The config file is located at /opt/trafficserver/etc/trafficserver/bg_fetch.config
    
    @plugin=background_fetch.so @pparam=bg_fetch.config
    
    This works:
    
    @plugin=background_fetch.so @pparam=etc/trafficserver/bg_fetch.config
    
    This PR restores the original behavior so that both configurations shown above work.


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/jrushford/trafficserver TS-5050

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/trafficserver/pull/1218.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1218
    
----
commit 42ffc818416a5486d42bb51190f064db05e7e4e5
Author: John J. Rushford <jr...@apache.org>
Date:   2016-11-11T20:31:33Z

    TS-5050: The background_fetch plugin fails to check for config files relative to the config dir etc/trafficserver.

----


Issue Time Tracking
-------------------

            Worklog Id:     (was: 31993)
            Time Spent: 10m
    Remaining Estimate: 0h

> 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: 10m
>  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)