search

NASA-PDS / registry-sweepers

Scripts that run regularly on the registry database, to clean and consolidate information
Apache License 2.0
0 stars 1 forks source link

40 add logging aggregation, LOGLEVEL env var, other logging tweaks #45

Closed alexdunnjpl closed 11 months ago

alexdunnjpl commented 11 months ago

🗒️ Summary

Tweaks logging in a handful of ways (see commit messages)

Biggest one is that product-level errors/warns are now aggregated, while maintaining explanatory details for troubleshooting.

Log level can now be set in dockerized processes via the LOGLEVEL environment variable

⚙️ Test Data and/or Report

Unit tests pass

♻️ Related Issues

fixes #40

sonatype-lift[bot] commented 11 months ago

Sonatype Lift is retiring

Sonatype Lift will be retiring on Sep 12, 2023, with its analysis stopping on Aug 12, 2023. We understand that this news may come as a disappointment, and Sonatype is committed to helping you transition off it seamlessly. If you’d like to retain your data, please export your issues from the web console. We are extremely grateful and thank you for your support over the years.

📖 Read about the impacts and timeline

alexdunnjpl commented 11 months ago

@tloubrieu-jpl could I trouble you to open tickets for your two comments? The issues raised relate to longstanding existing code and are out-of-scope for this PR (i.e. I've left it better than I found it and y'all need these fixes yesterday).

Happy to pick up these improvements once we're out of the woods on the immediate ops issues.

EDIT

I don't wnat to overcomplicate the changes right now, so let say it is for comment/discussion, not for change.

Helps if I read the whole thing before firing off a response :sweat_smile: