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

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

Github user SolidWallOfCode commented on a diff in the pull request:

    https://github.com/apache/trafficserver/pull/274#discussion_r36771917
  
    --- Diff: cmd/traffic_manager/traffic_manager.cc ---
    @@ -565,6 +612,25 @@ main(int argc, const char **argv)
         lmgmt->proxy_options = tsArgs;
         mgmt_log(stderr, "[main] Traffic Server Args: '%s'\n", 
lmgmt->proxy_options);
       }
    +
    +  // we must pass in bind_stdout and bind_stderr values to TS
    +  // we do it so TS is able to create BaseLogFiles for each value
    +  if (strcmp(bind_stdout, "") != 0) {
    +    lmgmt->proxy_options =
    +      (char *)ats_realloc(lmgmt->proxy_options, 
strlen(lmgmt->proxy_options) + 1 /* space */
    +                                                  + strlen("--bind_stdout 
") + strlen(bind_stdout) + 1 /* null term */);
    +    strcat(lmgmt->proxy_options, " --bind_stdout ");
    +    strcat(lmgmt->proxy_options, bind_stdout);
    +  }
    +
    +  if (strcmp(bind_stderr, "") != 0) {
    --- End diff --
    
    Maybe `0 == *bind_stdout` instead of a function call?


> enable log rotation for diags.log
> ---------------------------------
>
>                 Key: TS-306
>                 URL: https://issues.apache.org/jira/browse/TS-306
>             Project: Traffic Server
>          Issue Type: Improvement
>          Components: Logging
>            Reporter: Miles Libbey
>            Assignee: Daniel Xu
>              Labels: newbie
>             Fix For: 6.1.0
>
>
> (from yahoo bug 913896)
> Original description
> by Leif Hedstrom 3 years ago at 2006-12-04 12:42
> There might be reasons why this file might get filled up, e.g. libraries used 
> by plugins producing output on STDOUT/STDERR. A few suggestions have been
> made, to somehow rotate traffic.out. One possible solution (suggested by 
> Ryan) is to use cronolog (http://cronolog.org/), which seems like a fine idea.
>               
>  
> Comment 1
>  by Joseph Rothrock  2 years ago at 2007-10-17 09:13:24
> Maybe consider rolling diags.log as well. -Feature enhancement.
>               
> Comment 2
>  by Kevin Dalley 13 months ago at 2009-03-04 15:32:18
> When traffic.out gets filled up, error.log stops filing up, even though 
> rotation is turned on. This is
> counter-intuitive.  Rotation does not control traffic.out, but a large 
> traffic.out will stop error.log from being
> written.
>               



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

Reply via email to