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

PyReleaseValidation production-like workflow for Run3 (2021) #28773

Merged
merged 6 commits into from Jan 27, 2020

Conversation

fabiocos
Copy link
Contributor

@fabiocos fabiocos commented Jan 21, 2020

PR description:

This PR proposes a special set of Run3 (2021) workflows in a "production-like" setup, mimicking what already available for Phase2 by @kpedro88 . The purpose is to provide a tool for performance evaluation ready out-of-the-box, and it is not added to the standard set of workflows to avoid inflating it further for such a specific use case (so --what upgrade option needs to be invoked).

PR validation:

The added configuration cannot be tested directly by the bot. Here is the output:

12:35 cmsdev25 653> runTheMatrix.py --what upgrade -l 11634.21 --dryRun
ignoring non-requested file relval_standard
ignoring non-requested file relval_highstats
ignoring non-requested file relval_pileup
ignoring non-requested file relval_generator
ignoring non-requested file relval_extendedgen
ignoring non-requested file relval_production
ignoring non-requested file relval_ged
processing relval_upgrade
ignoring non-requested file relval_2017
ignoring non-requested file relval_2026
ignoring non-requested file relval_identity
ignoring non-requested file relval_machine
ignoring non-requested file relval_unsch
ignoring non-requested file relval_premix
Running in 4 thread(s)

Preparing to run 11634.21 TTbar_14TeV+TTbar_14TeV_TuneCUETP8M1_2021_GenSimFull+DigiFull_ProdLike_2021+RecoFull_ProdLike_2021+MiniAODFullGlobal_ProdLike_2021

# in: /build/fabiocos/111X/prodlikeWF/CMSSW_11_1_X_2020-01-21-2300/work dryRun for 'cd 11634.21_TTbar_14TeV+TTbar_14TeV_TuneCUETP8M1_2021_GenSimFull+DigiFull_ProdLike_2021+RecoFull_ProdLike_2021+MiniAODFullGlobal_ProdLike_2021
 cmsDriver.py TTbar_14TeV_TuneCUETP8M1_cfi  --conditions auto:phase1_2021_realistic -n 10 --era Run3 --eventcontent FEVTDEBUG --relval 9000,100 -s GEN,SIM --datatier GEN-SIM --beamspot Run3RoundOptics25ns13TeVLowSigmaZ --geometry DB:Extended --fileout file:step1.root  > step1_TTbar_14TeV+TTbar_14TeV_TuneCUETP8M1_2021_GenSimFull+DigiFull_ProdLike_2021+RecoFull_ProdLike_2021+MiniAODFullGlobal_ProdLike_2021.log  2>&1
 

# in: /build/fabiocos/111X/prodlikeWF/CMSSW_11_1_X_2020-01-21-2300/work dryRun for 'cd 11634.21_TTbar_14TeV+TTbar_14TeV_TuneCUETP8M1_2021_GenSimFull+DigiFull_ProdLike_2021+RecoFull_ProdLike_2021+MiniAODFullGlobal_ProdLike_2021
 cmsDriver.py step2  --conditions auto:phase1_2021_realistic -s DIGI,L1,DIGI2RAW,HLT:@relval2021 --datatier GEN-SIM-DIGI-RAW -n 10 --geometry DB:Extended --era Run3 --eventcontent RAWSIM --filein  file:step1.root  --fileout file:step2.root  > step2_TTbar_14TeV+TTbar_14TeV_TuneCUETP8M1_2021_GenSimFull+DigiFull_ProdLike_2021+RecoFull_ProdLike_2021+MiniAODFullGlobal_ProdLike_2021.log  2>&1
 

# in: /build/fabiocos/111X/prodlikeWF/CMSSW_11_1_X_2020-01-21-2300/work dryRun for 'cd 11634.21_TTbar_14TeV+TTbar_14TeV_TuneCUETP8M1_2021_GenSimFull+DigiFull_ProdLike_2021+RecoFull_ProdLike_2021+MiniAODFullGlobal_ProdLike_2021
 cmsDriver.py step3  --conditions auto:phase1_2021_realistic -n 10 --era Run3 --eventcontent AODSIM --runUnscheduled  -s RAW2DIGI,L1Reco,RECO,RECOSIM --datatier AODSIM --geometry DB:Extended --filein  file:step2.root  --fileout file:step3.root  > step3_TTbar_14TeV+TTbar_14TeV_TuneCUETP8M1_2021_GenSimFull+DigiFull_ProdLike_2021+RecoFull_ProdLike_2021+MiniAODFullGlobal_ProdLike_2021.log  2>&1
 

# in: /build/fabiocos/111X/prodlikeWF/CMSSW_11_1_X_2020-01-21-2300/work dryRun for 'cd 11634.21_TTbar_14TeV+TTbar_14TeV_TuneCUETP8M1_2021_GenSimFull+DigiFull_ProdLike_2021+RecoFull_ProdLike_2021+MiniAODFullGlobal_ProdLike_2021
 cmsDriver.py step4  --conditions auto:phase1_2021_realistic -n 10 --era Run3 --eventcontent MINIAODSIM --runUnscheduled  -s PAT --datatier MINIAODSIM --geometry DB:Extended --filein  file:step3.root  --fileout file:step4.root  > step4_TTbar_14TeV+TTbar_14TeV_TuneCUETP8M1_2021_GenSimFull+DigiFull_ProdLike_2021+RecoFull_ProdLike_2021+MiniAODFullGlobal_ProdLike_2021.log  2>&1
 
11634.21_TTbar_14TeV+TTbar_14TeV_TuneCUETP8M1_2021_GenSimFull+DigiFull_ProdLike_2021+RecoFull_ProdLike_2021+MiniAODFullGlobal_ProdLike_2021 Step0-PASSED Step1-PASSED Step2-PASSED Step3-PASSED  - time date Sun Jan 26 12:35:14 2020-date Sun Jan 26 12:35:14 2020; exit: 0 0 0 0
1 1 1 1 tests passed, 0 0 0 0 failed

@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-28773/13427

  • This PR adds an extra 20KB to repository

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @fabiocos (Fabio Cossutti) for master.

It involves the following packages:

Configuration/PyReleaseValidation

@chayanit, @cmsbuild, @pgunnell, @kpedro88, @zhenhu can you please review it and eventually sign? Thanks.
@makortel, @Martin-Grunewald this is something you requested to watch as well.
@davidlange6, @silviodonato, @fabiocos you are the release manager for this.

cms-bot commands are listed here

@fabiocos
Copy link
Contributor Author

please test

testing only that nothing old is broken, the addition needs to be tested with --what upgrade option for runTheMatrix.py

@fabiocos
Copy link
Contributor Author

@davidlange6 @slava77 FYI

@cmsbuild
Copy link
Contributor

cmsbuild commented Jan 21, 2020

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/4300/console Started: 2020/01/21 18:43

@cmsbuild
Copy link
Contributor

-1

Tested at: c67d7be

CMSSW: CMSSW_11_1_X_2020-01-21-1100
SCRAM_ARCH: slc7_amd64_gcc820
You can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-13ae09/4300/summary.html

I found follow errors while testing this PR

Failed tests: Python3

  • Python3:

I found errors:

*** Error compiling 'src/Configuration/PyReleaseValidation/python/upgradeWorkflowComponents.py'...

@cmsbuild
Copy link
Contributor

Comparison not run due to Build errors/Fireworks only changes/No short matrix requested (RelVals and Igprof tests were also skipped)

@kpedro88
Copy link
Contributor

@fabiocos I think the test failure is because of incorrect indentation, but see my review comment #28773 (comment) for a way to simplify the PR

@cmsbuild
Copy link
Contributor

cmsbuild commented Jan 22, 2020

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/4316/console Started: 2020/01/22 18:19

@cmsbuild
Copy link
Contributor

+1
Tested at: f2cb77a
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-13ae09/4316/summary.html
CMSSW: CMSSW_11_1_X_2020-01-22-1100
SCRAM_ARCH: slc7_amd64_gcc820

@cmsbuild
Copy link
Contributor

Comparison job queued.

@cmsbuild
Copy link
Contributor

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-13ae09/4316/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: 2697090
  • DQMHistoTests: Total failures: 1
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 2696743
  • DQMHistoTests: Total skipped: 346
  • 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

@fabiocos
Copy link
Contributor Author

@kpedro88 @chayanit further comments?

@chayanit
Copy link

@fabiocos , I have no comments about the code. For my knowledge, what difference is between prod-like and baseline workflows in general?

@fabiocos
Copy link
Contributor Author

@chayanit the main difference is the absence of DQM, and the output tiers are those used in production. For performance evaluation this may make a difference

@chayanit
Copy link

+1

@fabiocos thanks for your prompt response.

@kpedro88
Copy link
Contributor

@fabiocos can you update the PR description to indicate the new workflow is now 11634.21 rather than 11634.22?

@kpedro88
Copy link
Contributor

+upgrade

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @silviodonato, @fabiocos (and backports should be raised in the release meeting by the corresponding L2)

@fabiocos
Copy link
Contributor Author

@kpedro88 I have updated the description to leave a clean reference

@silviodonato
Copy link
Contributor

+1

@cmsbuild cmsbuild merged commit ceae341 into cms-sw:master Jan 27, 2020
@fabiocos fabiocos deleted the fc-prodlikeWF branch January 28, 2020 10:57
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