Also fetch McM data if total events is defined and approval time is not

ticoann · Dec 2, 2014 · c0e5e12 · c0e5e12
1 parent d6f36c4
commit c0e5e12
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/python/WMCore/WMDataMining/Utils.py b/src/python/WMCore/WMDataMining/Utils.py
@@ -82,7 +82,10 @@ def gatherWMDataMiningStats(wmstatsUrl, reqmgrUrl, wmMiningUrl,
                     pass # ReqMgr no longer has the workflow
                 report[wf].update({'filterEfficiency':filterEfficiency, 'runWhiteList':runWhiteList})
 
-            if not oldCouchDoc.has_key('mcmTotalEvents') or not oldCouchDoc.has_key('mcmApprovalTime'):
+            if (not oldCouchDoc.has_key('mcmTotalEvents') or
+                not oldCouchDoc.has_key('mcmApprovalTime') or
+                (oldCouchDoc.get('mcmTotalEvents', 'Unknown') != 'Unknown'
+                 and oldCouchDoc.get('mcmApprovalTime', 'Unknown')  == 'Unknown')):
                 prepID = oldCouchDoc.get('prepID', None)
                 if prepID and nMCMCalls <= maxMCMCalls:
                     nMCMCalls += 1