Skip to content
This repository has been archived by the owner on Apr 14, 2021. It is now read-only.

Retry gem downloads #4875

Merged
merged 1 commit into from
Aug 14, 2016
Merged

Retry gem downloads #4875

merged 1 commit into from
Aug 14, 2016

Conversation

jkeiser
Copy link
Contributor

@jkeiser jkeiser commented Aug 11, 2016

Fixes #4846

@segiddins
Copy link
Member

@indirect r?

@indirect
Copy link
Member

Not sure 100% of failures will be retried with that exception list, but I'm happy to start here.

@homu r+

@homu
Copy link
Contributor

homu commented Aug 14, 2016

📌 Commit ce82494 has been approved by indirect

homu added a commit that referenced this pull request Aug 14, 2016
@homu
Copy link
Contributor

homu commented Aug 14, 2016

⌛ Testing commit ce82494 with merge 0e286fd...

@homu
Copy link
Contributor

homu commented Aug 14, 2016

☀️ Test successful - status

@homu homu merged commit ce82494 into rubygems:master Aug 14, 2016
@jkeiser
Copy link
Contributor Author

jkeiser commented Aug 24, 2016

@indirect two things:

  1. We are getting bit by this a lot in our CI and this patch seems fairly safe (to me anyway :) ). Is there any possibility we can get this patch into a released bundler in the near future?
  2. Where would you like to see the retry at? I'm happy to make a new patch with it up further in the stack. This is going to be seen very intermittently, so it's very possible that people will never report it--I'd rather we just get it fully robust in one swoop while we have the energy.

@jkeiser jkeiser deleted the retry-download-gem branch August 24, 2016 16:30
@jkeiser
Copy link
Contributor Author

jkeiser commented Aug 24, 2016

@indirect regarding #2, here is what I considered the relevant stack (taken from #4846):

Gem::RemoteFetcher::FetchError: Errno::ECONNRESET: Connection reset by peer - SSL_connect (https://rubygems.org/gems/multipart-post-1.2.0.gem)
/opt/angry-omnibus-toolchain/embedded/lib/ruby/site_ruby/2.1.0/rubygems/remote_fetcher.rb:308:in `rescue in fetch_path'
/opt/angry-omnibus-toolchain/embedded/lib/ruby/site_ruby/2.1.0/rubygems/remote_fetcher.rb:281:in `fetch_path'
/opt/angry-omnibus-toolchain/embedded/lib/ruby/site_ruby/2.1.0/rubygems/remote_fetcher.rb:324:in `cache_update_path'
/opt/angry-omnibus-toolchain/embedded/lib/ruby/site_ruby/2.1.0/rubygems/remote_fetcher.rb:188:in `download'
/opt/angry-omnibus-toolchain/embedded/lib/ruby/gems/2.1.0/gems/bundler-1.12.5/lib/bundler/rubygems_integration.rb:615:in `download_gem'
/opt/angry-omnibus-toolchain/embedded/lib/ruby/gems/2.1.0/gems/bundler-1.12.5/lib/bundler/source/rubygems.rb:410:in `fetch_gem'
/opt/angry-omnibus-toolchain/embedded/lib/ruby/gems/2.1.0/gems/bundler-1.12.5/lib/bundler/source/rubygems.rb:116:in `install'
/opt/angry-omnibus-toolchain/embedded/lib/ruby/gems/2.1.0/gems/bundler-1.12.5/lib/bundler/installer/gem_installer.rb:57:in `install'
/opt/angry-omnibus-toolchain/embedded/lib/ruby/gems/2.1.0/gems/bundler-1.12.5/lib/bundler/installer/gem_installer.rb:15:in `install_from_spec'

@jkeiser
Copy link
Contributor Author

jkeiser commented Aug 24, 2016

I tried to put the retry as close to the boundary with rubygems as possible to have the maximum chance of catching all calls from bundler.

@chrismo
Copy link
Contributor

chrismo commented Aug 24, 2016

looks like it went out with 1.13.0.rc.2 on the 21st - ce82494

@jkeiser
Copy link
Contributor Author

jkeiser commented Aug 24, 2016

@chrismo hooray! Thanks!

@coilysiren coilysiren modified the milestone: Release Archive Sep 22, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants