Rename Phase2Timing_resources to resources - #2828
Conversation
|
A new Pull Request was created by @Parsifal-2045 for branch master. @akritkbehera, @cmsbuild, @iarspider, @raoatifshad, @smuzaffar can you please review it and eventually sign? Thanks. |
|
cms-bot internal usage |
|
test parameters:
|
|
@cmsbuild, please test |
|
Pull request #2828 was updated. |
|
@cmsbuild, please test |
|
@cmsbuild, please test for CMSSW_20_0_X |
|
-1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-78d331/55561/summary.html This pull request cannot be automatically merged, could you please rebase it? |
|
@cmsbuild, please test |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-78d331/55563/summary.html HLT P2 Timing: chart Comparison SummarySummary:
AMD_MI300X Comparison SummarySummary:
AMD_W7900 Comparison SummarySummary:
NVIDIA_H100 Comparison SummarySummary:
NVIDIA_L4 Comparison SummarySummary:
NVIDIA_L40S Comparison SummarySummary:
NVIDIA_T4 Comparison SummarySummary:
|
|
test parameters:
|
|
Pull request #2828 was updated. |
|
I managed to get in a quick fix, it should follow the better suggestion, thanks! |
|
@cmsbuild, please test |
|
@cmsbuild, please test for CMSSW_20_0_X |
|
please test with cms-sw/cmssw#51766 As current 20.1.X tests are running with cmssw change, so I see that https://cmssdt.cern.ch/jenkins/job/ib-run-pr-hlt_p2_timing/528/console is using let rerun 20.1.X tests with cmssw PR |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-78d331/55652/summary.html HLT P2 Timing: chart Comparison SummarySummary:
|
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-78d331/55653/summary.html HLT P2 Timing: chart Comparison SummarySummary:
AMD_MI300X Comparison SummarySummary:
AMD_W7900 Comparison SummarySummary:
NVIDIA_H100 Comparison SummarySummary:
NVIDIA_L4 Comparison SummarySummary:
NVIDIA_L40S Comparison SummarySummary:
NVIDIA_T4 Comparison SummarySummary:
Max Memory Comparisons exceeding threshold NVIDIA_H100@cms-sw/core-l2 , I found 1 workflow step(s) with memory usage exceeding the error threshold: Expand to see workflows ...
Max Memory Comparisons exceeding threshold NVIDIA_L40S@cms-sw/core-l2 , I found 1 workflow step(s) with memory usage exceeding the error threshold: Expand to see workflows ...
|
|
@mmusich , can you please checkif hlt p2 timing tests have generated the correct results/output? |
|
@smuzaffar it looks like things did run correctly for 20_1_X.
I am not sure what the problem might be. |
|
@cmsbuild, please test for CMSSW_20_0_X @mmusich , I also do not understand the issue with 20.0.X results. Looks like cms-bot/pr_testing/run-pr-hlt-p2-timing.sh Lines 43 to 44 in adf8838 ERROR: Failed to copy Generated CSV files message in log file but in jenkins job I do not see any failure in copying the csv files
let me re-run the tests. |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-78d331/55662/summary.html HLT P2 Timing: chart Comparison SummarySummary:
|
|
+externals looks good to go |
|
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. @mandrenguyen, @sextonkennedy, @ftenchini (and backports should be raised in the release meeting by the corresponding L2) |

Title says it all, some forward-looking changes in preparation for the reworked benchmark scripts cms-patatrack/patatrack-scripts#61