[GitHub] [trafficserver] bneradt opened a new pull request #7275: Respecting default rolling_enabled value in plugins.

[GitBox <gi...@apache.org>]

bneradt opened a new pull request #7275:
URL: https://github.com/apache/trafficserver/pull/7275


   We're using logrotate(3) via SIGUSR2. As such we're no longer using ATS log rotate and are disabling proxy.config.log.rolling_enabled. We noticed that some plugins indiscriminately enable log rolling without consideration of the system value. This looks like it is just a mistake that went unnoticed because we've always had this enabled. I'm changing a couple plugins to respect the proxy.config.log.rolling_enabled value by not overwriting it with TSTextLogObjectRollingEnabledSet unless explicitly configured to do so.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [trafficserver] bneradt merged pull request #7275: Respecting default rolling_enabled value in plugins.

[GitBox <gi...@apache.org>]

bneradt merged pull request #7275:
URL: https://github.com/apache/trafficserver/pull/7275


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org