-
Notifications
You must be signed in to change notification settings - Fork 246
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
Add resources-diff.py. Run script on FastTimerService resources.json output of PR versus IB release. #2343
Conversation
…R versus IB release.
A new Pull Request was created by @gartung for branch master. @aandvalenzuela, @cmsbuild, @iarspider, @smuzaffar can you please review it and eventually sign? Thanks. |
cms-bot internal usage |
Pull request #2343 was updated. |
enable profiling |
please test |
Pull request #2343 was updated. |
please test |
Pull request #2343 was updated. |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8e18c1/41754/summary.html Comparison SummarySummary:
|
please test |
…. Add links to profiling workflow summary.
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8e18c1/41822/summary.html Comparison SummarySummary:
|
Pull request #2343 was updated. |
Pull request #2343 was updated. |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8e18c1/41823/summary.html Comparison SummarySummary:
|
@gartung , is this ready to go in? |
Almost. I need to correct the links on the profiling summary page. |
ping me when it is ready |
Pull request #2343 was updated. |
@smuzaffar it should be ready to go. |
No description provided.