TS-1928 False warning when setting log rotation size to exactly 10

Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/2592249e
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/2592249e
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/2592249e

Branch: refs/heads/3.3.x
Commit: 2592249e120ef8e20cbc998dbce845a53dadd509
Parents: 2712c30
Author: Leif Hedstrom <[email protected]>
Authored: Wed May 29 15:07:44 2013 -0600
Committer: Leif Hedstrom <[email protected]>
Committed: Wed May 29 15:07:44 2013 -0600

----------------------------------------------------------------------
 CHANGES                    |    2 ++
 proxy/logging/LogObject.cc |    2 +-
 2 files changed, 3 insertions(+), 1 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/2592249e/CHANGES
----------------------------------------------------------------------
diff --git a/CHANGES b/CHANGES
index 3d9ed65..56a5dc1 100644
--- a/CHANGES
+++ b/CHANGES
@@ -2,6 +2,8 @@
   Changes with Apache Traffic Server 3.3.3
 
 
+  *) [TS-1928] False warning when setting log rotation size to exactly 10.
+
   *) [TS-1812] Remove obsolete syslog_thr_init calls.
 
   *) [TS-1825] TSPortDescriptorAccept does not use it's argument.

http://git-wip-us.apache.org/repos/asf/trafficserver/blob/2592249e/proxy/logging/LogObject.cc
----------------------------------------------------------------------
diff --git a/proxy/logging/LogObject.cc b/proxy/logging/LogObject.cc
index 8484b02..0edc7fa 100644
--- a/proxy/logging/LogObject.cc
+++ b/proxy/logging/LogObject.cc
@@ -684,7 +684,7 @@ LogObject::_setup_rolling(int rolling_enabled, int 
rolling_interval_sec, int rol
 
     if (rolling_enabled == LogConfig::ROLL_ON_SIZE_ONLY ||
         rolling_enabled == LogConfig::ROLL_ON_TIME_OR_SIZE || rolling_enabled 
== LogConfig::ROLL_ON_TIME_AND_SIZE) {
-      if (rolling_size_mb <= 10) {
+      if (rolling_size_mb < 10) {
         m_rolling_size_mb = 10;
         Note("Rolling size invalid(%d) for %s, setting it to 10 MB", 
rolling_size_mb, m_filename);
       } else {

Reply via email to