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

Migrate to boto3 #43

Closed
piskvorky opened this issue Dec 1, 2015 · 20 comments
Closed

Migrate to boto3 #43

piskvorky opened this issue Dec 1, 2015 · 20 comments
Assignees
Labels

Comments

@piskvorky
Copy link
Owner

Check if boto3 is production-ready and migrate all code from boto to boto3.

@piskvorky piskvorky changed the title boto3 Migrate to boto3 Dec 1, 2015
@asieira
Copy link
Contributor

asieira commented Dec 16, 2015

I would suggest that even if boto3 is supported, existing implementation using boto 2 is kept as well.

There are many things boto3 hasn't implemented yet. To give one example, the lack of explicit proxy support would prevent me from continuing to use smart_open if it went boto3-only. See #35 for supporting evidence. :)

@piskvorky
Copy link
Owner Author

Thanks @asieira , that's the kind of practical feedback we need!

I don't have experience with boto3, and certainly wouldn't want to inconvenience users by a hasty switch.

@tmylk tmylk removed the help wanted label Jan 10, 2016
@brunoalano
Copy link

+1

@vinayan3
Copy link

vinayan3 commented Apr 1, 2016

+1. I'm having issues with Python 3.5 and Boto 2.39. From what I've read Python 3x is better with boto3

@tmylk
Copy link
Contributor

tmylk commented Oct 5, 2016

@vinayan3 What kind of issues are you having? Would be great to check they are resolved in boto3.

@tmylk
Copy link
Contributor

tmylk commented Oct 5, 2016

Update on Boto3 : Proxy support still not implemented in boto/boto3#338

@tmylk
Copy link
Contributor

tmylk commented May 18, 2017

Our test dependeny moto has moved to boto3 unfortunately in ver 1.0.0
We still can't migrate though because of proxy support

  • The normal @mock_ decorators will no longer work with boto. It is suggested that you upgrade to boto3 or use the standalone-server mode. If you would still like to use boto, you must use the @mock__deprecated decorators which will be removed in a future release.

@combinatorist
Copy link

combinatorist commented Jul 6, 2017

+1

  • I came in at boto3 and I just started using this library.
  • Currently, I use this library by string interpolating keys into paths
    • e.g. smart_open('s3://mybucket/' + key)
  • but I really wish I could use boto3 objects (particularly for the advertised smart_open.s3_iter_bucket function)

@tim-schier-frontier
Copy link

I'm using moto and would like to move to moto>=1, but smart_open is holding me back due to boto3 vs boto support.
Even using @mock_s3_deprecated for my other moto usage is causing issues now with moto 1.0.1. It looks like a moto bug, but only a bug in the deprecated support for boto.

So yeah, boto3 momentum is building. Oh and it looks like proxy support has been added to boto3 now (ticket @tmylk linked to is closed as implemented).

@piskvorky
Copy link
Owner Author

@tim-schier-frontier can you help with the migration, open a PR?

@tim-schier-frontier
Copy link

@piskvorky great follow up - put money where my mouth is :)
I am best placed to help with testing/trialing PRs, and also probably chipping in for funding the move (from my employer).
Love the product, hope to help out.

@mpenkov
Copy link
Collaborator

mpenkov commented Dec 10, 2017

I think boto3 now supports proxies: boto/boto3#338

The S3 component is already boto3-based. The tests are still boto-based, however. @menshikh-iv What should we do about this?

@menshikh-iv
Copy link
Contributor

@mpenkov probably migrate tests to boto3 & close this issue

@mpenkov
Copy link
Collaborator

mpenkov commented Dec 22, 2017

We should also upgrade moto to the most recent version.

@inksink
Copy link
Contributor

inksink commented Mar 30, 2018

I have read the PR 164 which will migrate tests to boto3.
But there are many codes used boto in smart_open_lib.py. Do we have such a plan to migrate smart_open_lib to boto3?
I wish to reduce unnecessary external dependencies in my project.

@menshikh-iv
Copy link
Contributor

@inksink yes, we want to migrate all to boto3
CC: @mpenkov

@carolingfish
Copy link

I noticed that commit on this has been merged. Any estimate on when the new package including the commit will be released?

@menshikh-iv
Copy link
Contributor

@carolingfish I think I'll release it in the current month

@bkapoorUF
Copy link

Any update on when this will be released?

@menshikh-iv
Copy link
Contributor

@bkapoorUF already released as smart_open==1.6.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests