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

Fix Plugin Download Timeout #419

Merged
merged 2 commits into from
Nov 7, 2015

Conversation

mooreandrew
Copy link
Contributor

When downloading large plugins, it can timeout (e.g. ruby-runtime). The
archive::download supports a timeout value which defaults to 120
seconds, but this plugin class doesn’t have any support to override
this value. This allows for the ability to adjust the timeout value for
large plugins.

Error message when download timeout exceeds 120 seconds:

Error:
/Stage[main]/Jenkins::Plugins/Jenkins::Plugin[ruby-runtime]/Archive::Dow
nload[ruby-runtime.hpi]/Exec[download archive ruby-runtime.hpi and
check sum]/returns: change from notrun to 0 failed: Command exceeded
timeout

When downloading large plugins, it can timeout (e.g. ruby-runtime). The
archive::download supports a timeout value which defaults to 120
seconds, but this plugin class doesn’t have any support to override
this value. This allows for the ability to adjust the timeout value for
large plugins.

Error message when download timeout exceeds 120 seconds:

Error:
/Stage[main]/Jenkins::Plugins/Jenkins::Plugin[ruby-runtime]/Archive::Dow
nload[ruby-runtime.hpi]/Exec[download archive ruby-runtime.hpi and
check sum]/returns: change from notrun to 0 failed: Command exceeded
timeout
@mooreandrew mooreandrew changed the title Fix Download Timeout Fix Plugin Download Timeout Nov 7, 2015
@jenkinsadmin
Copy link

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

@rtyler
Copy link

rtyler commented Nov 7, 2015

I'm going to go ahead and merge this and add the rspec-puppet necessary myself

rtyler pushed a commit that referenced this pull request Nov 7, 2015
@rtyler rtyler merged commit 6fa5f5c into voxpupuli:master Nov 7, 2015
@rtyler rtyler added this to the 1.7.0 - Oddjob milestone Nov 7, 2015
rtyler pushed a commit to rtyler/puppet-jenkins that referenced this pull request Nov 7, 2015
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.

3 participants