graphite_carbon_conf_accumulator: Do not install whisper with :delete…

… action
sous-chefs · Jun 18, 2015 · fab355b · fab355b
1 parent 3dacdea
commit fab355b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/definitions/graphite_carbon_conf_accumulator.rb b/definitions/graphite_carbon_conf_accumulator.rb
@@ -16,6 +16,7 @@
     rescue Chef::Exceptions::ResourceNotFound
       python_pip backend_name do
         Chef::Log.info "Installing storage backend: #{package_name}"
+        only_if { params[:action] == :create }
         action :install
       end
     end

diff --git a/spec/definitions/carbon_conf_accumulator_spec.rb b/spec/definitions/carbon_conf_accumulator_spec.rb
@@ -65,6 +65,10 @@
     it 'delete the file template' do
       expect(recipe_run).to delete_template('carbon.conf')
     end
+
+    it 'does not install whisper' do
+      expect(recipe_run).to_not install_python_pip('whisper')
+    end
   end
 
   context 'with configuation options' do