Take advantage of simplified logging

[knoepfel]

Follow-on to HEPCloud/decisionengine#515.

[pep8speaks]

Hello @knoepfel! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2021-09-27 16:22:23 UTC

[codecov]

Codecov Report

Merging #366 (127bdfc) into master (f4752ab) will decrease coverage by 1.04%.
The diff coverage is 43.28%.

@@            Coverage Diff             @@
##           master     #366      +/-   ##
==========================================
- Coverage   37.30%   36.26%   -1.05%     
==========================================
  Files          80       80              
  Lines        3522     3444      -78     
  Branches      565      576      +11     
==========================================
- Hits         1314     1249      -65     
+ Misses       2118     2104      -14     
- Partials       90       91       +1     

Flag	Coverage Δ
python-3.6	36.16% <43.28%> (-1.05%)	⬇️
python-3.9	36.26% <43.28%> (-1.05%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...gine_modules/AWS/publishers/AWS_figure_of_merit.py	57.14% <ø> (-2.86%)	⬇️
...ne_modules/AWS/publishers/AWS_generic_publisher.py	77.14% <0.00%> (-0.64%)	⬇️
...ne_modules/AWS/publishers/AWS_price_performance.py	60.00% <ø> (-2.50%)	⬇️
...gine_modules/AWS/sources/AWSInstancePerformance.py	100.00% <ø> (ø)
...decisionengine_modules/AWS/sources/AWSJobLimits.py	86.95% <ø> (-0.55%)	⬇️
...decisionengine_modules/AWS/sources/AWSOccupancy.py	0.00% <0.00%> (ø)
...modules/AWS/sources/AWSOccupancyWithSourceProxy.py	69.69% <ø> (-0.46%)	⬇️
...decisionengine_modules/AWS/sources/AWSSpotPrice.py	0.00% <0.00%> (ø)
.../decisionengine_modules/AWS/sources/BillingInfo.py	0.00% <ø> (ø)
...nengine_modules/AWS/sources/FinancialParameters.py	0.00% <ø> (ø)
... and 44 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f4752ab...127bdfc. Read the comment docs.

[jcpunk]

Can you rebase off HEAD?

[BrunoCoimbra]

Can we also use the new logging boilerplate model for Sources and Publishers as well?
e.g.: FactoryClientManifests and JobClusteringPublisher

[knoepfel]

Good catch, @BrunoCoimbra. I'll update the PR.

[knoepfel]

@BrunoCoimbra, I adjusted the logging according to your suggestions. I also adjusted various utility functions so that a logger is "passed in" to the function if logging is desired--this way, we associate the messages from the utility function with the module that uses that utility.