Skip to content
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

Failures in two L1 TestDQMOnlineClient unit tests #32796

Closed
makortel opened this issue Feb 2, 2021 · 10 comments · Fixed by #32800
Closed

Failures in two L1 TestDQMOnlineClient unit tests #32796

makortel opened this issue Feb 2, 2021 · 10 comments · Fixed by #32800

Comments

@makortel
Copy link
Contributor

makortel commented Feb 2, 2021

Two unit tests in DQM/Integration package are failing since CMSSW_11_3_X_2021-02-02-1100

----- Begin Fatal Exception 02-Feb-2021 16:05:10 CET-----------------------
An exception of category 'NoRecord' occurred while
   [0] Processing  stream begin Run run: 334393 stream: 0
   [1] Calling method for module L1TStage2uGT/'l1tStage2uGT'
Exception Message:
No "L1TGlobalPrescalesVetosFractRcd" record found in the EventSetup.n
 Please add an ESSource or ESProducer that delivers such a record.
----- End Fatal Exception -------------------------------------------------
..
---> test TestDQMOnlineClient-l1tstage2_dqm_sourceclient had ERRORS

----- Begin Fatal Exception 02-Feb-2021 16:05:10 CET-----------------------
An exception of category 'NoRecord' occurred while
   [0] Processing  stream begin Run run: 334393 stream: 0
   [1] Calling method for module L1TStage2uGT/'l1tStage2uGtEmul'
Exception Message:
No "L1TGlobalPrescalesVetosFractRcd" record found in the EventSetup.n
 Please add an ESSource or ESProducer that delivers such a record.
----- End Fatal Exception -------------------------------------------------
---> test TestDQMOnlineClient-l1tstage2emulator_dqm_sourceclient had ERRORS

http://cmssdt.cern.ch/SDT/cgi-bin/logreader/slc7_amd64_gcc900/CMSSW_11_3_X_2021-02-02-1100/unitTestLogs/DQM/Integration

@makortel
Copy link
Contributor Author

makortel commented Feb 2, 2021

assign dqm,l1

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 2, 2021

New categories assigned: dqm,l1

@jfernan2,@andrius-k,@rekovic,@ahmad3213,@jmduarte,@kmaeshima,@rvenditti,@ErnestaP you have been requested to review this Pull request/Issue and eventually sign? Thanks

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 2, 2021

A new Issue was created by @makortel Matti Kortelainen.

@Dr15Jones, @dpiparo, @silviodonato, @smuzaffar, @makortel, @qliphy can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@makortel
Copy link
Contributor Author

makortel commented Feb 2, 2021

Likely to be caused by #32741 (only PR merged into that IB, and seems to be related)

@christopheralanwest
Copy link
Contributor

I suspect that the issue is that the global tag at

GlobalTag.globaltag = "111X_dataRun3_HLT_v3"
should be replaced with 112X_dataRun3_HLT_v1, which differs from the existing tag by the addition of the new record:
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/111X_dataRun3_HLT_v3/112X_dataRun3_HLT_v1

@jfernan2
Copy link
Contributor

jfernan2 commented Feb 3, 2021

I have created #32800 following @christopheralanwest 's comment

@silviodonato
Copy link
Contributor

@jfernan2 @christopheralanwest could you clarify if #32800 #32803 are needed for the MWGR? My understanding was that the global tag is taken elsewhere during the MWGR, but probably I'm wrong.

@jfernan2
Copy link
Contributor

jfernan2 commented Feb 3, 2021

At least #32800 is needed since DQM will be running in 112X
Despite GT can be changed manually at P5 DQM machines by DQM DOC, we usually make PRs for every big release change on this file to track and log which GTs are being used for MWGRs, see:

https://github.com/cms-sw/cmssw/commits/18442d459ff6789a404241f5dcd7633ad51a29e7/DQM/Integration/python/config/FrontierCondition_GT_cfi.py
Modification of Express GT is also needed as @mmusich pointed out in
#32800 (comment)
Thanks

@silviodonato
Copy link
Contributor

Solved by #32800 #32803

@jfernan2
Copy link
Contributor

jfernan2 commented Feb 5, 2021

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants