S3 logs

[bboynton97]

All logs go to file, then at the end of a trace, the file is uploaded with your API key for storage in S3

[codecov]

Codecov Report

Attention: Patch coverage is 61.42857% with 27 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
agentops/client/api/versions/v4.py	5.55%	17 Missing ⚠️
agentops/logging/instrument_logging.py	83.33%	7 Missing ⚠️
agentops/sdk/processors.py	66.66%	2 Missing ⚠️
agentops/sdk/core.py	50.00%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

[Dwij1704]

The agentops/common directory is currently being used as a shared space for various instrumentation-related functions and utilities. However, logging seems to function as a separate instrumentor in itself. I suggest moving log instrumentation to /agentops/logging cause we might move instrumentation out of agentops for better maintenance, It makes sense to manage and update instrumentation packages independently rather than bumping the sdk itself every time an instrumentation needs update.

[bboynton97]

The agentops/common directory is currently being used as a shared space for various instrumentation-related functions and utilities. However, logging seems to function as a separate instrumentor in itself. I suggest moving log instrumentation to /agentops/logging cause we might move instrumentation out of agentops for better maintenance, It makes sense to manage and update instrumentation packages independently rather than bumping the sdk itself every time an instrumentation needs update.

I totally understand this sentiment. We're instrumenting print and other loggers in this function so to me it felt like it belonged better in instrumentation, but I see what you're saying with instrumentation packages. i can move it :)

[tcdent]

LGTM