Reopen event log on 'logreopen' or 'reopen_transcript' (#8)

CHANGES.rst

@@ -24,6 +24,9 @@ Change log
 - Make ``logreopen`` also reopen the transcript log:
   https://github.com/zopefoundation/zdaemon/issues/9.
 
+- Reopen event log on ``logreopen`` or ``reopen_transcript``:
+  https://github.com/zopefoundation/zdaemon/issues/8.
+
 - Fix race condition where ``stop`` would be ignored if the daemon
   manager was waiting before respawning a crashed program.
   https://github.com/zopefoundation/zdaemon/issues/13.

src/zdaemon/tests/tests.py

@@ -14,10 +14,11 @@
 from __future__ import print_function
 
 import doctest
+import glob
 import os
 import re
-import signal
 import shutil
+import signal
 import subprocess
 import sys
 import tempfile
@@ -221,6 +222,48 @@ def test_logreopen():
     """
 
 
+def test_log_rotation():
+    """
+
+    >>> write('conf',
+    ... '''
+    ... <runner>
+    ...   program sleep 100
+    ...   transcript transcript.log
+    ... </runner>
+    ... <eventlog>
+    ...   <logfile>
+    ...     path event.log
+    ...   </logfile>
+    ... </eventlog>
+    ... ''')
+
+    >>> system("./zdaemon -Cconf start")
+    . .
+    daemon process started, pid=1234
+
+    Pretend we did a logrotate:
+
+    >>> os.rename('transcript.log', 'transcript.log.1')
+    >>> os.rename('event.log', 'event.log.1')
+
+    >>> system("./zdaemon -Cconf reopen_transcript")  # or logreopen
+
+    This reopens both transcript.log and event.log:
+
+    >>> sorted(glob.glob('transcript.log*'))
+    ['transcript.log', 'transcript.log.1']
+
+    >>> sorted(glob.glob('event.log*'))
+    ['event.log', 'event.log.1']
+
+    >>> system("./zdaemon -Cconf stop")
+    . .
+    daemon process stopped
+
+    """
+
+
 def test_start_test_program():
     """
     >>> write('t.py',

src/zdaemon/zdrun.py

@@ -46,6 +46,7 @@
             break
 
 from zdaemon.zdoptions import RunnerOptions
+from ZConfig.components.logger.loghandler import reopenFiles
 
 
 def string_list(arg):
@@ -589,6 +590,7 @@ def cmd_status(self, args):
                        "args=%r\n" % self.proc.args)
 
     def cmd_reopen_transcript(self, args):
+        reopenFiles()
         if self.transcript is not None:
             self.transcript.reopen()