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

Enforce ESGetToken use for ESProducers #28223

Merged
merged 1 commit into from Nov 6, 2019

Conversation

Dr15Jones
Copy link
Contributor

PR description:

EventSetupRecord and EventSetup now have an option to enforce the use of ESGetTokens. For ESProducers that option is enabled, for ED modules it is disabled.

This is one of the steps to having concurrent running of ES modules.

PR validation:

By including #28215 #28216 #28217 and #28218 there are no failures in runTheMatrix.py related to this pull request.

EventSetupRecord and EventSetup now have an option to enforce the use of ESGetTokens. For ESProducers that option is enabled,  for ED modules it is disabled.
@cmsbuild
Copy link
Contributor

The code-checks are being triggered in jenkins.

@cmsbuild
Copy link
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-28223/12360

  • This PR adds an extra 84KB to repository

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @Dr15Jones (Chris Jones) for master.

It involves the following packages:

FWCore/Framework
FWCore/Integration

@cmsbuild, @smuzaffar, @Dr15Jones can you please review it and eventually sign? Thanks.
@makortel, @Martin-Grunewald, @wddgit this is something you requested to watch as well.
@davidlange6, @slava77, @fabiocos you are the release manager for this.

cms-bot commands are listed here

@Dr15Jones
Copy link
Contributor Author

please test with #28215,#28216,#28217,#28218

@cmsbuild
Copy link
Contributor

cmsbuild commented Oct 21, 2019

The tests are being triggered in jenkins.
Tested with other pull request(s) #28215,#28216,#28217,#28218
https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/3056/console Started: 2019/10/21 16:18

@cmsbuild
Copy link
Contributor

-1

Tested at: cf2f131

You can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-a586f1/3056/summary.html

I found follow errors while testing this PR

Failed tests: UnitTests RelVals AddOn

  • Unit Tests:

I found errors in the following unit tests:

---> test test_MuonGeometryDBConverter had ERRORS
---> test testCalibTrackerSiStripESProducersByHand had ERRORS
---> test DetectorDescriptionDDCMSTestDriver had ERRORS
---> test GeometryDTGeometryBuilderTestDriver had ERRORS
---> test TestRecoParticleFlowPFTrackingTP had ERRORS
---> test DD4hepGeometryTestDriver had ERRORS
---> test testPhase2PixelNtuple had ERRORS
---> test testTauEmbeddingProducers had ERRORS

  • RelVals:

When I ran the RelVals I found an error in the following workflows:
136.8311 step2

runTheMatrix-results/136.8311_RunJetHT2017F_reminiaod+RunJetHT2017F_reminiaod+REMINIAOD_data2017+HARVEST2017_REMINIAOD_data2017/step2_RunJetHT2017F_reminiaod+RunJetHT2017F_reminiaod+REMINIAOD_data2017+HARVEST2017_REMINIAOD_data2017.log

136.731 step3
runTheMatrix-results/136.731_RunSinglePh2016B+RunSinglePh2016B+HLTDR2_2016+RECODR2_2016reHLT_skimSinglePh_HIPM+HARVESTDR2/step3_RunSinglePh2016B+RunSinglePh2016B+HLTDR2_2016+RECODR2_2016reHLT_skimSinglePh_HIPM+HARVESTDR2.log

136.85 step3
runTheMatrix-results/136.85_RunEGamma2018A+RunEGamma2018A+HLTDR2_2018+RECODR2_2018reHLT_skimEGamma_Offline_L1TEgDQM+HARVEST2018_L1TEgDQM/step3_RunEGamma2018A+RunEGamma2018A+HLTDR2_2018+RECODR2_2018reHLT_skimEGamma_Offline_L1TEgDQM+HARVEST2018_L1TEgDQM.log

20034.0 step2
runTheMatrix-results/20034.0_TTbar_14TeV+TTbar_14TeV_TuneCUETP8M1_2026D35_GenSimHLBeamSpotFull14+DigiFullTrigger_2026D35+RecoFullGlobal_2026D35+HARVESTFullGlobal_2026D35/step2_TTbar_14TeV+TTbar_14TeV_TuneCUETP8M1_2026D35_GenSimHLBeamSpotFull14+DigiFullTrigger_2026D35+RecoFullGlobal_2026D35+HARVESTFullGlobal_2026D35.log

20434.0 step2
runTheMatrix-results/20434.0_TTbar_14TeV+TTbar_14TeV_TuneCUETP8M1_2026D41_GenSimHLBeamSpotFull14+DigiFullTrigger_2026D41+RecoFullGlobal_2026D41+HARVESTFullGlobal_2026D41/step2_TTbar_14TeV+TTbar_14TeV_TuneCUETP8M1_2026D41_GenSimHLBeamSpotFull14+DigiFullTrigger_2026D41+RecoFullGlobal_2026D41+HARVESTFullGlobal_2026D41.log

21234.0 step2
runTheMatrix-results/21234.0_TTbar_14TeV+TTbar_14TeV_TuneCUETP8M1_2026D44_GenSimHLBeamSpotFull14+DigiFullTrigger_2026D44+RecoFullGlobal_2026D44+HARVESTFullGlobal_2026D44/step2_TTbar_14TeV+TTbar_14TeV_TuneCUETP8M1_2026D44_GenSimHLBeamSpotFull14+DigiFullTrigger_2026D44+RecoFullGlobal_2026D44+HARVESTFullGlobal_2026D44.log

22034.0 step2
runTheMatrix-results/22034.0_TTbar_14TeV+TTbar_14TeV_TuneCUETP8M1_2026D46_GenSimHLBeamSpotFull14+DigiFullTrigger_2026D46+RecoFullGlobal_2026D46+HARVESTFullGlobal_2026D46/step2_TTbar_14TeV+TTbar_14TeV_TuneCUETP8M1_2026D46_GenSimHLBeamSpotFull14+DigiFullTrigger_2026D46+RecoFullGlobal_2026D46+HARVESTFullGlobal_2026D46.log

  • AddOn:

I found errors in the following addon tests:

cmsDriver.py RelVal -s HLT:PRef,RAW2DIGI,L1Reco,RECO --data --scenario=pp -n 10 --conditions auto:run2_data_PRef --relval 9000,50 --datatier "RAW-HLT-RECO" --eventcontent FEVTDEBUGHLT --customise=HLTrigger/Configuration/CustomConfigs.L1THLT --era Run2_2018 --processName=HLTRECO --filein file:RelVal_Raw_PRef_DATA.root --fileout file:RelVal_Raw_PRef_DATA_HLT_RECO.root : FAILED - time: date Mon Oct 21 20:32:18 2019-date Mon Oct 21 20:25:41 2019 s - exit: 16640
cmsDriver.py RelVal -s HLT:HIon,RAW2DIGI,L1Reco,RECO --data --scenario=pp -n 10 --conditions auto:run2_data_HIon --relval 9000,50 --datatier "RAW-HLT-RECO" --eventcontent FEVTDEBUGHLT --customise=HLTrigger/Configuration/CustomConfigs.L1THLT --era Run2_2018_pp_on_AA --processName=HLTRECO --filein file:RelVal_Raw_HIon_DATA.root --fileout file:RelVal_Raw_HIon_DATA_HLT_RECO.root : FAILED - time: date Mon Oct 21 20:34:10 2019-date Mon Oct 21 20:25:46 2019 s - exit: 16640
cmsDriver.py RelVal -s HLT:PIon,RAW2DIGI,L1Reco,RECO --data --scenario=pp -n 10 --conditions auto:run2_data_PIon --relval 9000,50 --datatier "RAW-HLT-RECO" --eventcontent FEVTDEBUGHLT --customise=HLTrigger/Configuration/CustomConfigs.L1THLT --era Run2_2018 --processName=HLTRECO --filein file:RelVal_Raw_PIon_DATA.root --fileout file:RelVal_Raw_PIon_DATA_HLT_RECO.root : FAILED - time: date Mon Oct 21 20:32:18 2019-date Mon Oct 21 20:26:00 2019 s - exit: 16640
cmsDriver.py RelVal -s HLT:GRun,RAW2DIGI,L1Reco,RECO --data --scenario=pp -n 10 --conditions auto:run2_data_GRun --relval 9000,50 --datatier "RAW-HLT-RECO" --eventcontent FEVTDEBUGHLT --customise=HLTrigger/Configuration/CustomConfigs.L1THLT --era Run2_2018 --processName=HLTRECO --filein file:RelVal_Raw_GRun_DATA.root --fileout file:RelVal_Raw_GRun_DATA_HLT_RECO.root : FAILED - time: date Mon Oct 21 20:40:32 2019-date Mon Oct 21 20:26:29 2019 s - exit: 16640

@cmsbuild
Copy link
Contributor

Comparison not run due to runTheMatrix errors (RelVals and Igprof tests were also skipped)

@Dr15Jones
Copy link
Contributor Author

This pull request also needs #28226, #28227, #28228 and #28229. These four fix the problems seen in the 'please test' results.

@Dr15Jones
Copy link
Contributor Author

#28232 fixes unit test failures seen in this pull request

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 6, 2019

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 6, 2019

Comparison job queued.

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 6, 2019

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-f4cc8f/3361/summary.html

Comparison Summary:

  • No significant changes to the logs found
  • Reco comparison results: 0 differences found in the comparisons
  • DQMHistoTests: Total files compared: 34
  • DQMHistoTests: Total histograms compared: 2939026
  • DQMHistoTests: Total failures: 1
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 2938684
  • DQMHistoTests: Total skipped: 341
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 0.0 KiB( 33 files compared)
  • Checked 147 log files, 16 edm output root files, 34 DQM output files

@Dr15Jones
Copy link
Contributor Author

@fabiocos this looks good to go finally :).

@fabiocos
Copy link
Contributor

fabiocos commented Nov 6, 2019

+1

@Dr15Jones in next IB this development should be fully integrated

@cmsbuild cmsbuild merged commit 4460405 into cms-sw:master Nov 6, 2019
@Dr15Jones Dr15Jones deleted the enforceESGetTokenInESProducers branch November 7, 2019 22:10
fwyzard added a commit to cms-patatrack/cmssw that referenced this pull request Nov 29, 2019
fwyzard added a commit to cms-patatrack/cmssw that referenced this pull request Oct 8, 2020
fwyzard added a commit to cms-patatrack/cmssw that referenced this pull request Oct 19, 2020
fwyzard added a commit to cms-patatrack/cmssw that referenced this pull request Oct 20, 2020
fwyzard added a commit to cms-patatrack/cmssw that referenced this pull request Oct 23, 2020
fwyzard added a commit to cms-patatrack/cmssw that referenced this pull request Nov 6, 2020
fwyzard added a commit to cms-patatrack/cmssw that referenced this pull request Nov 16, 2020
fwyzard added a commit to cms-patatrack/cmssw that referenced this pull request Nov 27, 2020
fwyzard pushed a commit to cms-patatrack/cmssw that referenced this pull request Dec 25, 2020
fwyzard added a commit to cms-patatrack/cmssw that referenced this pull request Dec 29, 2020
fwyzard added a commit to cms-patatrack/cmssw that referenced this pull request Dec 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants