-
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
Remove left over uGMT ZS configuration #40107
Remove left over uGMT ZS configuration #40107
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-40107/33093
|
A new Pull Request was created by @dinyar (Dinyar Rabady) for master. It involves the following packages:
@emanueleusai, @ahmad3213, @cmsbuild, @jfernan2, @syuvivida, @pmandrik, @micsucmed, @rvenditti can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
Hi @pmandrik, This should fix the issue in [0] that you reported. Can you let me know if you need a backport too? Cheers, |
type l1t |
please test |
@dinyar yes, could you please provide 12_5 and 12_4 backports? Thanks! |
-1 Failed Tests: RelVals-INPUT RelVals-INPUTThe relvals timed out after 4 hours. Comparison SummarySummary:
|
please test
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-84581b/29291/summary.html Comparison SummarySummary:
|
+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. @perrotta, @dpiparo, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
+1 errors are spurious. |
PR description:
This removes some left over calls to modules that were removed in #38115.
cc @pmandrik and @vukasinmilosevic
PR validation:
None should be required beyond the ones run in this MR.