Backport edmStreamStallGrapher.py #16111
Conversation
Previously log file parser only used the first character of the stream id when converting from a string to a number. This change makes sure to get all the digits.
The algorithm for understanding the log files did not properly deal with logs from jobs which were running unscheduled.
Use the time difference between the end of one event on a stream and the start of the next event on a stream as an estimate for the maximum stall time the framework could have incurred from serializing the source.
|
A new Pull Request was created by @Dr15Jones (Chris Jones) for CMSSW_8_0_X. It involves the following packages: FWCore/Concurrency @cmsbuild, @smuzaffar, @Dr15Jones, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are list here #13028 |
|
please test |
|
The tests are being triggered in jenkins. |
|
+1 |
cmsbuild
added
core-approved
fully-signed
tests-started
and removed
core-pending
pending-signatures
tests-pending
labels
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_8_0_X IBs after it passes the integration tests. This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @smuzaffar |
|
This is a completely stand alone executable for which no code has any dependencies. |
|
+1 |
Backport appropriate fixes for edmStreamStallGrapher.py from CMSSW_8_1_X. This code now properly handles the output of the Tracer in CMSSW_8_0_X.