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 a conflict with UpgradeProperties (102X) #26152
Fix a conflict with UpgradeProperties (102X) #26152
Conversation
A new Pull Request was created by @kfjack (Kai-Feng Chen) for CMSSW_10_2_X. It involves the following packages: Configuration/PyReleaseValidation @cmsbuild, @zhenhu, @prebello, @pgunnell, @kpedro88 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test workflow 10801.0,10802.0,10803.0,10825.0 |
The tests are being triggered in jenkins. |
Comparison job queued. |
Hi @fabiocos no problems. Locally I have changed the Era to inject usual relvals and it has called the ParkingBPH string by mistake. It has to be fixed to avoid issues with relvals not related to ParkingBPH data relvals. |
Comparison is ready @slava77 comparisons for the following workflows were not done due to missing matrix map:
Comparison Summary:
|
+1 |
+1 the backport is consistent with the version merged in master |
+upgrade |
This pull request is fully signed and it will be integrated in one of the next CMSSW_10_2_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_10_6_X is complete. This pull request will be automatically merged. |
PR description:
To avoid the conflict between the code in relval_steps.py and the Era/UpgradeProperties definition in upgradeWorkflowComponents.py. Attaching additional term to the Era definition in UpgradeProperties should not generate an error in the tests with this PR.
This is back port from PR #26151.