Upload job result files to S3

[MrCreosote]

No description provided.

[MrCreosote]

codecov seems to be having issues again

[codecov]

Codecov Report

Attention: Patch coverage is 45.33333% with 41 lines in your changes missing coverage. Please review.

Project coverage is 55.18%. Comparing base (61fc7a6) to head (3bf25b2).
Report is 5 commits behind head on main.

Files with missing lines	Patch %	Lines
cdmtaskservice/nersc/manager.py	23.80%	16 Missing ⚠️
cdmtaskservice/jaws/output.py	51.72%	14 Missing ⚠️
cdmtaskservice/jobflows/nersc_jaws.py	28.57%	10 Missing ⚠️
cdmtaskservice/callback_url_paths.py	66.66%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #155      +/-   ##
==========================================
- Coverage   55.55%   55.18%   -0.37%     
==========================================
  Files          33       34       +1     
  Lines        2054     2120      +66     
==========================================
+ Hits         1141     1170      +29     
- Misses        913      950      +37     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Tianhao-Gu]

I am not sure why presign is a nested async function. But totally valid.

[MrCreosote]

it has to be async since the s3 client is async, and I'm passing in the function so that I don't add a dependency on the S3 client to the nersc manager. It shouldn't really care where the files end up, just that it gets presigned URLs. I should probably rename a bunch of stuff in the manager and s3 client to make that more clear, since the manager exclusively deals with presigned URLs which really could be for anything