Fix empty codec summary report

Don't generate summary report if revision has fluster job nodes and doesn't have any specific test nodes as codec summary report is intended to provide stats of number of passed, failed, and total tests of fluster jobs. Signed-off-by: Jeny Sadadia <[email protected]>
kernelci · Nov 30, 2024 · 2d22a30 · 2d22a30
1 parent 5df0c44
commit 2d22a30
Showing 1 changed file with 13 additions and 0 deletions.
diff --git a/kcidb/monitor/subscriptions/codec_summary.py b/kcidb/monitor/subscriptions/codec_summary.py
@@ -27,6 +27,19 @@ def match_revision(revision):
     ):
         return ()
 
+    # Don't generate summary report if revision has fluster job nodes
+    # and doesn't have any specific test nodes as codec summary
+    # report is intended to provide stats of number of passed, failed, and
+    # total tests of fluster jobs
+    # Below is the maestro convention for fluster tests:
+    # Job node path: "fluster.chromeos.v4l2.gstreamer_vp9"
+    # Test node path: "fluster.chromeos.v4l2.gstreamer_vp9.<test-name>"
+    if all(
+        len(t.path.split('.')) <= 4
+        for t in revision.tests_root["fluster"].tests
+    ):
+        return ()
+
     # Send notification 3 hours after a revision is created/updated
     return (Message(
         subject='KernelCI report for fluster tests: '