Skip to content
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

[10.4.X] [ROOT] Cherry-pick changes to resolve memory over-write due to execution of a I/O #4452

Merged
merged 1 commit into from
Nov 15, 2018

Conversation

smuzaffar
Copy link
Contributor

@smuzaffar
Copy link
Contributor Author

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Oct 26, 2018

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/31292/console

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @smuzaffar (Malik Shahzad Muzaffar) for branch IB/CMSSW_10_4_X/gcc700.

@cmsbuild, @smuzaffar, @gudrutis, @mrodozov can you please review it and eventually sign? Thanks.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.

@cmsbuild
Copy link
Contributor

-1

Tested at: fe76dd5

You can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4452/31292/summary.html

I found follow errors while testing this PR

Failed tests: RelVals

  • RelVals:

The relvals timed out after 2 hours.

@cmsbuild
Copy link
Contributor

Comparison not run due to runTheMatrix errors (RelVals and Igprof tests were also skipped)

@smuzaffar
Copy link
Contributor Author

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Oct 27, 2018

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/31320/console

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

Comparison job queued.

@cmsbuild
Copy link
Contributor

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4452/31320/summary.html

Comparison Summary:

  • No significant changes to the logs found
  • Reco comparison results: 178 differences found in the comparisons
  • DQMHistoTests: Total files compared: 32
  • DQMHistoTests: Total histograms compared: 2994843
  • DQMHistoTests: Total failures: 15
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 2994631
  • DQMHistoTests: Total skipped: 197
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 0.0 KiB( 31 files compared)
  • Checked 134 log files, 14 edm output root files, 32 DQM output files

@smuzaffar
Copy link
Contributor Author

+externals

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 1, 2018

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 1, 2018

Comparison job queued.

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 1, 2018

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4452/31414/summary.html

Comparison Summary:

  • No significant changes to the logs found
  • Reco comparison results: 178 differences found in the comparisons
  • DQMHistoTests: Total files compared: 32
  • DQMHistoTests: Total histograms compared: 2993155
  • DQMHistoTests: Total failures: 83
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 2992875
  • DQMHistoTests: Total skipped: 197
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 0.0 KiB( 31 files compared)
  • Checked 134 log files, 14 edm output root files, 32 DQM output files

@smuzaffar
Copy link
Contributor Author

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 15, 2018

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/31666/console

@fabiocos
Copy link
Contributor

@slava77 @perrotta there was any conclusion about these differences? I see that now @smuzaffar is producing again the set of plots for a further test

@smuzaffar
Copy link
Contributor Author

@fabiocos , the previous plots/logs were deleted that is why I am re-running.

@slava77
Copy link
Contributor

slava77 commented Nov 15, 2018

I didn't find time to make a local root build for further investigation.
it would be nice to get something scram-able centrally.

@smuzaffar
Copy link
Contributor Author

we can merge it for DEVEL IBs so that you can test there

@slava77
Copy link
Contributor

slava77 commented Nov 15, 2018

we can merge it for DEVEL IBs so that you can test there

sure, as long as it's the same 6.12 version in the DEVEL to compare with in the baseline as we have in the current 104X [just in case more may be already going on in the DEVEL which can affect this check/test]

@smuzaffar
Copy link
Contributor Author

only difference b/w normal and DEVEL IBs is boost version every thing else should be same.

@smuzaffar smuzaffar changed the base branch from IB/CMSSW_10_4_X/gcc700 to IB/CMSSW_10_4_X/gcc700next November 15, 2018 15:16
@smuzaffar smuzaffar merged commit 44ec98c into IB/CMSSW_10_4_X/gcc700next Nov 15, 2018
@smuzaffar
Copy link
Contributor Author

@slava77 , this is now merged for DEVEL IBs. Tonight's 23h DEVEL IB should have it.

@slava77
Copy link
Contributor

slava77 commented Nov 15, 2018

@slava77 , this is now merged for DEVEL IBs. Tonight's 23h DEVEL IB should have it.

thanks.
let's see if I manage to get to this in the next couple of days

@cmsbuild
Copy link
Contributor

-1

Tested at: fe76dd5

You can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4452/31666/summary.html

I found follow errors while testing this PR

Failed tests: RelVals

  • RelVals:

The relvals timed out after 2 hours.

@cmsbuild
Copy link
Contributor

Comparison not run due to runTheMatrix errors (RelVals and Igprof tests were also skipped)

@smuzaffar smuzaffar deleted the smuzaffar-patch-3 branch November 16, 2018 08:55
@smuzaffar
Copy link
Contributor Author

@slava77 , did you find time to look in to this? DEVEL IBs should have it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants