-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
renaming 2019 scenario to 2021 ... #26535
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-26535/9416
|
A new Pull Request was created by @boudoul (boudoul) for master. It involves the following packages: CalibCalorimetry/CaloTPG @perrotta, @cmsbuild, @prebello, @Dr15Jones, @alja, @cvuosalo, @civanch, @tlampen, @christopheralanwest, @ianna, @mdhildreth, @pgunnell, @rekovic, @franzoni, @tocheng, @zhenhu, @slava77, @pohsun, @fabiocos, @kpedro88, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
ah good point - Actually I would be happy that #26463 goes first (because it is not adressing exactly the same topic so better to factorize into 2 PRs ) . So when #26463 is merged I will (likely have to ) rebase this one, but that's fine (and it presumably doesn't prevent to start the review of this one, given the number of signatures.... ) |
please test |
The tests are being triggered in jenkins. |
@boudoul while we are considering this topic: should we rename this as Run3_2021, and then plan for later introduction of Run3_2022 and Run3_2023 for the subsequent years, if necessary? (Hopefully we will have less churn than we did in Run2...) |
Comparison job queued. |
+1
|
+1 |
+operations |
+upgrade |
+1 no objections |
+1 |
@mmusich right, but your new PR should fix this, am I correct? |
Yes, indeed that should be the case. |
PR description:
As decided by PPD (@franzoni ,@srimanob ) the 2019 scenario should be renamed to 2021 - This PR is adressing this - There were quite a lot of changes. I hope I didnt miss any.
Of course various reviewers can comment if for some special cases they want to keep the name of '2019' .
PR validation:
It is compiling (checkdeps is bringing a lot of packages to compile) -
I ran one of the ex-2019, now 2021, WF : TTBar number 11624.0 - Was running smoothly.