knife-inspect 0.13.1 → 0.14.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ead2b16a86eea1a04e3d299fe86c2a9b4c499ff6
-  data.tar.gz: 65647d0b7045f1ebdabc54e7073736a035cb409f
+  metadata.gz: 08fcbef7a39b4537d36bedc38ac1fe729de18ff7
+  data.tar.gz: 922b94ce6d628fd143f94bb10a1d45493575e742
 SHA512:
-  metadata.gz: e2c6bb1c4a48e9a3098414e62298158a443f3382d65ef38f1a6a9ca093f13487a94ab8bebe269b317b3dccd78014bca13acc5bc6f17b33b88d32e9fbec430021
-  data.tar.gz: 4ee255bb13de52a02762ffcc964664fbb17d8595ba2f6e2a48dce63bbdfdfb3da25b041c45802bd345eea8154d6f6c31a0a2f6cfc1a5439d8b78db7392d13763
+  metadata.gz: 5ff1db43b22fa66ce529d32e73be5de237e32f20f14dba8e4530aa204d19c4d07f8a213ab718161dffe52800490978df78c3c1bc71e9e7dbee2d42758ef1314a
+  data.tar.gz: 5902568d29aef45798d51fb1c8b2db44f958c616e3fb6a1a0249e9a2f547abad7e451cecc779364b68d4ac5bba81f3eda91fd2ea5871d83c72db755b8f7a930d

data/HISTORY.md

@@ -1,3 +1,13 @@
+## 0.14.0 ( 2016-04-22 )
+
+* New feature: Allow to deactivate inspection by component ([#34][#34]). Thanks
+@kamaradclimber
+
+```
+--no-cookbooks --no-data-bags --no-data-bag-items --no-environments --no-roles
+```
+
+
 ## 0.13.1 ( 2016-02-16 )
 
 * Don't rely on deprecated auto-inflation of JSON data. Switch to `#from_hash`
@@ -189,3 +199,4 @@ instead.
 [#23]: https://github.com/bmarini/knife-inspect/issues/23
 [#35]: https://github.com/bmarini/knife-inspect/issues/35
 [#42]: https://github.com/bmarini/knife-inspect/issues/42
+[#34]: https://github.com/bmarini/knife-inspect/pull/34

data/README.md

@@ -18,7 +18,7 @@ or individual components.
 
     knife inspect
 
-    knife cookbook inspect
+    knife cookbook inspect [--no-cookbooks --no-data-bags --no-data-bag-items --no-environments --no-roles]
     knife cookbook inspect [COOKBOOK]
 
     knife data bag inspect
@@ -45,6 +45,15 @@ So far it checks if...
 * your environments are in sync
 * your roles are in sync
 
+You can use the option switches to disable checking for a resource type when
+running `knife inspect`:
+
+* `--no-cookbooks`
+* `--no-data-bags`
+* `--no-data-bag-items`
+* `--no-environments`
+* `--no-roles`
+
 You can use it with your favorite Continuous Integration tool, it returns 0
 when everything is in sync or 1 if it's not.
 

data/knife-inspect.gemspec

@@ -30,5 +30,6 @@ Gem::Specification.new do |s|
 
   s.add_runtime_dependency 'chef',      chef_version
   s.add_runtime_dependency 'yajl-ruby', '~> 1.2'
-  s.add_runtime_dependency 'parallel', '~> 1.3'
+  s.add_runtime_dependency 'parallel',  '~> 1.3'
+  s.add_runtime_dependency 'inflecto',  '~> 0.0.2'
 end

data/lib/chef/knife/inspect.rb

@@ -13,13 +13,33 @@ class Chef
 
       banner 'knife inspect'
 
+      CHECKLISTS.each do |checklist|
+        checklist = HealthInspector::Checklists.const_get(checklist)
+
+        option checklist.option,
+          :long => "--[no-]#{checklist.option}",
+          :boolean => true,
+          :default => true,
+          :description => "Add or exclude #{checklist.title} from inspection"
+      end
+
       def run
-        results = CHECKLISTS.map do |checklist|
+        results = checklists_to_run.map do |checklist|
           HealthInspector::Checklists.const_get(checklist).run(self)
         end
 
         exit !results.include?(false)
       end
+
+      private
+
+      def checklists_to_run
+        CHECKLISTS.select do |checklist|
+          checklist = HealthInspector::Checklists.const_get(checklist)
+
+          config[checklist.option]
+        end
+      end
     end
   end
 end

data/lib/health_inspector/checklists/base.rb

@@ -2,6 +2,7 @@
 require 'pathname'
 require 'yajl'
 require 'parallel'
+require 'inflecto'
 
 module HealthInspector
   module Checklists
@@ -9,10 +10,8 @@ module HealthInspector
       include Color
 
       class << self
-        attr_reader :title
-
-        def title(val = nil)
-          val.nil? ? @title : @title = val
+        def title
+          Inflecto.humanize(underscored_class).downcase
         end
       end
 
@@ -20,6 +19,14 @@ module HealthInspector
         new(knife).run
       end
 
+      def self.option
+        Inflecto.dasherize(underscored_class).to_sym
+      end
+
+      def self.underscored_class
+        Inflecto.underscore(Inflecto.demodulize(self.to_s))
+      end
+
       def initialize(knife)
         @context = Context.new(knife)
       end

data/lib/health_inspector/checklists/cookbooks.rb

@@ -89,8 +89,6 @@ module HealthInspector
     end
 
     class Cookbooks < Base
-      title 'cookbooks'
-
       def load_item(name)
         Cookbook.new(@context,
                      name: name,

data/lib/health_inspector/checklists/data_bag_items.rb

@@ -9,8 +9,6 @@ module HealthInspector
     end
 
     class DataBagItems < Base
-      title 'data bag items'
-
       def load_item(name)
         DataBagItem.new(@context,
                         name: name,

data/lib/health_inspector/checklists/data_bags.rb

@@ -7,8 +7,6 @@ module HealthInspector
     end
 
     class DataBags < Base
-      title 'data bags'
-
       def load_item(name)
         DataBag.new(@context,
                     name: name,

data/lib/health_inspector/checklists/environments.rb

@@ -15,8 +15,6 @@ module HealthInspector
     end
 
     class Environments < Base
-      title 'environments'
-
       def load_item(name)
         Environment.new(@context,
                         name: name,

data/lib/health_inspector/checklists/roles.rb

@@ -8,8 +8,6 @@ module HealthInspector
     end
 
     class Roles < Base
-      title 'roles'
-
       def load_item(name)
         Role.new(@context,
                  name: name,

data/lib/health_inspector/version.rb

@@ -1,3 +1,3 @@
 module HealthInspector
-  VERSION = '0.13.1'
+  VERSION = '0.14.0'
 end

data/spec/chef/knife/inspect_spec.rb

@@ -15,6 +15,15 @@ RSpec.describe Chef::Knife::Inspect do
       knife_inspect.run
     end
 
+    before do
+      # FIXME: Default config does not appear to be used by Chef 11 & 12
+      described_class::CHECKLISTS.each do |checklist|
+        checklist = HealthInspector::Checklists.const_get(checklist)
+
+        knife_inspect.config[checklist.option] = true
+      end
+    end
+
     context 'when all the checklists pass' do
       it 'runs all the check lists and exits with 0' do
         { HealthInspector::Checklists::Cookbooks => true,

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: knife-inspect
 version: !ruby/object:Gem::Version
-  version: 0.13.1
+  version: 0.14.0
 platform: ruby
 authors:
 - Greg Karékinian
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-16 00:00:00.000000000 Z
+date: 2016-04-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -109,6 +109,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.3'
+- !ruby/object:Gem::Dependency
+  name: inflecto
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.0.2
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.0.2
 description: knife-inspect is a knife plugin to compare the content of your Chef repository
   and Chef server
 email: