Set GOMAXPROCS > 1

.ruby-gemset

@@ -0,0 +1 @@
+consul-cookbook

.ruby-version

@@ -0,0 +1 @@
+2.1.5

attributes/default.rb

@@ -43,6 +43,14 @@
 }
 default['consul']['source_revision'] = 'master'
 
+# GOMAXPROCS should be greater than 1, see
+# https://groups.google.com/forum/#!topic/consul-tool/qewFEqgAoF8
+if node['cpu']['total'] > 1
+  default['consul']['gomaxprocs'] = node['cpu']['total']
+else
+  default['consul']['gomaxprocs'] = 2
+end
+
 # Service attributes
 default['consul']['service_mode'] = 'bootstrap'
 default['consul']['retry_on_join'] = false

spec/unit/attributes/default_spec.rb

@@ -5,5 +5,16 @@
 
   context 'sets default attributes' do
     it { expect(chef_run.node.consul.base_url).to eq("https://dl.bintray.com/mitchellh/consul/%{version}.zip") }
+    it { expect(chef_run.node.consul.gomaxprocs).to eq(2) }
+  end
+
+  describe 'allow override of gomaxprocs' do
+    let(:chef_run) do
+      ChefSpec::ServerRunner.new(platform: 'ubuntu', version: '12.04') do |node|
+        node.set['consul']['gomaxprocs'] = 8
+      end.converge(described_recipe)
+    end
+
+    it { expect(chef_run.node.consul.gomaxprocs).to eq(8) }
   end
 end

spec/unit/recipes/_service_spec.rb

@@ -40,7 +40,7 @@
         .with(mode: 0755)
     end
     it do
-      expect(chef_run).to render_file('/etc/sysconfig/consul').with_content('GOMAXPROCS=1')
+      expect(chef_run).to render_file('/etc/sysconfig/consul').with_content('GOMAXPROCS=2')
     end
   end
 
@@ -59,7 +59,7 @@
         .with(mode: 0755)
     end
     it do
-      expect(chef_run).to render_file('/etc/default/consul').with_content('GOMAXPROCS=1')
+      expect(chef_run).to render_file('/etc/default/consul').with_content('GOMAXPROCS=2')
     end
   end
 

templates/default/consul-sysconfig.erb

@@ -1 +1 @@
-GOMAXPROCS=<%= node['cpu']['total'] %>
+GOMAXPROCS=<%= node['consul']['gomaxprocs'] %>