チケット内容を見る限り、以前の処理とは大分動作が異なっていますよね。

例えば、MAX_LOG_QUANTITYを１０→５に変えたり場合、過去のコードでは、ファイル数が５になりますが・・・今のコードでは、site.log.6からは無視されファイル数は１０のままです。

また過去のコードでは最後にsite.logを作成しておりますが、今のコードでは次回ログ書き込み時までsite.logファイルは存在しないかと・・・
どこで作成しているのかまで、まだおっかけてはいませんが、
呼び出し順序次第で、現在のコードではsite.logが存在しない場合、最後のrenameで必ずWarningがでてしまうかと思われるのですが。

他にディレクトリの存在チェックなんかもとっぱらわれていますし

う〜ん。チケットに記されている対応以外の処理も含まれてのことならわかりますが・・・
また明日時間をみて調べてみます。