summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTim Smith <tsmith@chef.io>2017-07-06 22:41:51 -0700
committerTim Smith <tsmith@chef.io>2018-03-12 11:14:27 -0700
commitc9b5e176033cce4d6bbdaec75b356fe6cdf569d2 (patch)
tree6a8439ce3f58e065ffbab31f78e7833b8dfeeb35
parent02976a0b8a4a33d07fbb71f91514a123c3fe9e0b (diff)
downloadchef-c9b5e176033cce4d6bbdaec75b356fe6cdf569d2.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.rb26
1 files changed, 3 insertions, 23 deletions
diff --git a/lib/chef/knife/bootstrap.rb b/lib/chef/knife/bootstrap.rb
index 88656359f9..03a639e4ea 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",
@@ -312,10 +293,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