chef-dk 2.4.17 → 2.5.3

data/lib/chef-dk/policyfile_services/push_archive.rb

@@ -1,200 +1,200 @@
-#
-# Copyright:: Copyright (c) 2015 Chef Software Inc.
-# License:: Apache License, Version 2.0
-#
-# 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 "zlib"
-require "archive/tar/minitar"
-
-require "chef-dk/service_exceptions"
-require "chef-dk/policyfile_lock"
-require "chef/server_api"
-require "chef-dk/policyfile/uploader"
-
-module ChefDK
-  module PolicyfileServices
-    class PushArchive
-
-      USTAR_INDICATOR = "ustar\0".force_encoding(Encoding::ASCII_8BIT).freeze
-
-      attr_reader :archive_file
-      attr_reader :policy_group
-      attr_reader :root_dir
-      attr_reader :ui
-      attr_reader :config
-
-      attr_reader :policyfile_lock
-
-      def initialize(archive_file: nil, policy_group: nil, root_dir: nil, ui: nil, config: nil)
-        @archive_file = archive_file
-        @policy_group = policy_group
-        @root_dir = root_dir || Dir.pwd
-        @ui = ui
-        @config = config
-
-        @policyfile_lock = nil
-      end
-
-      def archive_file_path
-        File.expand_path(archive_file, root_dir)
-      end
-
-      def run
-        unless File.exist?(archive_file_path)
-          raise InvalidPolicyArchive, "Archive file #{archive_file_path} not found"
-        end
-        stage_unpacked_archive do |staging_dir|
-          read_policyfile_lock(staging_dir)
-
-          uploader.upload
-        end
-
-      rescue => e
-        raise PolicyfilePushArchiveError.new("Failed to publish archived policy", e)
-      end
-
-      # @api private
-      def uploader
-        ChefDK::Policyfile::Uploader.new(policyfile_lock, policy_group,
-                                         ui: ui,
-                                         http_client: http_client,
-                                         policy_document_native_api: config.policy_document_native_api)
-      end
-
-      # @api private
-      def http_client
-        @http_client ||= Chef::ServerAPI.new(config.chef_server_url,
-                                             signing_key_filename: config.client_key,
-                                             client_name: config.node_name)
-      end
-
-      private
-
-      def read_policyfile_lock(staging_dir)
-        policyfile_lock_path = File.join(staging_dir, "Policyfile.lock.json")
-
-        if looks_like_old_format_archive?(staging_dir)
-          raise InvalidPolicyArchive, <<-MESSAGE
-This archive is in an unsupported format.
-
-This archive was created with an older version of ChefDK. This version of
-ChefDK does not support archives in the older format. Re-create the archive
-with a newer version of ChefDK or downgrade ChefDK.
-MESSAGE
-        end
-
-        unless File.exist?(policyfile_lock_path)
-          raise InvalidPolicyArchive, "Archive does not contain a Policyfile.lock.json"
-        end
-
-        unless File.directory?(File.join(staging_dir, "cookbook_artifacts"))
-          raise InvalidPolicyArchive, "Archive does not contain a cookbook_artifacts directory"
-        end
-
-        policy_data = load_policy_data(policyfile_lock_path)
-        storage_config = Policyfile::StorageConfig.new.use_policyfile_lock(policyfile_lock_path)
-        @policyfile_lock = ChefDK::PolicyfileLock.new(storage_config).build_from_archive(policy_data)
-
-        missing_cookbooks = policyfile_lock.cookbook_locks.select do |name, lock|
-          !lock.installed?
-        end
-
-        unless missing_cookbooks.empty?
-          message = "Archive does not have all cookbooks required by the Policyfile.lock. " +
-            "Missing cookbooks: '#{missing_cookbooks.keys.join('", "')}'."
-          raise InvalidPolicyArchive, message
-        end
-      end
-
-      def load_policy_data(policyfile_lock_path)
-        FFI_Yajl::Parser.parse(IO.read(policyfile_lock_path))
-      end
-
-      def stage_unpacked_archive
-        p = Process.pid
-        t = Time.new.utc.strftime("%Y%m%d%H%M%S")
-        Dir.mktmpdir("chefdk-push-archive-#{p}-#{t}") do |staging_dir|
-          unpack_to(staging_dir)
-          yield staging_dir
-        end
-      end
-
-      def unpack_to(staging_dir)
-        Zlib::GzipReader.open(archive_file_path) do |gz_file|
-          untar_to(gz_file, staging_dir)
-        end
-
-        # untar_to can raise InvalidPolicyArchive, let it through
-      rescue InvalidPolicyArchive
-        raise
-      rescue => e
-        raise InvalidPolicyArchive, "Archive file #{archive_file_path} could not be unpacked. #{e}"
-      end
-
-      def untar_to(tar_file, staging_dir)
-        # Minitar doesn't do much input checking, so if you feed it a
-        # garbage-enough file it will just do weird things and blow up. For
-        # example, if tar_file is just a bunch of nul characters, then tar will
-        # try to open a file named '.'; if you give it some random string that
-        # fits in the size of the filename header, it will create that file.
-        #
-        # Tar archives that we create via `chef export -a` and probably
-        # everything else we might encounter should be in ustar format. For
-        # such a tar file, bytes 257-263 should be "ustar\0", so we use this as
-        # a sanity check.
-        # https://en.wikipedia.org/wiki/Tar_(computing)
-
-        first_tar_header = tar_file.read(512)
-        ustar_indicator = first_tar_header[257, 6]
-
-        unless ustar_indicator == USTAR_INDICATOR
-          raise InvalidPolicyArchive, "Archive file #{archive_file_path} could not be unpacked. Tar archive looks corrupt."
-        end
-
-        # "undo" read of the first 512 bytes
-        tar_file.rewind
-
-        Archive::Tar::Minitar::Input.open(tar_file) do |stream|
-          stream.each do |entry|
-            stream.extract_entry(staging_dir, entry)
-          end
-        end
-      end
-
-      def looks_like_old_format_archive?(staging_dir)
-        cookbooks_dir = File.join(staging_dir, "cookbooks")
-        data_bags_dir = File.join(staging_dir, "data_bags")
-
-        cookbook_artifacts_dir = File.join(staging_dir, "cookbook_artifacts")
-        policies_dir = File.join(staging_dir, "policies")
-        policy_groups_dir = File.join(staging_dir, "policy_groups")
-
-        # Old archives just had these two dirs
-        have_old_dirs = File.exist?(cookbooks_dir) && File.exist?(data_bags_dir)
-
-        # New archives created by `chef export` will have all of these; it's
-        # also possible we'll encounter an "artisanal" archive, which might
-        # only be missing one of these by accident. In that case we want to
-        # trigger a different error than we're detecting here.
-        have_any_new_dirs = File.exist?(cookbook_artifacts_dir) ||
-          File.exist?(policies_dir) ||
-          File.exist?(policy_groups_dir)
-
-        have_old_dirs && !have_any_new_dirs
-      end
-
-    end
-  end
-end
+#
+# Copyright:: Copyright (c) 2015 Chef Software Inc.
+# License:: Apache License, Version 2.0
+#
+# 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 "zlib"
+require "archive/tar/minitar"
+
+require "chef-dk/service_exceptions"
+require "chef-dk/policyfile_lock"
+require "chef/server_api"
+require "chef-dk/policyfile/uploader"
+
+module ChefDK
+  module PolicyfileServices
+    class PushArchive
+
+      USTAR_INDICATOR = "ustar\0".force_encoding(Encoding::ASCII_8BIT).freeze
+
+      attr_reader :archive_file
+      attr_reader :policy_group
+      attr_reader :root_dir
+      attr_reader :ui
+      attr_reader :config
+
+      attr_reader :policyfile_lock
+
+      def initialize(archive_file: nil, policy_group: nil, root_dir: nil, ui: nil, config: nil)
+        @archive_file = archive_file
+        @policy_group = policy_group
+        @root_dir = root_dir || Dir.pwd
+        @ui = ui
+        @config = config
+
+        @policyfile_lock = nil
+      end
+
+      def archive_file_path
+        File.expand_path(archive_file, root_dir)
+      end
+
+      def run
+        unless File.exist?(archive_file_path)
+          raise InvalidPolicyArchive, "Archive file #{archive_file_path} not found"
+        end
+        stage_unpacked_archive do |staging_dir|
+          read_policyfile_lock(staging_dir)
+
+          uploader.upload
+        end
+
+      rescue => e
+        raise PolicyfilePushArchiveError.new("Failed to publish archived policy", e)
+      end
+
+      # @api private
+      def uploader
+        ChefDK::Policyfile::Uploader.new(policyfile_lock, policy_group,
+                                         ui: ui,
+                                         http_client: http_client,
+                                         policy_document_native_api: config.policy_document_native_api)
+      end
+
+      # @api private
+      def http_client
+        @http_client ||= Chef::ServerAPI.new(config.chef_server_url,
+                                             signing_key_filename: config.client_key,
+                                             client_name: config.node_name)
+      end
+
+      private
+
+      def read_policyfile_lock(staging_dir)
+        policyfile_lock_path = File.join(staging_dir, "Policyfile.lock.json")
+
+        if looks_like_old_format_archive?(staging_dir)
+          raise InvalidPolicyArchive, <<-MESSAGE
+This archive is in an unsupported format.
+
+This archive was created with an older version of ChefDK. This version of
+ChefDK does not support archives in the older format. Re-create the archive
+with a newer version of ChefDK or downgrade ChefDK.
+MESSAGE
+        end
+
+        unless File.exist?(policyfile_lock_path)
+          raise InvalidPolicyArchive, "Archive does not contain a Policyfile.lock.json"
+        end
+
+        unless File.directory?(File.join(staging_dir, "cookbook_artifacts"))
+          raise InvalidPolicyArchive, "Archive does not contain a cookbook_artifacts directory"
+        end
+
+        policy_data = load_policy_data(policyfile_lock_path)
+        storage_config = Policyfile::StorageConfig.new.use_policyfile_lock(policyfile_lock_path)
+        @policyfile_lock = ChefDK::PolicyfileLock.new(storage_config).build_from_archive(policy_data)
+
+        missing_cookbooks = policyfile_lock.cookbook_locks.select do |name, lock|
+          !lock.installed?
+        end
+
+        unless missing_cookbooks.empty?
+          message = "Archive does not have all cookbooks required by the Policyfile.lock. " +
+            "Missing cookbooks: '#{missing_cookbooks.keys.join('", "')}'."
+          raise InvalidPolicyArchive, message
+        end
+      end
+
+      def load_policy_data(policyfile_lock_path)
+        FFI_Yajl::Parser.parse(IO.read(policyfile_lock_path))
+      end
+
+      def stage_unpacked_archive
+        p = Process.pid
+        t = Time.new.utc.strftime("%Y%m%d%H%M%S")
+        Dir.mktmpdir("chefdk-push-archive-#{p}-#{t}") do |staging_dir|
+          unpack_to(staging_dir)
+          yield staging_dir
+        end
+      end
+
+      def unpack_to(staging_dir)
+        Zlib::GzipReader.open(archive_file_path) do |gz_file|
+          untar_to(gz_file, staging_dir)
+        end
+
+        # untar_to can raise InvalidPolicyArchive, let it through
+      rescue InvalidPolicyArchive
+        raise
+      rescue => e
+        raise InvalidPolicyArchive, "Archive file #{archive_file_path} could not be unpacked. #{e}"
+      end
+
+      def untar_to(tar_file, staging_dir)
+        # Minitar doesn't do much input checking, so if you feed it a
+        # garbage-enough file it will just do weird things and blow up. For
+        # example, if tar_file is just a bunch of nul characters, then tar will
+        # try to open a file named '.'; if you give it some random string that
+        # fits in the size of the filename header, it will create that file.
+        #
+        # Tar archives that we create via `chef export -a` and probably
+        # everything else we might encounter should be in ustar format. For
+        # such a tar file, bytes 257-263 should be "ustar\0", so we use this as
+        # a sanity check.
+        # https://en.wikipedia.org/wiki/Tar_(computing)
+
+        first_tar_header = tar_file.read(512)
+        ustar_indicator = first_tar_header[257, 6]
+
+        unless ustar_indicator == USTAR_INDICATOR
+          raise InvalidPolicyArchive, "Archive file #{archive_file_path} could not be unpacked. Tar archive looks corrupt."
+        end
+
+        # "undo" read of the first 512 bytes
+        tar_file.rewind
+
+        Archive::Tar::Minitar::Input.open(tar_file) do |stream|
+          stream.each do |entry|
+            stream.extract_entry(staging_dir, entry)
+          end
+        end
+      end
+
+      def looks_like_old_format_archive?(staging_dir)
+        cookbooks_dir = File.join(staging_dir, "cookbooks")
+        data_bags_dir = File.join(staging_dir, "data_bags")
+
+        cookbook_artifacts_dir = File.join(staging_dir, "cookbook_artifacts")
+        policies_dir = File.join(staging_dir, "policies")
+        policy_groups_dir = File.join(staging_dir, "policy_groups")
+
+        # Old archives just had these two dirs
+        have_old_dirs = File.exist?(cookbooks_dir) && File.exist?(data_bags_dir)
+
+        # New archives created by `chef export` will have all of these; it's
+        # also possible we'll encounter an "artisanal" archive, which might
+        # only be missing one of these by accident. In that case we want to
+        # trigger a different error than we're detecting here.
+        have_any_new_dirs = File.exist?(cookbook_artifacts_dir) ||
+          File.exist?(policies_dir) ||
+          File.exist?(policy_groups_dir)
+
+        have_old_dirs && !have_any_new_dirs
+      end
+
+    end
+  end
+end

data/lib/chef-dk/policyfile_services/rm_policy.rb

@@ -1,141 +1,141 @@
-#
-# Copyright:: Copyright (c) 2015 Chef Software Inc.
-# License:: Apache License, Version 2.0
-#
-# 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 "chef-dk/service_exceptions"
-require "chef/server_api"
-require "chef-dk/policyfile/undo_stack"
-require "chef-dk/policyfile/undo_record"
-
-module ChefDK
-  module PolicyfileServices
-
-    class RmPolicy
-
-      attr_reader :policy_name
-
-      # @api private
-      attr_reader :chef_config
-
-      # @api private
-      attr_reader :ui
-
-      # @api private
-      attr_reader :undo_record
-
-      # @api private
-      attr_reader :undo_stack
-
-      def initialize(config: nil, ui: nil, policy_name: nil)
-        @chef_config = config
-        @ui = ui
-        @policy_name = policy_name
-
-        @policy_revision_data = nil
-        @policy_exists = false
-        @policy_group_data = nil
-
-        @undo_record = Policyfile::UndoRecord.new
-        @undo_stack = Policyfile::UndoStack.new
-      end
-
-      def run
-        unless policy_exists?
-          ui.err("Policy '#{policy_name}' does not exist on the server")
-          return false
-        end
-
-        undo_record.description = "delete-policy #{policy_name}"
-
-        unless policy_has_no_revisions?
-          gather_policy_data_for_undo
-        end
-
-        http_client.delete("/policies/#{policy_name}")
-        undo_stack.push(undo_record)
-        ui.err("Removed policy '#{policy_name}'.")
-      rescue => e
-        raise DeletePolicyError.new("Failed to delete policy '#{policy_name}'", e)
-      end
-
-      # @api private
-      # An instance of Chef::ServerAPI configured with the user's
-      # server URL and credentials.
-      def http_client
-        @http_client ||= Chef::ServerAPI.new(chef_config.chef_server_url,
-                                             signing_key_filename: chef_config.client_key,
-                                             client_name: chef_config.node_name)
-      end
-
-      private
-
-      def policy_has_no_revisions?
-        policy_revision_data.empty? || policy_revision_data["revisions"].empty?
-      end
-
-      def gather_policy_data_for_undo
-        revisions = policy_revision_data["revisions"].keys
-
-        revisions.each do |revision_id|
-          policy_revision_data = http_client.get("/policies/#{policy_name}/revisions/#{revision_id}")
-          policy_groups = policy_groups_using_revision(revision_id)
-          if policy_groups.empty?
-            undo_record.add_policy_revision(policy_name, nil, policy_revision_data)
-          else
-            policy_groups.each do |policy_group|
-              undo_record.add_policy_revision(policy_name, policy_group, policy_revision_data)
-            end
-          end
-        end
-      end
-
-      def policy_groups_using_revision(revision_id)
-        groups = []
-        policy_group_data.each do |group_name, group_info|
-          next unless group_info.key?("policies") && !group_info["policies"].empty?
-          next unless group_info["policies"].key?(policy_name)
-          next unless group_info["policies"][policy_name]["revision_id"] == revision_id
-          groups << group_name if group_info
-        end
-        groups
-      end
-
-      def policy_group_data
-        @policy_group_data ||= http_client.get("/policy_groups")
-      end
-
-      def policy_exists?
-        return true if @policy_exists
-        fetch_policy_revision_data
-        @policy_exists
-      end
-
-      def policy_revision_data
-        return @policy_revision_data if @policy_exists
-        fetch_policy_revision_data
-      end
-
-      def fetch_policy_revision_data
-        @policy_revision_data = http_client.get("/policies/#{policy_name}")
-        @policy_exists = true
-      rescue Net::HTTPServerException => e
-        raise unless e.response.code == "404"
-        @policy_exists = false
-      end
-
-    end
-  end
-end
+#
+# Copyright:: Copyright (c) 2015 Chef Software Inc.
+# License:: Apache License, Version 2.0
+#
+# 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 "chef-dk/service_exceptions"
+require "chef/server_api"
+require "chef-dk/policyfile/undo_stack"
+require "chef-dk/policyfile/undo_record"
+
+module ChefDK
+  module PolicyfileServices
+
+    class RmPolicy
+
+      attr_reader :policy_name
+
+      # @api private
+      attr_reader :chef_config
+
+      # @api private
+      attr_reader :ui
+
+      # @api private
+      attr_reader :undo_record
+
+      # @api private
+      attr_reader :undo_stack
+
+      def initialize(config: nil, ui: nil, policy_name: nil)
+        @chef_config = config
+        @ui = ui
+        @policy_name = policy_name
+
+        @policy_revision_data = nil
+        @policy_exists = false
+        @policy_group_data = nil
+
+        @undo_record = Policyfile::UndoRecord.new
+        @undo_stack = Policyfile::UndoStack.new
+      end
+
+      def run
+        unless policy_exists?
+          ui.err("Policy '#{policy_name}' does not exist on the server")
+          return false
+        end
+
+        undo_record.description = "delete-policy #{policy_name}"
+
+        unless policy_has_no_revisions?
+          gather_policy_data_for_undo
+        end
+
+        http_client.delete("/policies/#{policy_name}")
+        undo_stack.push(undo_record)
+        ui.err("Removed policy '#{policy_name}'.")
+      rescue => e
+        raise DeletePolicyError.new("Failed to delete policy '#{policy_name}'", e)
+      end
+
+      # @api private
+      # An instance of Chef::ServerAPI configured with the user's
+      # server URL and credentials.
+      def http_client
+        @http_client ||= Chef::ServerAPI.new(chef_config.chef_server_url,
+                                             signing_key_filename: chef_config.client_key,
+                                             client_name: chef_config.node_name)
+      end
+
+      private
+
+      def policy_has_no_revisions?
+        policy_revision_data.empty? || policy_revision_data["revisions"].empty?
+      end
+
+      def gather_policy_data_for_undo
+        revisions = policy_revision_data["revisions"].keys
+
+        revisions.each do |revision_id|
+          policy_revision_data = http_client.get("/policies/#{policy_name}/revisions/#{revision_id}")
+          policy_groups = policy_groups_using_revision(revision_id)
+          if policy_groups.empty?
+            undo_record.add_policy_revision(policy_name, nil, policy_revision_data)
+          else
+            policy_groups.each do |policy_group|
+              undo_record.add_policy_revision(policy_name, policy_group, policy_revision_data)
+            end
+          end
+        end
+      end
+
+      def policy_groups_using_revision(revision_id)
+        groups = []
+        policy_group_data.each do |group_name, group_info|
+          next unless group_info.key?("policies") && !group_info["policies"].empty?
+          next unless group_info["policies"].key?(policy_name)
+          next unless group_info["policies"][policy_name]["revision_id"] == revision_id
+          groups << group_name if group_info
+        end
+        groups
+      end
+
+      def policy_group_data
+        @policy_group_data ||= http_client.get("/policy_groups")
+      end
+
+      def policy_exists?
+        return true if @policy_exists
+        fetch_policy_revision_data
+        @policy_exists
+      end
+
+      def policy_revision_data
+        return @policy_revision_data if @policy_exists
+        fetch_policy_revision_data
+      end
+
+      def fetch_policy_revision_data
+        @policy_revision_data = http_client.get("/policies/#{policy_name}")
+        @policy_exists = true
+      rescue Net::HTTPServerException => e
+        raise unless e.response.code == "404"
+        @policy_exists = false
+      end
+
+    end
+  end
+end