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

[fastlane] migrate from AWS-SDK v2 to AWS-SDK v3 #16143

Merged
merged 3 commits into from Mar 18, 2020
Merged

[fastlane] migrate from AWS-SDK v2 to AWS-SDK v3 #16143

merged 3 commits into from Mar 18, 2020

Conversation

Kaspik
Copy link
Contributor

@Kaspik Kaspik commented Mar 9, 2020

Checklist

  • I've run bundle exec rspec from the root directory to see all new and existing tests pass
  • I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
  • I've read the Contribution Guidelines
  • I've updated the documentation if necessary.

Motivation and Context

Fixes #16130.
Repairs #15200.

Description

Currently Fastlane uses AWS-SDK v2 and is deprecated.
We could easily migrate to v3 just by changing the version number (via https://docs.aws.amazon.com/sdk-for-ruby/v3/api/ - Upgrade from version 2 - If you depend on aws-sdk or aws-sdk-resources, you don't need to change anything. Meanwhile we recommend you to revisit following options to explore modularization benefits.), but because we need only S3 part of the library, and AWS-SDK V3 introduced modularization, it makes sense not to include 200 sub-dependencies, but point only to the required one.

Testing Steps

Everything mentioned in Checklist.

Other notes

Introducing AWS-SDK in latest Fastlane causes a lot of issues so I think there should be a minor hotfix release.

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@Kaspik
Copy link
Contributor Author

Kaspik commented Mar 9, 2020

@googlebot I signed it!

@googlebot
Copy link

CLAs look good, thanks!

ℹ️ Googlers: Go here for more info.

@googlebot googlebot added cla: yes and removed cla: no labels Mar 9, 2020
@Kaspik
Copy link
Contributor Author

Kaspik commented Mar 9, 2020

I'm pretty sure tests are green, but not sure why that one failed. Locally everything works.

@liamnichols
Copy link
Contributor

liamnichols commented Mar 9, 2020

Ahh amazing, I've been running around in circles trying to find out why bundler has started getting stuck resolving dependencies.

We have a private plugin that is using aws-sdk-s3 as well and with a simple Gemfile like the following:

source 'https://rubygems.org'

gem 'fastlane'
gem 'fastlane-plugin-my_private_plugin' # .gemspec contains `spec.add_dependency 'aws-sdk-s3'`

Running bundle install will now loop indefinitely due to some madness within the aws-sdk gemspec that I haven't quite figured out just yet.

I tried using your fork and everything now works again 🙏

Edit: I just read the linked ticket, wish I found that earlier 😅

@Kaspik
Copy link
Contributor Author

Kaspik commented Mar 9, 2020

My fork will be removed once that happens, until then feel free to use it. :)

Copy link
Member

@janpio janpio left a comment

Choose a reason for hiding this comment

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

LGTM

@Kaspik
Copy link
Contributor Author

Kaspik commented Mar 9, 2020

Let me know @joshdholtz what you think about this, as I mentioned - I think this could be a reason for hotfix (minor 2.143.1) release.

@liamnichols
Copy link
Contributor

BTW the workaround I suggested in #16130 isn't really ideal in many scenarios since the bloat from the non-modularised version of the v3 SDK is quiet significant... It would be reallllllyyyy awesome if this simple change could make it into the next release 🙏

@joshdholtz joshdholtz changed the title Migrate from AWS-SDK v2 to AWS-SDK v3 [fastlane] migrate from AWS-SDK v2 to AWS-SDK v3 Mar 18, 2020
Copy link
Member

@joshdholtz joshdholtz left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this! I've pulled this down and tested it with my S3 integration and things seem good 😊 Appreciate the contribution ❤️

@joshdholtz joshdholtz merged commit 13e1b2a into fastlane:master Mar 18, 2020
@Kaspik Kaspik deleted the patch-1 branch March 18, 2020 14:33
@liamnichols
Copy link
Contributor

Thanks @joshdholtz and @Kaspik for this! 🙏

@fastlane-bot
Copy link

Hey @Kaspik 👋

Thank you for your contribution to fastlane and congrats on getting this pull request merged 🎉
The code change now lives in the master branch, however it wasn't released to RubyGems yet.
We usually ship about once a week, and your PR will be included in the next one.

Please let us know if this change requires an immediate release by adding a comment here 👍
We'll notify you once we shipped a new release with your changes 🚀

@Kaspik
Copy link
Contributor Author

Kaspik commented Mar 18, 2020

I think it should go sooner than later, and it has been 2 weeks since last release. :)

@joshdholtz
Copy link
Member

@Kaspik Will be making a new release today and keeping up better in the future! My apologies on not getting a release out. Been balancing a lot of stuff lately and I've let things slip 😔

@Kaspik
Copy link
Contributor Author

Kaspik commented Mar 23, 2020

Glad to hear that @joshdholtz , thanks! :)

Copy link

@fastlane-bot fastlane-bot left a comment

Choose a reason for hiding this comment

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

Congratulations! 🎉 This was released as part of fastlane 2.144.0 🚀

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

Successfully merging this pull request may close these issues.

aws-sdk with version 2, upgrade to v3
6 participants