From f8f8c6c8460f092b4fbb43ccc026a85c34e094f8 Mon Sep 17 00:00:00 2001
From: Muhammad Hassan Ahmed <33365802+hassan11196@users.noreply.github.com>
Date: Mon, 7 Oct 2024 20:37:06 +0000
Subject: [PATCH] Disable MS-output DID not found

---
 .../WMCore/MicroService/MSOutput/MSOutput.py  | 22 ++++++++-----------
 1 file changed, 9 insertions(+), 13 deletions(-)

diff --git a/src/python/WMCore/MicroService/MSOutput/MSOutput.py b/src/python/WMCore/MicroService/MSOutput/MSOutput.py
index 0cf8d256f0..e989f32938 100644
--- a/src/python/WMCore/MicroService/MSOutput/MSOutput.py
+++ b/src/python/WMCore/MicroService/MSOutput/MSOutput.py
@@ -705,8 +705,8 @@ def docInfoUpdate(self, msOutDoc):
 
         # if there were containers not found in Rucio, create an email alert
         if notFoundDIDs:
-            # log and send alert via AlertManager API
-            self.alertDIDNotFound(msOutDoc["RequestName"], notFoundDIDs)
+            # only log warning msg, the previous alerts to AlertManager were too noisy
+            self.logDIDNotFound(msOutDoc["RequestName"], notFoundDIDs)
 
         try:
             msOutDoc.updateDoc({"OutputMap": updatedOutputMap}, throw=True)
@@ -975,23 +975,19 @@ def docCleaner(self, doc):
         """
         return doc.clear()
 
-    def alertDIDNotFound(self, wflowName, containerList):
+    def logDIDNotFound(self, wflowName, containerList):
         """
-        Send an alert to Prometheus for output containers not found within
+        Log a warning message for output containers not found within
         a given workflow.
         :param wflowName: string with the workflow name
         :param containerList: list of container names
         :return: none
         """
-        alertName = "ms-output: output containers not found for workflow: {}".format(wflowName)
-        alertSeverity = "high"
-        alertSummary = "[MSOutput] Workflow '{}' has output datasets unknown to Rucio".format(wflowName)
-        alertDescription = "Dataset(s): {} cannot be found in Rucio. ".format(containerList)
-        alertDescription += "Thus, we are skipping these datasets from the final output "
-        alertDescription += "data placement, such that this workflow can get archived."
-        self.logger.warning(alertDescription)
-        if self.msConfig["sendNotification"]:
-            self.sendAlert(alertName, alertSeverity, alertSummary, alertDescription, self.alertServiceName)
+        msg = "[MSOutput] Workflow '{}' has output datasets unknown to Rucio ".format(wflowName)
+        msg += "Dataset(s): {} cannot be found in Rucio. ".format(containerList)
+        msg += "Thus, we are skipping these datasets from the final output "
+        msg += "data placement, such that this workflow can get archived."
+        self.logger.warning(msg)
 
     def alertCampaignNotFound(self, campaignName, containerName):
         """