From b19e7c59460a6a5f11261bfe12098ae6e484fe88 Mon Sep 17 00:00:00 2001
From: Jean-Paul Calderone <exarkun@twistedmatrix.com>
Date: Wed, 27 Feb 2019 13:06:20 -0500
Subject: [PATCH] fix and improve callback logging

---
 src/allmydata/watchdog/inotify.py | 23 ++++++++++++++++++-----
 1 file changed, 18 insertions(+), 5 deletions(-)

diff --git a/src/allmydata/watchdog/inotify.py b/src/allmydata/watchdog/inotify.py
index 11752b69a..a83f25aa7 100644
--- a/src/allmydata/watchdog/inotify.py
+++ b/src/allmydata/watchdog/inotify.py
@@ -21,6 +21,7 @@ __all__ = [
 
 from watchdog.observers import Observer
 from watchdog.events import (
+    FileSystemEvent,
     FileSystemEventHandler, DirCreatedEvent, FileCreatedEvent,
     DirDeletedEvent, FileDeletedEvent, FileModifiedEvent
 )
@@ -31,7 +32,6 @@ from allmydata.util.fileutil import abspath_expanduser_unicode
 
 from eliot import (
     ActionType,
-    MessageType,
     Message,
     Field,
     preserve_context,
@@ -47,6 +47,11 @@ from allmydata.util.fake_inotify import humanReadableMask, \
     IN_MOVE_SELF, IN_UNMOUNT, IN_Q_OVERFLOW, IN_IGNORED, IN_ONLYDIR, IN_DONT_FOLLOW, \
     IN_MASK_ADD, IN_ISDIR, IN_ONESHOT, IN_CLOSE, IN_MOVED, IN_CHANGED
 
+from ..util.eliotutil import (
+    INOTIFY_EVENTS,
+    validateInstanceOf,
+)
+
 TRUE  = 0
 FALSE = 1
 
@@ -68,15 +73,23 @@ MAYBE_NOTIFY = ActionType(
     u"An inotify event is being considered for dispatch to an application handler.",
 )
 
+_EVENT = Field(
+    u"event",
+    lambda e: e.event_type,
+    u"The watchdog event that has taken place.",
+    validateInstanceOf(FileSystemEvent),
+)
+
 ANY_INOTIFY_EVENT = ActionType(
     u"watchdog:inotify:any-event",
-    [_PATH],
+    [_PATH, _EVENT],
     [],
     u"An inotify event is being dispatched.",
 )
 
-CALLBACK = MessageType(
+CALLBACK = ActionType(
     u"watchdog:inotify:callback",
+    [INOTIFY_EVENTS],
     [],
     u"An inotify event is being dispatched to an application callback."
 )
@@ -107,7 +120,7 @@ class INotifyEventHandler(FileSystemEventHandler):
                 return
             for cb in self._callbacks:
                 try:
-                    with CALLBACK():
+                    with CALLBACK(inotify_events=event_mask):
                         cb(None, FilePath(path), event_mask)
                 except:
                     # Eliot already logged the exception for us.
@@ -129,7 +142,7 @@ class INotifyEventHandler(FileSystemEventHandler):
         )
 
     def on_any_event(self, event):
-        with ANY_INOTIFY_EVENT(path=self._path):
+        with ANY_INOTIFY_EVENT(path=self._path, event=event):
             self.process(event)