Refactor #9837
Refactor `puppet resource` application
| Status: | Closed | Start date: | 09/30/2011 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: | - | % Done: | 0% | |
| Category: | - | |||
| Target version: | 2.7.6 | |||
| Affected Puppet version: | Branch: | |||
| Keywords: | ||||
Description
The puppet resource application should be refactored so that aspects of its logic can be used elsewhere.
History
#1
Updated by Daniel Pittman over 1 year ago
You should do this by building a newer version of the resource face, supporting the high level facilities that you want, and then porting the older application to use that. That will give us a win on many concurrent fronts. :)
#2
Updated by Michael Stahnke over 1 year ago
I see a lot of commits against this ticket, but don’t know if it’s actually completed.
Author: Max Martin max@puppetlabs.com Date: Mon Oct 3 11:06:28 2011 -0700
Merge branch 'maint/2.7.x/resource_app_cleanup' into 2.7.x
* maint/2.7.x/resource_app_cleanup:
(#9838) Return the tranaction report when doing a ral save
(#9837) Split parameter pruning from manifest formatting
(#9837) Move resource formatting method to Puppet::Resource
(#9837) Move properties in prep to move proc to method
(#9837) Make a clearer variable name in the specs
(#9837) Call puppet apply to avoid deprecation warning
(#9837) Extract methods from the main section of the resource application
(#9837) Start the cleanup of the puppet resource application
Conflicts (resolved manually):
lib/puppet/application/resource.rb
If it’s complete, please set Target version and close.
If not, please update the ticket.
#3
Updated by Max Martin over 1 year ago
- Status changed from Accepted to Merged - Pending Release
As of commit:24d6c518e167138a64a0d3e789a002c6761a7912 this work has been merged into 2.7.x
#4
Updated by Jonathan Grochowski over 1 year ago
- Status changed from Merged - Pending Release to Closed
- Target version changed from 2.7.x to 2.7.6
Released in 2.7.6
#5
Updated by Daniel Pittman about 1 year ago
- Assignee deleted (
Matt Robinson)