chef 0.9.14.beta.1 → 0.9.14.rc.1

data/lib/chef/api_client.rb

@@ -24,7 +24,7 @@ require 'chef/couchdb'
 require 'chef/certificate'
 require 'chef/index_queue'
 require 'extlib'
-require 'chef/json'
+require 'chef/json_compat'
 
 class Chef
   class ApiClient 

data/lib/chef/application/client.rb

@@ -181,7 +181,7 @@ class Chef::Application::Client < Chef::Application
       end
 
       begin
-        @chef_client_json = Chef::JSON.from_json(json_io.read)
+        @chef_client_json = Chef::JSONCompat.from_json(json_io.read)
         json_io.close unless json_io.closed?
       rescue JSON::ParserError => error
         Chef::Application.fatal!("Could not parse the provided JSON file (#{Chef::Config[:json_attribs]})!: " + error.message, 2)

data/lib/chef/application/solo.rb

@@ -145,7 +145,7 @@ class Chef::Application::Solo < Chef::Application
       end
 
       begin
-        @chef_solo_json = Chef::JSON.from_json(json_io.read)
+        @chef_solo_json = Chef::JSONCompat.from_json(json_io.read)
         json_io.close unless json_io.closed?
       rescue JSON::ParserError => error
         Chef::Application.fatal!("Could not parse the provided JSON file (#{Chef::Config[:json_attribs]})!: " + error.message, 2)

data/lib/chef/checksum_cache.rb

@@ -151,7 +151,7 @@ class Chef
     end
 
     def checksum_file(file, digest)
-      File.open(file) { |f| checksum_io(f, digest) }
+      File.open(file, 'rb') { |f| checksum_io(f, digest) }
     end
 
     def checksum_io(io, digest)

data/lib/chef/cookbook/metadata.rb

@@ -430,12 +430,12 @@ class Chef
       end
 
       def self.from_json(string)
-        o = Chef::JSON.from_json(string)
+        o = Chef::JSONCompat.from_json(string)
         self.from_hash(o)
       end
 
       def from_json(string)
-        o = Chef::JSON.from_json(string)
+        o = Chef::JSONCompat.from_json(string)
         from_hash(o)
       end
 

data/lib/chef/couchdb.rb

@@ -21,7 +21,7 @@ require 'chef/config'
 require 'chef/rest'
 require 'chef/log'
 require 'digest/sha2'
-require 'chef/json'
+require 'chef/json_compat'
 
 # We want to fail on create if uuidtools isn't installed
 begin

data/lib/chef/data_bag.rb

@@ -25,7 +25,7 @@ require 'chef/couchdb'
 require 'chef/data_bag_item'
 require 'chef/index_queue'
 require 'extlib'
-require 'chef/json'
+require 'chef/json_compat'
 
 class Chef
   class DataBag 

data/lib/chef/data_bag_item.rb

@@ -27,7 +27,7 @@ require 'chef/couchdb'
 require 'chef/index_queue'
 require 'chef/data_bag'
 require 'extlib'
-require 'chef/json'
+require 'chef/json_compat'
 
 class Chef
   class DataBagItem

data/lib/chef/exceptions.rb

@@ -24,6 +24,7 @@ class Chef
     class Cron < RuntimeError; end
     class Env < RuntimeError; end
     class Exec < RuntimeError; end
+    class ErlCall < RuntimeError; end
     class FileNotFound < RuntimeError; end  
     class Package < RuntimeError; end
     class Service < RuntimeError; end

data/lib/chef/file_cache.rb

@@ -18,7 +18,7 @@
 require 'chef/mixin/params_validate'
 require 'chef/mixin/create_path'
 require 'chef/exceptions'
-require 'chef/json'
+require 'chef/json_compat'
 require 'fileutils'
 
 class Chef

data/lib/chef/handler/json_file.rb

@@ -41,7 +41,7 @@ class Chef
         build_report_dir
         savetime = Time.now.strftime("%Y%m%d%H%M%S")
         File.open(File.join(config[:path], "chef-run-report-#{savetime}.json"), "w") do |file|
-          file.puts Chef::JSON.to_json_pretty(data)
+          file.puts Chef::JSONCompat.to_json_pretty(data)
         end
       end
 

data/lib/chef/index_queue/amqp_client.rb

@@ -76,7 +76,7 @@ class Chef
       def send_action(action, data)
         retries = 0
         begin
-          exchange.publish(Chef::JSON.to_json({"action" => action.to_s, "payload" => data}))
+          exchange.publish(Chef::JSONCompat.to_json({"action" => action.to_s, "payload" => data}))
         rescue Bunny::ServerDownError, Bunny::ConnectionError, Errno::ECONNRESET
           disconnected!
           if (retries += 1) < 2

data/lib/chef/index_queue/consumer.rb

@@ -56,7 +56,7 @@ class Chef
       alias :start :run
       
       def call_action_for_message(message)
-        amqp_payload  = Chef::JSON.from_json(message[:payload], :create_additions => false, :max_nesting => false)
+        amqp_payload  = Chef::JSONCompat.from_json(message[:payload], :create_additions => false, :max_nesting => false)
         action        = amqp_payload["action"].to_sym
         app_payload   = amqp_payload["payload"]
         assert_method_whitelisted(action)

data/lib/chef/index_queue/indexable.rb

@@ -70,7 +70,6 @@ class Chef
         Chef::Log.debug("pushing item to index queue for deletion: #{self.with_indexer_metadata(metadata)}")
         AmqpClient.instance.send_action(:delete, self.with_indexer_metadata(metadata))
       end
-      
     end
   end
 end

data/lib/chef/{json.rb → json_compat.rb}

@@ -20,7 +20,7 @@
 require 'json'
 
 class Chef
-  class JSON
+  class JSONCompat
     JSON_MAX_NESTING = 1000
 
     class <<self

data/lib/chef/knife.rb

@@ -284,7 +284,7 @@ class Chef
     def output(data)
       case config[:format]
       when "json", nil
-        stdout.puts Chef::JSON.to_json_pretty(data)
+        stdout.puts Chef::JSONCompat.to_json_pretty(data)
       when "yaml"
         require 'yaml'
         stdout.puts YAML::dump(data)
@@ -330,7 +330,7 @@ class Chef
     end
 
     def edit_data(data, parse_output=true)
-      output = Chef::JSON.to_json_pretty(data)
+      output = Chef::JSONCompat.to_json_pretty(data)
       
       if (!config[:no_editor])
         filename = "knife-edit-"
@@ -348,7 +348,7 @@ class Chef
         File.unlink(filename)
       end
 
-      parse_output ? Chef::JSON.from_json(output) : output
+      parse_output ? Chef::JSONCompat.from_json(output) : output
     end
 
     def confirm(question, append_instructions=true)
@@ -399,7 +399,7 @@ class Chef
 
       case from_file
       when /\.(js|json)$/
-        Chef::JSON.from_json(IO.read(filename))
+        Chef::JSONCompat.from_json(IO.read(filename))
       when /\.rb$/
         r = klass.new
         r.from_file(filename)
@@ -429,8 +429,8 @@ class Chef
       # We wouldn't have to do these shenanigans if all the editable objects 
       # implemented to_hash, or if to_json against a hash returned a string 
       # with stable key order.
-      object_parsed_again = Chef::JSON.from_json(Chef::JSON.to_json(object), :create_additions => false)
-      output_parsed_again = Chef::JSON.from_json(Chef::JSON.to_json(output), :create_additions => false)
+      object_parsed_again = Chef::JSONCompat.from_json(Chef::JSONCompat.to_json(object), :create_additions => false)
+      output_parsed_again = Chef::JSONCompat.from_json(Chef::JSONCompat.to_json(output), :create_additions => false)
       if object_parsed_again != output_parsed_again
         output.save
         self.msg("Saved #{output}")

data/lib/chef/knife/bluebox_images_list.rb

@@ -17,7 +17,7 @@
 #
 
 require 'chef/knife'
-require 'chef/json'
+require 'chef/json_compat'
 
 class Chef
   class Knife

data/lib/chef/knife/bluebox_server_create.rb

@@ -17,7 +17,7 @@
 #
 
 require 'chef/knife'
-require 'chef/json'
+require 'chef/json_compat'
 
 class Chef
   class Knife

data/lib/chef/knife/bluebox_server_delete.rb

@@ -17,7 +17,7 @@
 #
 
 require 'chef/knife'
-require 'chef/json'
+require 'chef/json_compat'
 
 class Chef
   class Knife

data/lib/chef/knife/bluebox_server_list.rb

@@ -17,7 +17,7 @@
 #
 
 require 'chef/knife'
-require 'chef/json'
+require 'chef/json_compat'
 
 class Chef
   class Knife

data/lib/chef/knife/bootstrap.rb

@@ -17,7 +17,7 @@
 #
 
 require 'chef/knife'
-require 'chef/json'
+require 'chef/json_compat'
 require 'tempfile'
 require 'erubis'
 

data/lib/chef/knife/client_bulk_delete.rb

@@ -18,7 +18,7 @@
 
 require 'chef/knife'
 require 'chef/api_client'
-require 'chef/json'
+require 'chef/json_compat'
 
 class Chef
   class Knife

data/lib/chef/knife/client_create.rb

@@ -18,7 +18,7 @@
 
 require 'chef/knife'
 require 'chef/api_client'
-require 'chef/json'
+require 'chef/json_compat'
 
 class Chef
   class Knife

data/lib/chef/knife/client_delete.rb

@@ -18,7 +18,7 @@
 
 require 'chef/knife'
 require 'chef/api_client'
-require 'chef/json'
+require 'chef/json_compat'
 
 class Chef
   class Knife

data/lib/chef/knife/client_edit.rb

@@ -18,7 +18,7 @@
 
 require 'chef/knife'
 require 'chef/api_client'
-require 'chef/json'
+require 'chef/json_compat'
 
 class Chef
   class Knife

data/lib/chef/knife/client_list.rb

@@ -18,7 +18,7 @@
 
 require 'chef/knife'
 require 'chef/api_client'
-require 'chef/json'
+require 'chef/json_compat'
 
 class Chef
   class Knife

data/lib/chef/knife/client_reregister.rb

@@ -18,7 +18,7 @@
 
 require 'chef/knife'
 require 'chef/api_client'
-require 'chef/json'
+require 'chef/json_compat'
 
 class Chef
   class Knife

data/lib/chef/knife/client_show.rb

@@ -18,7 +18,7 @@
 
 require 'chef/knife'
 require 'chef/api_client'
-require 'chef/json'
+require 'chef/json_compat'
 
 class Chef
   class Knife

data/lib/chef/knife/cookbook_create.rb

@@ -17,7 +17,7 @@
 #
 
 require 'chef/knife'
-require 'chef/json'
+require 'chef/json_compat'
 require 'uri'
 require 'chef/mixin/shell_out'
 

data/lib/chef/knife/cookbook_list.rb

@@ -17,7 +17,7 @@
 #
 
 require 'chef/knife'
-require 'chef/json'
+require 'chef/json_compat'
 
 class Chef
   class Knife

data/lib/chef/knife/cookbook_metadata.rb

@@ -69,7 +69,7 @@ class Chef
           md.from_file(file)
           json_file = File.join(File.dirname(file), 'metadata.json')
           File.open(json_file, "w") do |f|
-            f.write(Chef::JSON.to_json_pretty(md))
+            f.write(Chef::JSONCompat.to_json_pretty(md))
           end
           generated = true
           Chef::Log.debug("Generated #{json_file}")

data/lib/chef/knife/cookbook_show.rb

@@ -17,7 +17,7 @@
 #
 
 require 'chef/knife'
-require 'chef/json'
+require 'chef/json_compat'
 require 'uri'
 
 class Chef

data/lib/chef/knife/cookbook_site_share.rb

@@ -85,7 +85,7 @@ class Chef
            :cookbook => category_string
          })
 
-         res = Chef::JSON.from_json(http_resp.body)
+         res = Chef::JSONCompat.from_json(http_resp.body)
          if http_resp.code.to_i != 201
            if res['error_messages']
              if res['error_messages'][0] =~ /Version already exists/

data/lib/chef/knife/cookbook_site_unshare.rb

@@ -18,6 +18,7 @@
 #
 
 require 'chef/knife'
+require 'chef/json_compat'
 
 class Chef
   class Knife

data/lib/chef/knife/data_bag_delete.rb

@@ -18,6 +18,7 @@
 
 require 'chef/knife'
 require 'chef/data_bag'
+require 'chef/json_compat'
 
 class Chef
   class Knife

data/lib/chef/knife/data_bag_from_file.rb

@@ -19,6 +19,7 @@
 require 'chef/knife'
 require 'chef/data_bag'
 require 'chef/data_bag_item'
+require 'chef/json_compat'
 
 class Chef
   class Knife

data/lib/chef/knife/ec2_instance_data.rb

@@ -17,7 +17,7 @@
 #
 
 require 'chef/knife'
-require 'chef/json'
+require 'chef/json_compat'
 
 class Chef
   class Knife

data/lib/chef/knife/ec2_server_create.rb

@@ -18,7 +18,7 @@
 
 require 'socket'
 require 'chef/knife'
-require 'chef/json'
+require 'chef/json_compat'
 
 class Chef
   class Knife
@@ -155,7 +155,7 @@ class Chef
           :groups => config[:security_groups],
           :flavor_id => config[:flavor],
           :key_name => Chef::Config[:knife][:aws_ssh_key_id],
-          :availability_zone => Chef::Config[:availability_zone]
+          :availability_zone => Chef::Config[:knife][:availability_zone]
         )
 
         puts "#{h.color("Instance ID", :cyan)}: #{server.id}"

data/lib/chef/knife/ec2_server_delete.rb

@@ -17,7 +17,7 @@
 #
 
 require 'chef/knife'
-require 'chef/json'
+require 'chef/json_compat'
 
 class Chef
   class Knife

data/lib/chef/knife/ec2_server_list.rb

@@ -17,7 +17,7 @@
 #
 
 require 'chef/knife'
-require 'chef/json'
+require 'chef/json_compat'
 
 class Chef
   class Knife

data/lib/chef/knife/node_bulk_delete.rb

@@ -18,7 +18,7 @@
 
 require 'chef/knife'
 require 'chef/node'
-require 'chef/json'
+require 'chef/json_compat'
 
 class Chef
   class Knife