diff options
author | jmink <jmink@getchef.com> | 2014-08-25 09:40:58 -0400 |
---|---|---|
committer | jmink <jmink@getchef.com> | 2014-08-25 15:32:48 -0400 |
commit | f190f8cba42b51774365dcc1bcf87836547c0a4d (patch) | |
tree | d86e55d009a4055339eb05eadbfbe4ec6d0760f3 /lib/chef | |
parent | 920d190cd817da50dcf87c3046b7c5f3bdcae7f9 (diff) | |
download | chef-f190f8cba42b51774365dcc1bcf87836547c0a4d.tar.gz |
Warn on using 'everything' for knife download on a hosted setup PR #1913
Diffstat (limited to 'lib/chef')
-rw-r--r-- | lib/chef/chef_fs/config.rb | 13 | ||||
-rw-r--r-- | lib/chef/chef_fs/knife.rb | 2 |
2 files changed, 12 insertions, 3 deletions
diff --git a/lib/chef/chef_fs/config.rb b/lib/chef/chef_fs/config.rb index e08b976961..536409a109 100644 --- a/lib/chef/chef_fs/config.rb +++ b/lib/chef/chef_fs/config.rb @@ -25,14 +25,19 @@ class Chef # Helpers to take Chef::Config and create chef_fs and local_fs from it # class Config - def initialize(chef_config = Chef::Config, cwd = Dir.pwd, options = {}) + def initialize(chef_config = Chef::Config, cwd = Dir.pwd, options = {}, ui = nil) @chef_config = chef_config @cwd = cwd @cookbook_version = options[:cookbook_version] + if @chef_config[:repo_mode] == 'everything' && is_hosted? && !ui.nil? + ui.warn %Q{You have repo_mode set to 'everything', but your chef_server_url + looks like it might be a hosted setup. If this is the case please use + hosted_everything or allow repo_mode to default} + end # Default to getting *everything* from the server. if !@chef_config[:repo_mode] - if @chef_config[:chef_server_url] =~ /\/+organizations\/.+/ + if is_hosted? @chef_config[:repo_mode] = 'hosted_everything' else @chef_config[:repo_mode] = 'everything' @@ -44,6 +49,10 @@ class Chef attr_reader :cwd attr_reader :cookbook_version + def is_hosted? + @chef_config[:chef_server_url] =~ /\/+organizations\/.+/ + end + def chef_fs @chef_fs ||= create_chef_fs end diff --git a/lib/chef/chef_fs/knife.rb b/lib/chef/chef_fs/knife.rb index 652c728550..86872dab71 100644 --- a/lib/chef/chef_fs/knife.rb +++ b/lib/chef/chef_fs/knife.rb @@ -68,7 +68,7 @@ class Chef end end - @chef_fs_config = Chef::ChefFS::Config.new(Chef::Config, Dir.pwd, config) + @chef_fs_config = Chef::ChefFS::Config.new(Chef::Config, Dir.pwd, config, ui) Chef::ChefFS::Parallelizer.threads = (Chef::Config[:concurrency] || 10) - 1 end |