[ 
https://issues.apache.org/jira/browse/TS-3848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14710279#comment-14710279
 ] 

ASF GitHub Bot commented on TS-3848:
------------------------------------

GitHub user pushkarpradhan opened a pull request:

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

    TS3848

    Fixes TS-3848.
    A new variable proxy.config.http.cache.required (range: 0-2) controls 
whether we will exit on cache or volume initialization failure. 
    0     Continue serving requests even if we failed to initialize the cache
    1     Stop serving requests if none of the cache disks or volumes could be 
initialized
    2     Stop serving requests even if one or some of the cache disks or 
volumes could not be initialized


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

    $ git pull https://github.com/pushkarpradhan/trafficserver ts3848

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

    https://github.com/apache/trafficserver/pull/280.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 #280
    
----
commit bfb4547378b02668f71aebfac0dc5d1f944c34e6
Author: Pushkar Pradhan <[email protected]>
Date:   2015-08-17T22:54:05Z

    Fix for TS-3848

commit 8de2a11d613b8331917856e6d9dacd8e0ba3167e
Author: Pushkar Pradhan <[email protected]>
Date:   2015-08-18T00:59:32Z

    Also need to check for the wait_for_cache variable

commit ee065b864076065950a9ac3cdeb1b29ebcd52013
Author: Pushkar Pradhan <[email protected]>
Date:   2015-08-19T19:06:26Z

    Refactoring code to use method CacheProcessor::cacheRequired

commit ce7e983c0f282a2aaa704d5de15a7154744c5bcd
Author: Pushkar Pradhan <[email protected]>
Date:   2015-08-19T19:33:07Z

    Fail if total_nvol is defined, it can be zero

commit 28c124ed953e26d242f152113b52ca8faefc6df7
Author: Pushkar Pradhan <[email protected]>
Date:   2015-08-24T23:13:51Z

    Checking for null pointer access before checking cache status

----


> ATS runs without cache or partial cache on disk errors
> ------------------------------------------------------
>
>                 Key: TS-3848
>                 URL: https://issues.apache.org/jira/browse/TS-3848
>             Project: Traffic Server
>          Issue Type: Bug
>          Components: Cache
>            Reporter: Pushkar Pradhan
>            Assignee: Alan M. Carroll
>             Fix For: 6.1.0
>
>
> Problem:
> If ATS fails to initialize one or more disks it continues to run without 
> cache. This can cause origin overload.
> The situation can be somewhat mitigated by setting 
> proxy.config.http.wait_for_cache = 1 and if none of the disks failed to 
> initialize.
> However, even if wait_for_cache = 1 and only one or a few disks failed to 
> initialize, ATS will continue to serve traffic. 
> Proposed Solution:
> Define a new variable: proxy.config.http.cache.required
> Value range: 0-2
> 0 (default) - Do nothing
> 1 - Abort trafficserver if it failed to initialize all the disks/volumes
> 2 - Abort trafficserver if it failed to initialize even one of the disks or 
> volumes.
> If proxy.config.http.cache.required = 1 and proxy.config.http.wait_for_cache 
> = 1 and if proxy.config.http.cache.required > 0 then abort the traffic server 
> if one or more cache disks/volumes could not be initialized.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to