depoy resource: number of releases kept should be configurable

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 0.10.0
  • Fix Version/s: 10.14.0
  • Component/s: Chef Client
  • Labels:
    None

Description

Currently the default number of releases kept by the deploy resources is 5. you should be able to override this value using a resource attribute. In Capistrano this config value is called keep_releases.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Ben Lindsey added a comment -

I would like to take a crack at fixing this as it would be a nice feature for our integration server to keep more than 5 releases.

Show
Ben Lindsey added a comment - I would like to take a crack at fixing this as it would be a nice feature for our integration server to keep more than 5 releases.
Hide
Permalink
Ben Lindsey added a comment -

https://github.com/opscode/chef/pull/226

Show
Ben Lindsey added a comment - https://github.com/opscode/chef/pull/226
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

Thank you for taking the time to help make Chef better! We require a Contributor License Agreement (CLA) from contributors to help protect all Chef users. It is easy to complete one online. Instructions to do so are on our wiki, which is linked below.

http://wiki.opscode.com/display/chef/How+to+Contribute

Show
Bryan McLellan [Opscode] added a comment - Thank you for taking the time to help make Chef better! We require a Contributor License Agreement (CLA) from contributors to help protect all Chef users. It is easy to complete one online. Instructions to do so are on our wiki, which is linked below. http://wiki.opscode.com/display/chef/How+to+Contribute
Hide
Permalink
Ben Lindsey added a comment -

Done.

Show
Ben Lindsey added a comment - Done.
Hide
Permalink
Daniel DeLeo added a comment -

Marking resolved so we see it in patch review.

Show
Daniel DeLeo added a comment - Marking resolved so we see it in patch review.
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

Reviewed and marked for merge

Show
Bryan McLellan [Opscode] added a comment - Reviewed and marked for merge
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

Merged to master, thanks for the fix Ben.

Show
Bryan McLellan [Opscode] added a comment - Merged to master, thanks for the fix Ben.

People

  • Assignee:
    Unassigned
    Reporter:
    Seth Chisamore
Vote (1)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: