diff options
author | Tim Smith <tsmith@chef.io> | 2017-07-06 22:41:51 -0700 |
---|---|---|
committer | Tim Smith <tsmith@chef.io> | 2017-09-11 10:11:58 -0700 |
commit | 81bbd05f13cb8b6f9d5c01bcf993ac1e5cd5eaff (patch) | |
tree | ecebca61f7b3d4cf98169635c35c823ed003d750 | |
parent | 6fbe116aa8ed4ec82be2234172089c1c51487947 (diff) | |
download | chef-81bbd05f13cb8b6f9d5c01bcf993ac1e5cd5eaff.tar.gz |
Remove deprecated bootstrap options
We said we would remove these in Chef 13 in the code. Lets do it for 14 instead.
Signed-off-by: Tim Smith <tsmith@chef.io>
-rw-r--r-- | lib/chef/knife/bootstrap.rb | 26 |
1 files changed, 3 insertions, 23 deletions
diff --git a/lib/chef/knife/bootstrap.rb b/lib/chef/knife/bootstrap.rb index f9dca7896a..b2f2d63f16 100644 --- a/lib/chef/knife/bootstrap.rb +++ b/lib/chef/knife/bootstrap.rb @@ -119,16 +119,6 @@ class Chef :description => "Do not proxy locations for the node being bootstrapped; this option is used internally by Opscode", :proc => Proc.new { |np| Chef::Config[:knife][:bootstrap_no_proxy] = np } - # DEPR: Remove this option in Chef 13 - option :distro, - :short => "-d DISTRO", - :long => "--distro DISTRO", - :description => "Bootstrap a distro using a template. [DEPRECATED] Use -t / --bootstrap-template option instead.", - :proc => Proc.new { |v| - Chef::Log.warn("[DEPRECATED] -d / --distro option is deprecated. Use -t / --bootstrap-template option instead.") - v - } - option :bootstrap_template, :short => "-t TEMPLATE", :long => "--bootstrap-template TEMPLATE", @@ -149,15 +139,6 @@ class Chef :description => "Execute the bootstrap via sudo with password", :boolean => false - # DEPR: Remove this option in Chef 13 - option :template_file, - :long => "--template-file TEMPLATE", - :description => "Full path to location of template to use. [DEPRECATED] Use -t / --bootstrap-template option instead.", - :proc => Proc.new { |v| - Chef::Log.warn("[DEPRECATED] --template-file option is deprecated. Use -t / --bootstrap-template option instead.") - v - } - option :run_list, :short => "-r RUN_LIST", :long => "--run-list RUN_LIST", @@ -307,10 +288,9 @@ class Chef end def bootstrap_template - # The order here is important. We want to check if we have the new Chef 12 option is set first. - # Knife cloud plugins unfortunately all set a default option for the :distro so it should be at - # the end. - config[:bootstrap_template] || config[:template_file] || config[:distro] || default_bootstrap_template + # + # @return [String] The CLI specific bootstrap template or the default + config[:bootstrap_template] || default_bootstrap_template end def find_template |