-
Notifications
You must be signed in to change notification settings - Fork 4.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Code still consumes CASTOR tags in Run-3 #36944
Comments
A new Issue was created by @tvami Tamas Vami. @Dr15Jones, @perrotta, @dpiparo, @makortel, @smuzaffar, @qliphy can you please review it and eventually sign/assign? Thanks. cms-bot commands are listed here |
assign dqm |
New categories assigned: dqm @jfernan2,@ahmad3213,@rvenditti,@emanueleusai,@pbo0,@pmandrik you have been requested to review this Pull request/Issue and eventually sign? Thanks |
isn't this a duplicate of the first part of #36806 (comment) ? seems to me something like: diff --git a/DQM/Integration/python/clients/ecal_dqm_sourceclient-live_cfg.py b/DQM/Integration/python/clients/ecal_dqm_sourceclient-live_cfg.py
index 43bf00c57aa..d2ccb842dd8 100644
--- a/DQM/Integration/python/clients/ecal_dqm_sourceclient-live_cfg.py
+++ b/DQM/Integration/python/clients/ecal_dqm_sourceclient-live_cfg.py
@@ -3,7 +3,8 @@ import FWCore.ParameterSet.Config as cms
from EventFilter.OnlineMetaDataRawToDigi.tcdsRawToDigi_cfi import * # To monitor LHC status, e.g. to mask trigger primitives quality alarm during Cosmics
import sys
-process = cms.Process("process")
+from Configuration.Eras.Era_Run3_cff import Run3
+process = cms.Process("process", Run3)
unitTest = False
if 'unitTest=True' in sys.argv:
diff --git a/DQM/Integration/python/clients/hlt_dqm_sourceclient-live_cfg.py b/DQM/Integration/python/clients/hlt_dqm_sourceclient-live_cfg.py
index 19d4a18a757..26462d10dbd 100644
--- a/DQM/Integration/python/clients/hlt_dqm_sourceclient-live_cfg.py
+++ b/DQM/Integration/python/clients/hlt_dqm_sourceclient-live_cfg.py
@@ -1,8 +1,8 @@
import FWCore.ParameterSet.Config as cms
import sys
-from Configuration.ProcessModifiers.run2_HECollapse_2018_cff import run2_HECollapse_2018
-process = cms.Process("DQM", run2_HECollapse_2018)
+from Configuration.Eras.Era_Run3_cff import Run3
+process = cms.Process("DQM", Run3)
unitTest = False
if 'unitTest=True' in sys.argv: on top of #36940 should solve. |
This error came up also in the Tier0 replay dmwm/T0#4644 (comment)
|
Thanks Marco, I submitted this in this PR #36946 |
I think a possible way of investigation is dmwm/T0#4644 (comment). This smells again like a stale era definition in the T0 configuration, given that Tamas' PR actually tests positively, setting a side the online DQM |
+1 |
This issue is fully signed and ready to be closed. |
In #36940
we discovered that the there is still some code that consumes CASTOR tags in Run-3.
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-0f9c0d/22366/unitTests/src/DQM/Integration/test/TestDQMOnlineClient-ecal_dqm_sourceclient/testing.log
and
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-0f9c0d/22366/unitTests/src/DQM/Integration/test/TestDQMOnlineClient-hlt_dqm_sourceclient/testing.log
and
The text was updated successfully, but these errors were encountered: