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
Adding RelVal Global Tags for Run2 data #13280
Conversation
A new Pull Request was created by @mmusich (Marco Musich) for CMSSW_8_0_X. It involves the following packages: Configuration/AlCa @diguida, @boudoul, @franzoni, @cerminar, @fabozzi, @cmsbuild, @srimanob, @mmusich, @hengne, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are list here #13028 |
please test |
The tests are being triggered in jenkins. |
+1 |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_8_0_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @Degano, @smuzaffar |
@mmusich - what are the run 1 GT changes? |
@davidlange6 no specific run changes, run1 and run2 GT keys are the same since beginning of the 80X release cycle, so that when we add new records, like here for the benefit of #13115, both keys are involved. |
ah - maybe we can also unify the keys in the alca matrix - I guess I will continue to be confused:)
|
@davidlange6 we could do that, but we would need also to clean the relval steps configuration and probably several tens of occurrences in privately owned configuration files. Shall I do this with this PR? |
is not an emergency! but seems like the right solution assuming you plan to keep one GT for data (I guess you do)
|
+1 |
Adding RelVal Global Tags for Run2 data
Addresses issues as outlined in:
https://hypernews.cern.ch/HyperNews/CMS/get/relval/4478/1/2/1/1/2/1/1.html
To avoid mergeing conflicts it takes commits to
Configuration/PyReleaseValidation/python/relval_steps.py
from #13267 and also toConfiguration/AlCa/python/autoCond.py
from #13115.Differences of the newly introduced Global Tags:
RunII data