continue with removing python2 versions

[smuzaffar]

backported from DEVEL IBs

[cmsbuild]

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

@cmsbuild, @smuzaffar, @mrodozov, @iarspider can you please review it and eventually sign? Thanks.
cms-bot commands are listed here

[smuzaffar]

@davidlange6 , can you please fix pip/requirement.txt file. last night I updated pillow and that is now creating conflicts with this PR. just update

pillow==8.2.0 ; python_version>'3.0'

in your branch

[cmsbuild]

Pull request #7008 was updated.

[cmsbuild]

Pull request #7008 was updated.

[davidlange6]

hum, I think I got it..

[smuzaffar]

test parameters:

• full_cmssw = true
• pull_request = change python2 to python3 in a few unit tests cmssw#33991

[smuzaffar]

please test

[smuzaffar]

@davidlange6 , in your next round, please

• drop py2-Pygments , it was only used by DXR but DXR is now pyY3 only.
• drop py2-pillow, it is only required by bokeh which we only build for py3

This should fix the 2 security issue https://github.com/cms-sw/cmsdist/security/dependabot

[davidlange6]

On Jun 10, 2021, at 8:43 AM, Malik Shahzad Muzaffar ***@***.***> wrote: @davidlange6 , in your next round, please • drop py2-Pygments , it was onlyused by DXR but DXR is now pyY3 only.

Its already gone.. fab1a81

• drop py2-pillow, it is only required by bokeh which we only builf for py3

Thats in my next round already

…

This should fix the 2 security issue https://github.com/cms-sw/cmsdist/security/dependabot — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

[smuzaffar]

please test
restart the tests with cms-sw/cms-bot#1562 which should allow to run clang, static analysis , py3, code rules on cmssw PRs only instead of full cmssw. Hopefully this save some PR test time

[smuzaffar]

please test
restarted after fixing the test parameters

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4261f3/15860/summary.html
COMMIT: 13ed469
CMSSW: CMSSW_12_0_X_2021-06-09-2300/slc7_amd64_gcc900
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmsdist/7008/15860/install.sh to create a dev area with all the needed externals and cmssw changes.

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:

• Oracle: update instantclient 19.11 #6986 @cms-sw: Oracle: update instantclient 19.11
• Update pillow to 8.2.0 to fix the security issue reported by GH #7005 @cms-sw: Update pillow to 8.2.0 to fix the security issue reported by GH
• [DXR] Py3 updates needed to avoid creashes with multi-threading #7004 @cms-sw: [DXR] Py3 updates needed to avoid creashes with multi-threading
• [DXR] Added deps on py3-Pygments which is required at runtime by dxr #7007 @cms-sw: [DXR] Added deps on py3-Pygments which is required at runtime by dxr
• [BuildRules] Fix issue with generating rootmap files during PR tests #7006 @cms-sw: [BuildRules] Fix issue with generating rootmap files during PR tests

You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4261f3/15860/git-recent-commits.json
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4261f3/15860/git-merge-result

Comparison Summary

Summary:

• No significant changes to the logs found
• Reco comparison results: 4 differences found in the comparisons
• DQMHistoTests: Total files compared: 38
• DQMHistoTests: Total histograms compared: 2862520
• DQMHistoTests: Total failures: 7
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2862491
• DQMHistoTests: Total skipped: 22
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 37 files compared)
• Checked 160 log files, 37 edm output root files, 38 DQM output files
• TriggerResults: no differences found

[smuzaffar]

+externals

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_12_0_X/master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2)