Config dir permissions are getting double-set

[ewr]

There are two places in the cookbook that are creating the config dir, and they're setting different permissions...

In the service recipe:

chef-consul-template/recipes/service.rb

Lines 44 to 51 in adda5d8

	# Create service directories
	consul_template_directories.each do |dirname|
	directory dirname do
	owner consul_template_user
	group consul_template_group
	mode 0755
	end
	end

In the config provider:

chef-consul-template/providers/config.rb

Lines 25 to 32 in adda5d8

	# Ensure config directory exists
	directory node['consul_template']['config_dir'] do
	user consul_template_user
	group consul_template_group
	mode node['consul_template']['template_mode']
	recursive true
	action :create
	end

I'm guessing that the 0755 that the service recipe sets is probably correct for the directory, and that the config provider should be using that (and just using node['consul_template']['template_mode'] for the template config file). As it is now, the config resource is always setting its changed flag, which is causing our wrapper template to restart consul-template unnecessarily.

[adamkrone]

Thanks for pointing this out. It should now be fixed on master.

[ewr]

Thanks!