Skip to content

Commit

Permalink
Fixes #26, remove the ownership and permissions
Browse files Browse the repository at this point in the history
Setting the owner and permissions on the config file's directory
conflicts with product ctl commands that also manage those
directories. For example some have the ownership set to opscode
instead of root, meaning this is updated every time and causes a
reconfigure to run again, extending the length of time it takes to
converge a node.
  • Loading branch information
jtimberman committed Aug 11, 2015
1 parent ade1fb4 commit 145e4d8
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions libraries/ingredient_config_provider.rb
Original file line number Diff line number Diff line change
Expand Up @@ -32,16 +32,11 @@ def whyrun_supported?
return if target_config.nil?

directory ::File.dirname(target_config) do
owner 'root'
group 'root'
mode '0755'
recursive true
action :create
end

file target_config do
owner 'root'
group 'root'
action :create
sensitive new_resource.sensitive
content get_config(new_resource.product_name)
Expand Down

0 comments on commit 145e4d8

Please sign in to comment.