<p>Corey Farrell <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/6861">View Change</a></p><p>Patch set 1:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">For someone who has strategy set to NONE or just defaults, will<br>this change any behavior?  Do we need something in CHANGES?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">On further investigation I think this patch is unneeded.  I was concerned about configs with:<br>rotatestrategy=none<br>exec_after_rotate=logrotate --force ${filename}</p><p style="white-space: pre-wrap; word-wrap: break-word;">We would not automatically initiate external logrotate when the log file grows above 1gb.  But this amount is arbitrary and doesn't necessarily mean anything.  SIGXFSZ is what really matters, and in response to SIGXFSZ we ultimately call reload_logger(-1, NULL).  In this case we don't need to check the size of the file in question, we'll end up calling rotate_file on all file based channels.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Going to abandon this, thanks for looking.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/6861">change 6861</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/6861"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I3633c24242b88da9daa1d2217dcde0b302e03653 </div>
<div style="display:none"> Gerrit-Change-Number: 6861 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 23 Oct 2017 17:14:03 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>