-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Fix the python configuration for the TICLDumper #47576
base: master
Are you sure you want to change the base?
Conversation
cms-bot internal usage |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-47576/44061
|
A new Pull Request was created by @AuroraPerego for master. It involves the following packages:
@AdrianoDee, @Moanwar, @cmsbuild, @DickyChant, @jfernan2, @mandrenguyen, @miquork, @srimanob, @subirsarkar can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
test parameters:
|
please test |
-1 Failed Tests: RelVals RelValsValueError: Undefined workflows: 29688.206, 29688.207 |
test parameters:
|
please test |
-1 Failed Tests: RelVals RelVals-INPUT AddOn
RelVals
Expand to see more relval errors ...RelVals-INPUT
Expand to see more relval errors ...
AddOn Tests
Expand to see more addon errors ...
|
PR description:
When using the TICLDumper with the
ticl_v5
process modifier, the wrong collection of trackster was used. This PR fixes this.In addition, workflows that run also the dumper have been added.
PR validation:
dumped and ran the new workflows with
runTheMatrix.py -w upgrade -l 29688.206,29688.207
FYI @felicepantaleo @waredjeb @Moanwar