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
add TestOldDigi workflow #28775
add TestOldDigi workflow #28775
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-28775/13430
|
A new Pull Request was created by @kpedro88 (Kevin Pedro) for master. It involves the following packages: Configuration/PyReleaseValidation @chayanit, @cmsbuild, @pgunnell, @kpedro88, @zhenhu can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
def setup_(self, step, stepName, stepDict, k, properties): | ||
if 'Reco' in step: | ||
# use existing DIGI-RAW file from old release | ||
stepDict[stepName][k] = merge([{'--filein': 'das:/RelValTTbar_14TeV/CMSSW_11_0_0_pre13-110X_mcRun4_realistic_v2_2026D49noPU-v1/GEN-SIM-DIGI-RAW'}, stepDict[step][k]]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we use the 11.0.0 relval, /RelValTTbar_14TeV/CMSSW_11_0_0-110X_mcRun4_realistic_v2_2026D49noPU-v2/GEN-SIM-DIGI-RAW
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
since there's no PU one (yet), I was just going to wait and update it later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
makes sense
test parameters |
please test |
The tests are being triggered in jenkins.
|
+1 |
Comparison job queued. |
Comparison is ready @slava77 comparisons for the following workflows were not done due to missing matrix map:
Comparison Summary:
|
+upgrade |
+1 |
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) |
+1 |
Hi @kpedro88 ,
the cmsDriver command is
I think the reason is |
@silviodonato see #28782 |
PR description:
A new special upgrade workflow is created to allow testing of 11_0_X GEN-SIM-DIGI-RAW files in 11_1_X. This supports the compatibility needed for the planned HLT TDR re-reco campaign.
The new workflows 23234.1001, 23434.1001 (without and with PU) are forwarded to the regular matrix so they can be called on demand in PR tests.
Technically, this PR required a few additional modifications in
relval_steps.py
to allow PU-specific changes by special workflows, as well as improved handling of steps that are None (and therefore should be removed by the matrix read).PR validation:
Ran workflow 23234.1001 locally and it succeeded.
attn: @fwyzard @trtomei