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
Update GENSIM string for 11_3_X IB test #33090
Conversation
2026 GENSIM is still running so I will trigger the test when they finish (tomorrow) |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-33090/21414
|
A new Pull Request was created by @chayanit (Chayanit Asawatangtrakuldee) for master. It involves the following packages: Configuration/PyReleaseValidation @jordan-martins, @chayanit, @wajidalikhan, @kpedro88, @cmsbuild, @srimanob can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
-1 Failed Tests: RelVals-INPUT
RelVals-INPUT
Expand to see more relval errors ...
Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
As an example, looking at I see that the DAS query is empty because |
I had the impression the IB system would automatically cache files that are generally available but not at CERN. |
please test |
-1 Failed Tests: RelVals-INPUT RelVals-INPUT
Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
1 similar comment
-1 Failed Tests: RelVals-INPUT RelVals-INPUT
Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-933fe4/13599/summary.html Comparison SummarySummary:
|
I also have no clue @qliphy |
please test |
-1 Failed Tests: RelVals-INPUT RelVals-INPUT
Comparison SummarySummary:
|
@cms-sw/upgrade-l2 @cms-sw/pdmv-l2 we are changing the input GEN-SIM files, so the differences are expected. I think we can merge this PR as it is. |
+1 |
+Upgrade |
This pull request is fully signed and it will be integrated in one of the next master IBs (but tests are reportedly failing). This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
merge |
Update GENSIM string in 11_3_X according to #33010 for PPS workflows, and add back PPS workflows to IB test (removed in #33020).
In addition, I think we should also update all Run3 workflows to 14TeV instead of 13TeV. This is for TTbar design Run3 2021 (12024 to 12034)
All GENSIM have been produced in 11_3_0_pre4
https://dmytro.web.cern.ch/dmytro/cmsprodmon/workflows.php?campaign=CMSSW_11_3_0_pre4__FullSim_2017GSonly_forIBTest-1615036406
https://dmytro.web.cern.ch/dmytro/cmsprodmon/workflows.php?campaign=CMSSW_11_3_0_pre4__FullSim_2018GSonly_forIBTest-1615036939
https://dmytro.web.cern.ch/dmytro/cmsprodmon/workflows.php?campaign=CMSSW_11_3_0_pre4__FullSim_Run3GSonly_forIBTest-1615037975
https://dmytro.web.cern.ch/dmytro/cmsprodmon/workflows.php?campaign=CMSSW_11_3_0_pre4__FullSim_2026GSonly_forIBTest-1615039204