[report] Collect times about plugins postprocessing

Likewise we collect plugin setup and execution times, let collect plugin postproc times. Note that postprocessing is done after all plugins are collected and its time is not covered in either current manifest data. Resolves: #3834 Signed-off-by: Pavel Moravec <[email protected]>
sosreport · Nov 13, 2024 · 32af5d5 · 32af5d5
1 parent ba6c200
commit 32af5d5
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/sos/report/__init__.py b/sos/report/__init__.py
@@ -1485,7 +1485,12 @@ def postproc(self):
         for plugname, plug in self.loaded_plugins:
             try:
                 if plug.get_option('postproc'):
+                    start = datetime.now()
+                    plug.manifest.add_field('postproc_start', start)
                     plug.postproc()
+                    end = datetime.now()
+                    plug.manifest.add_field('postproc_end', end)
+                    plug.manifest.add_field('postproc_time', end - start)
                 else:
                     self.soslog.info(
                         f"Skipping postproc for plugin {plugname}")

diff --git a/sos/report/plugins/__init__.py b/sos/report/plugins/__init__.py
@@ -620,6 +620,9 @@ def set_plugin_manifest(self, manifest):
         self.manifest.add_field('setup_start', '')
         self.manifest.add_field('setup_end', '')
         self.manifest.add_field('setup_time', '')
+        self.manifest.add_field('postproc_start', '')
+        self.manifest.add_field('postproc_end', '')
+        self.manifest.add_field('postproc_time', '')
         self.manifest.add_field('timeout', self.timeout)
         self.manifest.add_field('timeout_hit', False)
         self.manifest.add_field('command_timeout', self.cmdtimeout)