Update mongodb-shard.conf.erb

[andreish]

fix the bind_ip usage in mongdb-shard.conf.erb, it is defined as array in manifests but config file wants it as a string ( comma separated list of ip adresses )

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

[ekohl]

Good catch. There are tests for server:

puppet-mongodb/spec/classes/server_spec.rb

Lines 119 to 142 in 6f48edd

	describe 'with specific bind_ip values and ipv6' do
	let :params do
	{
	bind_ip: ['127.0.0.1', 'fd00:beef:dead:55::143'],
	ipv6: true
	}
	end
	it do
	is_expected.to contain_file(config_file).
	with_content(%r{^net\.bindIp: 127\.0\.0\.1\,fd00:beef:dead:55::143$}).
	with_content(%r{^net\.ipv6: true$})
	end
	end
	describe 'with specific bind_ip values' do
	let :params do
	{
	bind_ip: ['127.0.0.1', '10.1.1.13']
	}
	end
	it { is_expected.to contain_file(config_file).with_content(%r{^net\.bindIp: 127\.0\.0\.1\,10\.1\.1\.13$}) }
	end

However, these don't exist for mongos. Could you add these so we don't risk this in the future?

[ekohl]

Would you mind moving this below the context 'with defaults' block? I think it's good to always first list tests with defaults, then variations on that.

[ekohl]

Thanks!