[AL-1672] Add profile_name to creds argument

[AbhinavTuli]

🚀 🚀 Pull Request

Checklist:

• My code follows the style guidelines of this project and the Contributing document
• I have commented my code, particularly in hard-to-understand areas
• I have kept the coverage-rate up
• I have performed a self-review of my own code and resolved any problems
• I have checked to ensure there aren't any other open Pull Requests for the same change
• I have described and made corresponding changes to the relevant documentation
• New and existing unit tests pass locally with my changes

Changes

[codecov]

Codecov Report

Merging #1450 (6593ee8) into main (b74da96) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1450      +/-   ##
==========================================
- Coverage   91.20%   91.15%   -0.05%     
==========================================
  Files         179      179              
  Lines       14550    14545       -5     
==========================================
- Hits        13270    13259      -11     
- Misses       1280     1286       +6     

Flag	Coverage Δ
unittests	91.15% <100.00%> (-0.05%)	⬇️

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

Impacted Files	Coverage Δ
hub/api/dataset.py	87.71% <ø> (ø)
hub/core/storage/s3.py	71.18% <100.00%> (-0.95%)	⬇️
hub/util/storage.py	96.55% <100.00%> (+3.56%)	⬆️
hub/core/lock.py	85.24% <0.00%> (-6.56%)	⬇️

---

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 b74da96...6593ee8. Read the comment docs.

[davidbuniat]

Please move S3 profile into creds, I don’t think it should be a separate argument for the end user

[farizrahman4u]

Not sure about adding storage specific srgs to core dataset and api.

changes made