chef-dk 0.9.0 → 0.10.0

data/lib/kitchen/provisioner/policyfile_zero.rb

@@ -1,193 +1,189 @@
-# -*- encoding: utf-8 -*-
-#
-# Author:: Fletcher Nichol (<fnichol@nichol.ca>)
-#
-# Copyright (C) 2013, Fletcher Nichol
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-#    http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-
-require "kitchen/provisioner/chef_base"
-
-# TODO: chef-dk and kitchen can only co-exist if kitchen and chef-dk agree on
-# the version of mixlib-shellout to use. Kitchen currently locked at 1.4,
-# chef-dk is on 2.x
-require 'chef-dk/policyfile_services/export_repo'
-
-module Kitchen
-
-  module Provisioner
-
-    class Base
-
-      # PolicyfileZero needs to access the base behavior of creating the
-      # sandbox directory without invoking the behavior of
-      # ChefBase#create_sandbox, because that will trigger the use of
-      # Chef::CommonSandbox, which we need to override.
-      alias_method :create_sandbox_directory, :create_sandbox
-
-    end
-
-    class PolicyfileSandbox < Chef::CommonSandbox
-
-      # Stub #prepare_cookbooks because we have implemented this in the
-      # provisioner, below. If a Berksfile is present, the default
-      # implementation will try to run Berkshelf, which can lead to dependency
-      # issues since berks is not yet using Solve 2.x. See also
-      # PolicyfileZero#load_needed_dependencies! which is stubbed to prevent
-      # berks from loading.
-      def prepare_cookbooks
-      end
-
-    end
-
-    # Policyfile + Chef Zero provisioner.
-    #
-    # @author Daniel DeLeo <dan@chef.io>
-    class PolicyfileZero < ChefBase
-
-      # This provsioner will forcibly set the following config options:
-      # * `use_policyfile`: `true`
-      # * `versioned_cookbooks`: `true`
-      # * `deployment_group`: `POLICY_NAME-local`
-      # Since it makes no sense to modify these, they are hardcoded elsewhere.
-      default_config :client_rb, {}
-      default_config :json_attributes, true
-      default_config :chef_zero_host, nil
-      default_config :chef_zero_port, 8889
-      default_config :policyfile, "Policyfile.rb"
-
-      default_config :chef_client_path do |provisioner|
-        provisioner.
-          remote_path_join(%W[#{provisioner[:chef_omnibus_root]} bin chef-client]).
-          tap { |path| path.concat(".bat") if provisioner.windows_os? }
-      end
-
-      default_config :ruby_bindir do |provisioner|
-        provisioner.
-          remote_path_join(%W[#{provisioner[:chef_omnibus_root]} embedded bin])
-      end
-
-      # Emit a warning that Policyfile stuff is still experimental.
-      #
-      # (see Base#finalize_config!)
-      def finalize_config!(*args)
-        super
-        banner("Using experimental policyfile mode for chef-client")
-        warn("The Policyfile feature is under active development.")
-        warn("For best results, always use the latest chef-client version")
-      end
-
-      # (see Base#create_sandbox)
-      def create_sandbox
-        create_sandbox_directory
-        PolicyfileSandbox.new(config, sandbox_path, instance).populate
-        prepare_cookbooks
-        prepare_validation_pem
-        prepare_client_rb
-      end
-
-      # (see Base#run_command)
-      def run_command
-        level = config[:log_level] == :info ? :auto : config[:log_level]
-        chef_client_bin = sudo(config[:chef_client_path])
-
-        cmd = "#{chef_client_bin} --local-mode"
-        args = [
-          "--config #{config[:root_path]}/client.rb",
-          "--log_level #{level}",
-          "--force-formatter",
-          "--no-color"
-        ]
-        if config[:chef_zero_port]
-          args <<  "--chef-zero-port #{config[:chef_zero_port]}"
-        end
-        if config[:log_file]
-          args << "--logfile #{config[:log_file]}"
-        end
-
-        wrap_shell_code(
-          [cmd, *args].join(" ").
-          tap { |str| str.insert(0, reload_ps1_path) if windows_os? }
-        )
-      end
-
-      # We don't want to load Berkshelf or Librarian; Policyfile is managing
-      # dependencies, so these can only cause trouble.
-      def load_needed_dependencies!
-      end
-
-      private
-
-      # Overrides behavior of parent class so that dna.json isn't created; we
-      # don't need it.
-      #
-      # @api private
-      def prepare_json
-      end
-
-      # Copies the policyfile's cookbooks to the sandbox.
-      #
-      # @api private
-      def prepare_cookbooks
-        Kitchen.mutex.synchronize do
-          policy_exporter.run
-        end
-      end
-
-      # An instance of ChefDK::PolicyfileServices::ExportRepo, configured with
-      # the sandbox path. Calling `#run` on this copies the cookbooks to the
-      # sandbox. Calling `#policy_name` returns the policy's name.
-      #
-      # @api private
-      def policy_exporter
-        # Must force this because TK by default copies the current cookbook to the sandbox
-        # See ChefDK::PolicyfileServices::ExportRepo#assert_export_dir_clean!
-        @policy_exporter ||= ChefDK::PolicyfileServices::ExportRepo.new(policyfile: config[:policyfile],
-                                                                        export_dir: sandbox_path,
-                                                                        force: true)
-      end
-
-      # Writes a fake (but valid) validation.pem into the sandbox directory.
-      #
-      # @api private
-      def prepare_validation_pem
-        info("Preparing validation.pem")
-        debug("Using a dummy validation.pem")
-
-        source = File.join(Kitchen.source_root, %w[support dummy-validation.pem])
-        FileUtils.cp(source, File.join(sandbox_path, "validation.pem"))
-      end
-
-      # Writes a client.rb configuration file to the sandbox directory.
-      #
-      # @api private
-      def prepare_client_rb
-        data = default_config_rb.merge(config[:client_rb])
-
-        data["use_policyfile"] = true
-        data["versioned_cookbooks"] = true
-        data["deployment_group"] = "#{policy_exporter.policy_name}-local"
-        # TODO this will need to be updated when chef-zero supports erchef paths (policy_group vs policies)
-        data["policy_document_native_api"] = false
-
-        info("Preparing client.rb")
-        debug("Creating client.rb from #{data.inspect}")
-
-        File.open(File.join(sandbox_path, "client.rb"), "wb") do |file|
-          file.write(format_config_file(data))
-        end
-      end
-
-    end
-  end
-end
+# -*- encoding: utf-8 -*-
+#
+# Author:: Fletcher Nichol (<fnichol@nichol.ca>)
+#
+# Copyright (C) 2013, Fletcher Nichol
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#    http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+require "kitchen/provisioner/chef_base"
+
+# TODO: chef-dk and kitchen can only co-exist if kitchen and chef-dk agree on
+# the version of mixlib-shellout to use. Kitchen currently locked at 1.4,
+# chef-dk is on 2.x
+require 'chef-dk/policyfile_services/export_repo'
+
+module Kitchen
+
+  module Provisioner
+
+    class Base
+
+      # PolicyfileZero needs to access the base behavior of creating the
+      # sandbox directory without invoking the behavior of
+      # ChefBase#create_sandbox, because that will trigger the use of
+      # Chef::CommonSandbox, which we need to override.
+      alias_method :create_sandbox_directory, :create_sandbox
+
+    end
+
+    class PolicyfileSandbox < Chef::CommonSandbox
+
+      # Stub #prepare_cookbooks because we have implemented this in the
+      # provisioner, below. If a Berksfile is present, the default
+      # implementation will try to run Berkshelf, which can lead to dependency
+      # issues since berks is not yet using Solve 2.x. See also
+      # PolicyfileZero#load_needed_dependencies! which is stubbed to prevent
+      # berks from loading.
+      def prepare_cookbooks
+      end
+
+    end
+
+    # Policyfile + Chef Zero provisioner.
+    #
+    # @author Daniel DeLeo <dan@chef.io>
+    class PolicyfileZero < ChefBase
+
+      # This provsioner will forcibly set the following config options:
+      # * `use_policyfile`: `true`
+      # * `versioned_cookbooks`: `true`
+      # * `deployment_group`: `POLICY_NAME-local`
+      # Since it makes no sense to modify these, they are hardcoded elsewhere.
+      default_config :client_rb, {}
+      default_config :json_attributes, true
+      default_config :chef_zero_host, nil
+      default_config :chef_zero_port, 8889
+      default_config :policyfile, "Policyfile.rb"
+
+      default_config :chef_client_path do |provisioner|
+        provisioner.
+          remote_path_join(%W[#{provisioner[:chef_omnibus_root]} bin chef-client]).
+          tap { |path| path.concat(".bat") if provisioner.windows_os? }
+      end
+
+      default_config :ruby_bindir do |provisioner|
+        provisioner.
+          remote_path_join(%W[#{provisioner[:chef_omnibus_root]} embedded bin])
+      end
+
+      # (see Base#finalize_config!)
+      def finalize_config!(*args)
+        super
+        banner("Using policyfile mode for chef-client")
+      end
+
+      # (see Base#create_sandbox)
+      def create_sandbox
+        create_sandbox_directory
+        PolicyfileSandbox.new(config, sandbox_path, instance).populate
+        prepare_cookbooks
+        prepare_validation_pem
+        prepare_client_rb
+      end
+
+      # (see Base#run_command)
+      def run_command
+        level = config[:log_level] == :info ? :auto : config[:log_level]
+        chef_client_bin = sudo(config[:chef_client_path])
+
+        cmd = "#{chef_client_bin} --local-mode"
+        args = [
+          "--config #{config[:root_path]}/client.rb",
+          "--log_level #{level}",
+          "--force-formatter",
+          "--no-color"
+        ]
+        if config[:chef_zero_port]
+          args <<  "--chef-zero-port #{config[:chef_zero_port]}"
+        end
+        if config[:log_file]
+          args << "--logfile #{config[:log_file]}"
+        end
+
+        wrap_shell_code(
+          [cmd, *args].join(" ").
+          tap { |str| str.insert(0, reload_ps1_path) if windows_os? }
+        )
+      end
+
+      # We don't want to load Berkshelf or Librarian; Policyfile is managing
+      # dependencies, so these can only cause trouble.
+      def load_needed_dependencies!
+      end
+
+      private
+
+      # Overrides behavior of parent class so that dna.json isn't created; we
+      # don't need it.
+      #
+      # @api private
+      def prepare_json
+      end
+
+      # Copies the policyfile's cookbooks to the sandbox.
+      #
+      # @api private
+      def prepare_cookbooks
+        Kitchen.mutex.synchronize do
+          policy_exporter.run
+        end
+      end
+
+      # An instance of ChefDK::PolicyfileServices::ExportRepo, configured with
+      # the sandbox path. Calling `#run` on this copies the cookbooks to the
+      # sandbox. Calling `#policy_name` returns the policy's name.
+      #
+      # @api private
+      def policy_exporter
+        # Must force this because TK by default copies the current cookbook to the sandbox
+        # See ChefDK::PolicyfileServices::ExportRepo#assert_export_dir_clean!
+        @policy_exporter ||= ChefDK::PolicyfileServices::ExportRepo.new(policyfile: config[:policyfile],
+                                                                        export_dir: sandbox_path,
+                                                                        force: true)
+      end
+
+      # Writes a fake (but valid) validation.pem into the sandbox directory.
+      #
+      # @api private
+      def prepare_validation_pem
+        info("Preparing validation.pem")
+        debug("Using a dummy validation.pem")
+
+        source = File.join(Kitchen.source_root, %w[support dummy-validation.pem])
+        FileUtils.cp(source, File.join(sandbox_path, "validation.pem"))
+      end
+
+      # Writes a client.rb configuration file to the sandbox directory.
+      #
+      # @api private
+      def prepare_client_rb
+        data = default_config_rb.merge(config[:client_rb])
+
+        data["use_policyfile"] = true
+        data["versioned_cookbooks"] = true
+        data["deployment_group"] = "#{policy_exporter.policy_name}-local"
+        # TODO this will need to be updated when chef-zero supports erchef paths (policy_group vs policies)
+        data["policy_document_native_api"] = false
+
+        info("Preparing client.rb")
+        debug("Creating client.rb from #{data.inspect}")
+
+        File.open(File.join(sandbox_path, "client.rb"), "wb") do |file|
+          file.write(format_config_file(data))
+        end
+      end
+
+    end
+  end
+end

data/spec/shared/a_file_generator.rb

@@ -1,125 +1,125 @@
-shared_examples_for "a file generator" do
-
-  let(:stdout_io) { StringIO.new }
-  let(:stderr_io) { StringIO.new }
-
-  def stdout
-    stdout_io.string
-  end
-
-  def stderr
-    stderr_io.string
-  end
-
-  let(:expected_cookbook_root) { tempdir }
-  let(:cookbook_name) { "example_cookbook" }
-
-  let(:cookbook_path) { File.join(tempdir, cookbook_name) }
-
-  subject(:recipe_generator) do
-    generator = described_class.new(argv)
-    allow(generator).to receive(:stdout).and_return(stdout_io)
-    allow(generator).to receive(:stderr).and_return(stderr_io)
-    generator
-  end
-
-  def generator_context
-    ChefDK::Generator.context
-  end
-
-  before do
-    ChefDK::Generator.reset
-    reset_tempdir
-  end
-
-  after(:each) do
-    ChefDK::Generator::Context.reset
-  end
-
-  context "when argv is empty" do
-    let(:argv) { [] }
-
-    it "emits an error message and exits" do
-      expected_stdout = "Usage: chef generate #{generator_name} [path/to/cookbook] NAME [options]"
-
-      expect(recipe_generator.run).to eq(1)
-      expect(stdout).to include(expected_stdout)
-    end
-  end
-
-  context "when CWD is a cookbook" do
-
-    let(:argv) { [ new_file_name ] }
-
-    before do
-      FileUtils.cp_r(File.join(fixtures_path, "example_cookbook"), tempdir)
-    end
-
-    it "configures the generator context" do
-      Dir.chdir(cookbook_path) do
-        recipe_generator.read_and_validate_params
-        recipe_generator.setup_context
-
-        expect(generator_context.cookbook_root).to eq(expected_cookbook_root)
-        expect(generator_context.cookbook_name).to eq(cookbook_name)
-        expect(generator_context.new_file_basename).to eq(new_file_name)
-        expect(generator_context.recipe_name).to eq(new_file_name)
-      end
-    end
-
-    it "creates a new recipe" do
-      Dir.chdir(cookbook_path) do
-        allow(recipe_generator.chef_runner).to receive(:stdout).and_return(stdout_io)
-        recipe_generator.run
-      end
-
-        generated_files.each do |expected_file|
-          expect(File).to exist(File.join(cookbook_path, expected_file))
-        end
-    end
-
-  end
-
-  context "when CWD is not a cookbook" do
-    context "and path to the cookbook is not given in the agv" do
-      let(:argv) { [ new_file_name ] }
-
-      it "emits an error message and exits" do
-        expected_stdout = "Usage: chef generate #{generator_name} [path/to/cookbook] NAME [options]"
-        expected_stderr = "Error: Directory #{Dir.pwd} is not a cookbook\n"
-
-        expect(recipe_generator.run).to eq(1)
-        expect(stdout).to include(expected_stdout)
-        expect(stderr).to eq(expected_stderr)
-      end
-    end
-
-    context "and path to the cookbook is given in the argv" do
-      let(:argv) { [cookbook_path, new_file_name ] }
-
-      before do
-        FileUtils.cp_r(File.join(fixtures_path, "example_cookbook"), tempdir)
-      end
-
-      it "configures the generator context" do
-        recipe_generator.read_and_validate_params
-        recipe_generator.setup_context
-
-        expect(generator_context.cookbook_root).to eq(File.dirname(cookbook_path))
-        expect(generator_context.cookbook_name).to eq(cookbook_name)
-        expect(generator_context.new_file_basename).to eq(new_file_name)
-      end
-
-      it "creates a new recipe" do
-        allow(recipe_generator.chef_runner).to receive(:stdout).and_return(stdout_io)
-        recipe_generator.run
-
-        generated_files.each do |expected_file|
-          expect(File).to exist(File.join(cookbook_path, expected_file))
-        end
-      end
-
-    end
-  end
-
-end
+shared_examples_for "a file generator" do
+
+  let(:stdout_io) { StringIO.new }
+  let(:stderr_io) { StringIO.new }
+
+  def stdout
+    stdout_io.string
+  end
+
+  def stderr
+    stderr_io.string
+  end
+
+  let(:expected_cookbook_root) { tempdir }
+  let(:cookbook_name) { "example_cookbook" }
+
+  let(:cookbook_path) { File.join(tempdir, cookbook_name) }
+
+  subject(:recipe_generator) do
+    generator = described_class.new(argv)
+    allow(generator).to receive(:stdout).and_return(stdout_io)
+    allow(generator).to receive(:stderr).and_return(stderr_io)
+    generator
+  end
+
+  def generator_context
+    ChefDK::Generator.context
+  end
+
+  before do
+    ChefDK::Generator.reset
+    reset_tempdir
+  end
+
+  after(:each) do
+    ChefDK::Generator::Context.reset
+  end
+
+  context "when argv is empty" do
+    let(:argv) { [] }
+
+    it "emits an error message and exits" do
+      expected_stdout = "Usage: chef generate #{generator_name} [path/to/cookbook] NAME [options]"
+
+      expect(recipe_generator.run).to eq(1)
+      expect(stdout).to include(expected_stdout)
+    end
+  end
+
+  context "when CWD is a cookbook" do
+
+    let(:argv) { [ new_file_name ] }
+
+    before do
+      FileUtils.cp_r(File.join(fixtures_path, "example_cookbook"), tempdir)
+    end
+
+    it "configures the generator context" do
+      Dir.chdir(cookbook_path) do
+        recipe_generator.read_and_validate_params
+        recipe_generator.setup_context
+
+        expect(generator_context.cookbook_root).to eq(expected_cookbook_root)
+        expect(generator_context.cookbook_name).to eq(cookbook_name)
+        expect(generator_context.new_file_basename).to eq(new_file_name)
+        expect(generator_context.recipe_name).to eq(new_file_name)
+      end
+    end
+
+    it "creates a new recipe" do
+      Dir.chdir(cookbook_path) do
+        allow(recipe_generator.chef_runner).to receive(:stdout).and_return(stdout_io)
+        recipe_generator.run
+      end
+
+        generated_files.each do |expected_file|
+          expect(File).to exist(File.join(cookbook_path, expected_file))
+        end
+    end
+
+  end
+
+  context "when CWD is not a cookbook" do
+    context "and path to the cookbook is not given in the agv" do
+      let(:argv) { [ new_file_name ] }
+
+      it "emits an error message and exits" do
+        expected_stdout = "Usage: chef generate #{generator_name} [path/to/cookbook] NAME [options]"
+        expected_stderr = "Error: Directory #{Dir.pwd} is not a cookbook\n"
+
+        expect(recipe_generator.run).to eq(1)
+        expect(stdout).to include(expected_stdout)
+        expect(stderr).to eq(expected_stderr)
+      end
+    end
+
+    context "and path to the cookbook is given in the argv" do
+      let(:argv) { [cookbook_path, new_file_name ] }
+
+      before do
+        FileUtils.cp_r(File.join(fixtures_path, "example_cookbook"), tempdir)
+      end
+
+      it "configures the generator context" do
+        recipe_generator.read_and_validate_params
+        recipe_generator.setup_context
+
+        expect(generator_context.cookbook_root).to eq(File.dirname(cookbook_path))
+        expect(generator_context.cookbook_name).to eq(cookbook_name)
+        expect(generator_context.new_file_basename).to eq(new_file_name)
+      end
+
+      it "creates a new recipe" do
+        allow(recipe_generator.chef_runner).to receive(:stdout).and_return(stdout_io)
+        recipe_generator.run
+
+        generated_files.each do |expected_file|
+          expect(File).to exist(File.join(cookbook_path, expected_file))
+        end
+      end
+
+    end
+  end
+
+end