-
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
short name also in the final report #40235
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-40235/33284
|
A new Pull Request was created by @vlimant (vlimant) for master. It involves the following packages:
@bbilin, @cmsbuild, @AdrianoDee, @srimanob, @kskovpen, @sunilUIET can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
-1 Failed Tests: RelVals-INPUT RelVals-INPUTThe relvals timed out after 4 hours. Comparison SummarySummary:
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-bc1358/29449/summary.html Comparison SummarySummary:
|
+upgrade |
+upgrade |
yes, I restarted the failed RelVal INPUT test directly in Jenkins |
@perrotta, @rappoccio can we get this in for next IB. This should fix the IB Relval log issue reported here |
+pdmv |
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) |
@perrotta, @rappoccio can we merge this please. This fixes the IB dashboard issue with load relvals logs |
+1 |
PR description:
remove the step list from the final matrix report, as requested in #40106