[13_0_X] Update run3 data GTs #41611
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
backport of #41609 |
|
A new Pull Request was created by @francescobrivio for CMSSW_13_0_X. It involves the following packages:
@cmsbuild, @tvami, @saumyaphor4252, @francescobrivio can you please review it and eventually sign? Thanks. cms-bot commands are listed here
|
|
test parameters:
|
|
@cmsbuild please test |
|
-1 Failed Tests: RelVals RelVals-INPUT AddOn RelValsExpand to see more relval errors ...RelVals-INPUT
Expand to see more relval errors ...
AddOn TestsExpand to see more addon errors ... |
|
well, https://cmssdt.cern.ch/SDT/html/cmssdt-ib/#/ib/CMSSW_13_0_X are broken as well, seems cms-sw/cmsdist#8493 wasn't merged at the same time as #41434. |
Thanks Marco! Will this morning IB already work? Or should I wait for tonight to re-trigger the tests? |
I am not sure in which IB cms-sw/cmsdist#8493 was picked up. Maybe @smuzaffar can help clarifying that. |
|
test parameters:
|
no need to wait, we can just test together with the cms dist PR |
|
from
the morning IB should be fine. |
|
@cmsbuild , please test |
|
yes, CMSSW_13_0_X_2023-05-10-1200 which is still in build phase contains it. |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-78aa34/32538/summary.html The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Comparison SummarySummary:
|
|
+alca |
cmsbuild
added
alca-approved
fully-signed
and removed
alca-pending
pending-signatures
labels
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_13_0_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_13_2_X is complete. 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 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR description:
Backport of #41609
This PR updates the run3 data GTs:
Important Note:
In CMSSW_13_0_X we still had the 126X data GTs (both for online and offline) - so, with respect to the master PR, the GT differences linked here also show the tag cleanup that was included in 13_1_X (at the time master branch) in #41076.
GT differences:
2023 HLT data:
2023 Express data:
2023 Prompt data:
Run3 offline GT:
PR validation:
Code compiles - will use the bot to run the matrix tests
Backport:
Backport of #41609