Fix #11239: Replaced instances of logging.warn to logging.warning

[d-ylee]

Fixes #11239

Status

Ready

Description

Since logging.warn is deprecated, instances of logging.warn have been replaced with logging.warning.

Is it backward compatible (if not, which system it affects?)

YES

Related PRs

N/A

External dependencies / deployment changes

No

[cmsdmwmbot]

Jenkins results:

• Python3 Unit tests: succeeded
  • 1 tests no longer failing
  • 2 changes in unstable tests
• Python3 Pylint check: failed
  • 29 warnings and errors that must be fixed
  • 9 warnings
  • 106 comments to review
• Pylint py3k check: succeeded
• Pycodestyle check: succeeded
  • 71 comments to review

Details at https://cmssdt.cern.ch/dmwm-jenkins/view/All/job/DMWM-WMCore-PR-test/14597/artifact/artifacts/PullRequestReport.html

[amaltaro]

@d-ylee Dennis, thank you for creating this pull request. I have the impression we had also seen this "warn" usage in one of the test/ files, no?

[d-ylee]

@amaltaro Thanks for the reminder. It looks like my search of logging.warnmissed the logger.warn in the test file. I also found a special case in Requests.py

[cmsdmwmbot]

Jenkins results:

• Python3 Unit tests: succeeded
  • 1 tests no longer failing
  • 1 changes in unstable tests
• Python3 Pylint check: failed
  • 31 warnings and errors that must be fixed
  • 9 warnings
  • 183 comments to review
• Pylint py3k check: failed
  • 2 errors and warnings that should be fixed
• Pycodestyle check: succeeded
  • 72 comments to review

Details at https://cmssdt.cern.ch/dmwm-jenkins/view/All/job/DMWM-WMCore-PR-test/14598/artifact/artifacts/PullRequestReport.html

[cmsdmwmbot]

Jenkins results:

• Python3 Unit tests: succeeded
  • 1 tests no longer failing
  • 2 changes in unstable tests
• Python3 Pylint check: failed
  • 31 warnings and errors that must be fixed
  • 11 warnings
  • 242 comments to review
• Pylint py3k check: failed
  • 2 errors and warnings that should be fixed
• Pycodestyle check: succeeded
  • 95 comments to review

Details at https://cmssdt.cern.ch/dmwm-jenkins/view/All/job/DMWM-WMCore-PR-test/14600/artifact/artifacts/PullRequestReport.html

[amaltaro]

@d-ylee thanks Dennis. Changes look good to me.
Just an observation, the 2nd commit contains src/* code changes as well. But this is your 1st PR in WMCore, so we are all good ;)