-
Notifications
You must be signed in to change notification settings - Fork 65
fix: boto3 session options #604
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: boto3 session options #604
Conversation
Hi @robmarkcole Can you try with this pr, setting: s3_uri = "s3://my-dummy-bucket-litdata/simple_data/"
ds = ld.StreamingDataset(s3_uri, s3_session_options = {"profile_name":"default"})
Make sure to either uninstall s5cmd, or set |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #604 +/- ##
===================================
Coverage 79% 79%
===================================
Files 41 41
Lines 6138 6143 +5
===================================
+ Hits 4838 4843 +5
Misses 1300 1300 🚀 New features to boost your workflow:
|
@deependujha now my test script returns
|
can you provide the traceback |
btw, can you try using |
@deependujha cli has no issues, it's boto3 not pickup up the sso creds somehow
|
from chatgpt: 🧠 TL;DRYour AWS CLI works fine because it supports AWS SSO out of the box, but boto3 isn't picking up the SSO credentials automatically unless extra setup is done. That’s why you're seeing:
🪵 What's really going on?When you use SSO-based profiles ( But: 🔥
|
@deependujha I've another test script which includes import boto3
session = boto3.Session(profile_name='my_profile') this one IS successful. As I say, for some reason litdata is not using the profile |
I tested it on lightning studio with profile and updated condition. It works for me. sorry for your inconvenience, can I get status on this? |
Same error |
@tchaton I assume you have a company AWS account and could test out SSO auth? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add an example ?
Before submitting
What does this PR do?
Fixes #603
Adds support to provide
s3_session_options
while creatings3_client
.PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in GitHub issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃