knife bootstrap - search for bootstrap templates in external knife plugins

Details

Description

This will allow knife plugin authors to include custom bootstrap templates when they ship the plugins as Ruby gems.

Issue Links

is duplicated by

Bug - A problem which impairs or prevents the functions of the product. CHEF-2597 Bootstrap Template Lookup

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.

Bug - A problem which impairs or prevents the functions of the product. CHEF-2808 "can't convert Array into String" in dirname

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.

Bug - A problem which impairs or prevents the functions of the product. CHEF-2736 knife bootstrap should check for a bootstrap template before trying to use it.

  • Unknown - Issue untriaged, priority unassigned or questioned
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Seth Chisamore added a comment -

Fix can be pulled from here:
https://github.com/schisamo/chef/tree/CHEF-2193

Show
Seth Chisamore added a comment - Fix can be pulled from here: https://github.com/schisamo/chef/tree/CHEF-2193
Seth Chisamore made changes -
Field Original Value New Value
Status Open [ 1 ] Resolved [ 5 ]
Assignee Seth Chisamore [ schisamo ] Daniel DeLeo [ dan@kallistec.com ]
Fix Version/s 0.10.0 [ 10081 ]
Resolution Fixed [ 1 ]
Bryan McLellan [Opscode] made changes -
Fix Version/s 0.10.2 [ 10203 ]
Fix Version/s 0.10.0 [ 10081 ]
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

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 [Opscode] added a comment - This patch has been merged to opscode/master and will be included in the next release. Thank you for contributing to Chef!
Bryan McLellan [Opscode] made changes -
Status Resolved [ 5 ] Closed [ 6 ]
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

Reopening for fix version change.

Show
Bryan McLellan [Opscode] added a comment - Reopening for fix version change.
Bryan McLellan [Opscode] made changes -
Resolution Fixed [ 1 ]
Status Closed [ 6 ] Reopened [ 4 ]
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

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

Show
Bryan McLellan [Opscode] added a comment - 0.10.2 was a security release, 0.10.4 is the next maintenance release.
Bryan McLellan [Opscode] made changes -
Fix Version/s 0.10.4 [ 10400 ]
Fix Version/s 0.10.2 [ 10203 ]
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

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

Show
Bryan McLellan [Opscode] added a comment - Bulk edit: this ticket should have been previously merged for 0.10.2, but will be released in 0.10.4
Bryan McLellan [Opscode] made changes -
Status Reopened [ 4 ] Closed [ 6 ]
Resolution Fixed [ 1 ]
Hide
Permalink
Gabe McArthur added a comment -

The original fix doesn't always seem to work (I don't understand how it ever could). The Gem#find_files, returns an array, which, when evaluated in the block in File#exists?, causes the a whole method to blow up. See here:

https://github.com/gabemc/chef/commit/509d44d6a3c732a2eaa40740c4041ae20de8e7f8

Show
Gabe McArthur added a comment - The original fix doesn't always seem to work (I don't understand how it ever could). The Gem#find_files, returns an array, which, when evaluated in the block in File#exists?, causes the a whole method to blow up. See here: https://github.com/gabemc/chef/commit/509d44d6a3c732a2eaa40740c4041ae20de8e7f8
Hide
Permalink
Seth Chisamore added a comment -

Actually the fix is much easier than that...just need to flatten the boostrap_files array:

bootstrap_files.flatten!

This fix already exists in knife-windows:
https://github.com/opscode/knife-windows/blob/master/lib/chef/knife/bootstrap_windows_base.rb#L98

Show
Seth Chisamore added a comment - Actually the fix is much easier than that...just need to flatten the boostrap_files array: bootstrap_files.flatten! This fix already exists in knife-windows: https://github.com/opscode/knife-windows/blob/master/lib/chef/knife/bootstrap_windows_base.rb#L98
Hide
Permalink
Seth Chisamore added a comment -

reopening per comments above.

Show
Seth Chisamore added a comment - reopening per comments above.
Seth Chisamore made changes -
Resolution Fixed [ 1 ]
Status Closed [ 6 ] Reopened [ 4 ]
Assignee Daniel DeLeo [ dan@kallistec.com ] Seth Chisamore [ schisamo ]
Hide
Permalink
Seth Chisamore added a comment -

Updated fix can be pulled from:
https://github.com/schisamo/chef/tree/CHEF-2193

I even wrote a spec this time to catch this in the future!

Show
Seth Chisamore added a comment - Updated fix can be pulled from: https://github.com/schisamo/chef/tree/CHEF-2193 I even wrote a spec this time to catch this in the future!
Seth Chisamore made changes -
Status Reopened [ 4 ] Resolved [ 5 ]
Fix Version/s 0.10.4 [ 10400 ]
Resolution Fixed [ 1 ]
Bryan McLellan [Opscode] made changes -
Triage Status [Triaged]
Hide
Permalink
Seth Chisamore added a comment -

this has been merged:
https://github.com/opscode/chef/commit/d9819c62a5c1d54bd186b8fef43f7bac00c1a416

Show
Seth Chisamore added a comment - this has been merged: https://github.com/opscode/chef/commit/d9819c62a5c1d54bd186b8fef43f7bac00c1a416
Seth Chisamore made changes -
Status Resolved [ 5 ] Closed [ 6 ]
Fix Version/s 0.10.6 [ 10403 ]
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

Note that the final fix is in 0.10.6 and this was broken in 0.10.4

Show
Bryan McLellan [Opscode] added a comment - Note that the final fix is in 0.10.6 and this was broken in 0.10.4
Bryan McLellan [Opscode] made changes -
Link This issue is duplicated by CHEF-2808 [ CHEF-2808 ]
Bryan McLellan [Opscode] made changes -
Link This issue is duplicated by CHEF-2736 [ CHEF-2736 ]
Bryan McLellan [Opscode] made changes -
Link This issue is duplicated by CHEF-2597 [ CHEF-2597 ]
Bryan McLellan [Opscode] made changes -
Workflow jira [ 13220 ] New OSS [ 19984 ]
Bryan McLellan [Opscode] made changes -
Workflow New OSS [ 19984 ] OSS: Product Management [ 27896 ]
Transition Time In Source Status Execution Times Last Executer Last Execution Date
Open Open Resolved Resolved
2m 42s 1 Seth Chisamore 06/Apr/11 10:25 PM
Reopened Reopened Closed Closed
3m 53s 1 Bryan McLellan [Opscode] 07/Jul/11 6:04 PM
Closed Closed Reopened Reopened
106d 42m 2 Seth Chisamore 03/Sep/11 1:39 AM
Reopened Reopened Resolved Resolved
30m 57s 1 Seth Chisamore 03/Sep/11 2:10 AM
Resolved Resolved Closed Closed
67d 20h 36m 2 Seth Chisamore 27/Sep/11 8:19 PM

People

  • Assignee:
    Seth Chisamore
    Reporter:
    Seth Chisamore
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: