Chef

Resources need a retry parameter which allows the action to repeat a set number of times until success is achieved.

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 0.5.1, 0.5.2, 0.5.4, 0.5.6, 0.6.0, 0.6.2, 0.7.0, 0.7.2, 0.7.4, 0.7.6, 0.7.8, 0.7.10, 0.7.12, 0.7.14, 0.7.16, 0.8.2, 0.8.4, 0.8.6, 0.8.8, 0.8.10, 0.8.12, 0.8.14, 0.8.16, 0.9.0, 0.9.2, 0.9.4, 0.9.6, 0.9.8
  • Fix Version/s: 0.10.4
  • Component/s: Resources
  • Labels:
    None
  • Environment:

    any

  • Triage Status:
    Triaged

Description

gem_package needs to rescue connection errors and do exponential backoff and retry since rubyforge is unavailable often and this ends up ruining a whole chef run. i;d rather have chef sit there and retry to install a gem a few times before it gives up and fails as often you can just retry to install the gem and it will work.

Activity

Hide
AJ Christensen added a comment - 30/Jan/09 9:43 AM

There also appears to be no specifications at all for the rubygems provider.

Show
AJ Christensen added a comment - 30/Jan/09 9:43 AM There also appears to be no specifications at all for the rubygems provider.
Hide
AJ Christensen added a comment - 30/Jan/09 9:49 AM

Going to bump this from 0.5.2 to unknown as I don't consider it important enough to block release.

Ez, when you wake up if there is still time feel free to complete this for the milestone and have a word to Adam about getting it in.

Show
AJ Christensen added a comment - 30/Jan/09 9:49 AM Going to bump this from 0.5.2 to unknown as I don't consider it important enough to block release. Ez, when you wake up if there is still time feel free to complete this for the milestone and have a word to Adam about getting it in.
Hide
Adam Jacob added a comment - 03/Feb/09 2:02 AM

This is going in 0.5.4, hook or by crook.

Show
Adam Jacob added a comment - 03/Feb/09 2:02 AM This is going in 0.5.4, hook or by crook.
Hide
Adam Jacob added a comment - 13/Feb/09 8:22 PM

Bumping these to 0.5.6

Show
Adam Jacob added a comment - 13/Feb/09 8:22 PM Bumping these to 0.5.6
Hide
AJ Christensen added a comment - 05/May/09 6:26 AM

Renaming.

Want to make it so that any resource can have a configurable number of retries (similar to ignore_failure true)

Show
AJ Christensen added a comment - 05/May/09 6:26 AM Renaming. Want to make it so that any resource can have a configurable number of retries (similar to ignore_failure true)
Hide
AJ Christensen added a comment - 24/Jun/09 11:12 AM

Have the code for this almost complete, going into 0.7.4

Show
AJ Christensen added a comment - 24/Jun/09 11:12 AM Have the code for this almost complete, going into 0.7.4
Hide
Adam Jacob added a comment - 20/Jul/10 5:45 AM

This is still a good idea!

Show
Adam Jacob added a comment - 20/Jul/10 5:45 AM This is still a good idea!
Hide
Michael Leinartas added a comment - 23/Sep/10 7:50 AM

Retry added to the base resource here:
http://github.com/mleinart/chef/tree/CHEF-39

Unit tests updated as well.

Show
Michael Leinartas added a comment - 23/Sep/10 7:50 AM Retry added to the base resource here: http://github.com/mleinart/chef/tree/CHEF-39 Unit tests updated as well.
Hide
Bryan McLellan added a comment - 24/Sep/10 7:03 PM

Can someone please add a retry timer to limit the immediate retries and double-check the test coverage? Thanks.

Show
Bryan McLellan added a comment - 24/Sep/10 7:03 PM Can someone please add a retry timer to limit the immediate retries and double-check the test coverage? Thanks.
Hide
Daniel DeLeo added a comment - 19/Oct/10 9:44 PM

bulk updating fix versions to next release

Show
Daniel DeLeo added a comment - 19/Oct/10 9:44 PM bulk updating fix versions to next release
Hide
Dan Peterson added a comment - 29/Mar/11 10:22 PM

So would it just take some backoff delay to get this merged?

Show
Dan Peterson added a comment - 29/Mar/11 10:22 PM So would it just take some backoff delay to get this merged?
Hide
Dan Peterson added a comment - 10/Apr/11 3:11 AM

In https://github.com/dpiddy/chef/tree/CHEF-39:

  • Rebased to latest master
  • Added retry_delay for sleeping between retries
  • Added a Runner spec to verify retries work if specified
Show
Dan Peterson added a comment - 10/Apr/11 3:11 AM In https://github.com/dpiddy/chef/tree/CHEF-39:
  • Rebased to latest master
  • Added retry_delay for sleeping between retries
  • Added a Runner spec to verify retries work if specified
Hide
Bryan McLellan added a comment - 20/May/11 1:03 AM

This patch has been merged to opscode/master and will be included in the next release. Thank you for contributing to Chef!

Show
Bryan McLellan added a comment - 20/May/11 1:03 AM This patch has been merged to opscode/master and will be included in the next release. Thank you for contributing to Chef!
Hide
Bryan McLellan added a comment - 07/Jul/11 6:00 PM

Reopening for fix version change.

Show
Bryan McLellan added a comment - 07/Jul/11 6:00 PM Reopening for fix version change.
Hide
Bryan McLellan added a comment - 07/Jul/11 6:03 PM

0.10.2 was a security release, 0.10.4 is the next maintenance release.

Show
Bryan McLellan added a comment - 07/Jul/11 6:03 PM 0.10.2 was a security release, 0.10.4 is the next maintenance release.
Hide
Bryan McLellan added a comment - 07/Jul/11 6:04 PM

Bulk edit: this ticket should have been previously merged for 0.10.2, but will be released in 0.10.4

Show
Bryan McLellan added a comment - 07/Jul/11 6:04 PM Bulk edit: this ticket should have been previously merged for 0.10.2, but will be released in 0.10.4

People

Vote (1)
Watch (1)

Dates

  • Created:
    19/Jan/09 7:27 PM
    Updated:
    07/Jul/11 6:04 PM
    Resolved:
    07/Jul/11 6:04 PM