fix: Address code review feedback

- Add command field to training-health-cronjob and eval-health-cronjob
- Fix workflow exclusion logic to check both workflow name and ID

Author: Akkikens

devops/charts/helmfile.yaml

@@ -223,6 +223,10 @@ releases:
           tag: latest-dashboard
       - datadog:
           service: training-health-collector
+      - command:
+          - python
+          - -m
+          - devops.datadog.cli
       - args:
           - collect
           - training
@@ -236,6 +240,10 @@ releases:
           tag: latest-dashboard
       - datadog:
           service: eval-health-collector
+      - command:
+          - python
+          - -m
+          - devops.datadog.cli
       - args:
           - collect
           - eval

devops/datadog/collectors/ci_collector.py

@@ -320,10 +320,10 @@ def _count_other_workflows_failing(self) -> int:
         failing_count = 0
         for workflow in all_workflows:
             workflow_name = workflow.get("name") or workflow.get("path", "").split("/")[-1]
-            workflow_id = workflow.get("id")
+            workflow_id = str(workflow.get("id", ""))
 
-            # Skip if this is a test/benchmark workflow
-            if workflow_name in exclude_workflows:
+            # Skip if this workflow matches by name OR id
+            if workflow_name in exclude_workflows or workflow_id in exclude_workflows:
                 continue
 
             # Get latest run for this workflow