-
Notifications
You must be signed in to change notification settings - Fork 265
List maxmem tests with FLAVORS later. #2653
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
Conversation
|
A new Pull Request was created by @gartung for branch master. @akritkbehera, @cmsbuild, @iarspider, @raoatifshad, @smuzaffar can you please review it and eventually sign? Thanks. |
|
cms-bot internal usage |
|
Pull request #2653 was updated. |
1 similar comment
|
Pull request #2653 was updated. |
|
Pull request #2653 was updated. |
|
Pull request #2653 was updated. |
|
Pull request #2653 was updated. |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-886287/50668/summary.html Comparison SummaryThe workflows 2024.0030001 have different files in step1_dasquery.log than the ones found in the baseline. You may want to check and retrigger the tests if necessary. You can check it in the "files" directory in the results of the comparisons Summary:
|
|
Pull request #2653 was updated. |
|
please test |
|
Pull request #2653 was updated. |
|
Moving the sorting of the maxmem errors by workflow to another PR after testing logic locally. |
|
+externals trivial log message update |
|
This pull request is fully signed and it will be integrated in one of the next master IBs after it passes the integration tests. This pull request will now be reviewed by the release team before it's merged. @ftenchini, @mandrenguyen, @sextonkennedy (and backports should be raised in the release meeting by the corresponding L2) |
Put the flavor of the test at the end of the text line so they appear later in the summary message when sorted by alphabetical order.