Skip to content
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

Update .env.local in README #6

Merged
merged 2 commits into from
Sep 28, 2023
Merged

Update .env.local in README #6

merged 2 commits into from
Sep 28, 2023

Conversation

kanterov
Copy link
Contributor

@kanterov kanterov commented Jun 16, 2020

Now when we support S3, use settings compatible with Flyte sandbox

Fixes flyteorg/flyte#332

@kanterov kanterov requested a review from kumare3 June 16, 2020 09:14
@kanterov
Copy link
Contributor Author

@kumare3 we have added support for S3. Can you please try it out, or find somebody else who can do that?

@wild-endeavor
Copy link

This is the issue for this PR right? flyteorg/flyte#332. Mind adding it to the description @kanterov ?

@kumare3
Copy link
Contributor

kumare3 commented Jun 16, 2020

So you already merged the s3 client in. We will try it in some time

@kumare3
Copy link
Contributor

kumare3 commented Jun 23, 2020

ok let me try it out now

@kanterov kanterov force-pushed the master branch 6 times, most recently from 0d535a3 to 9a4113b Compare June 14, 2021 19:24
Copy link
Member

@sbrunk sbrunk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just tried that successfully with a new sandbox. So LGTM with the suggested bucket name change.

README.md Outdated Show resolved Hide resolved
kanterov and others added 2 commits September 28, 2023 13:38
Now when we support S3, use settings compatible with Flyte sandbox

Signed-off-by: Hongxin Liang <[email protected]>
Co-authored-by: Sören Brunk <[email protected]>
Signed-off-by: Hongxin Liang <[email protected]>
@honnix honnix merged commit c656650 into master Sep 28, 2023
3 checks passed
@honnix honnix deleted the kanterov-patch-1 branch September 28, 2023 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] Flytekit-java, add support for S3
5 participants