From: Grant Slater <git@firefishy.com>
Date: Fri, 7 Aug 2020 00:33:19 +0000 (+0100)
Subject: squid: use different exec_start based on ubuntu release
X-Git-Url: https://git.openstreetmap.org/chef.git/commitdiff_plain/b0fa48094234e76b98eff642e0339c3626c7bfeb

squid: use different exec_start based on ubuntu release
---

diff --git a/cookbooks/squid/recipes/default.rb b/cookbooks/squid/recipes/default.rb
index fbfc1daef..37352b2c9 100644
--- a/cookbooks/squid/recipes/default.rb
+++ b/cookbooks/squid/recipes/default.rb
@@ -98,6 +98,12 @@ file "/etc/logrotate.d/squid.dpkg-dist" do
   action :delete
 end
 
+squid_service_exec = if node[:lsb][:release].to_f < 20.04
+                       "/usr/sbin/squid -YC"
+                     else
+                       "/usr/sbin/squid --foreground -sYC"
+                     end
+
 systemd_service "squid" do
   dropin "chef"
   limit_nofile 98304
@@ -107,7 +113,7 @@ systemd_service "squid" do
   protect_home true
   restrict_address_families address_families
   restart "always"
-  exec_start "/usr/sbin/squid --foreground -YC"
+  exec_start "#{squid_service_exec}"
 end
 
 # Quick hack to cleanup bloated journal
diff --git a/cookbooks/squid/templates/default/squid.conf.erb b/cookbooks/squid/templates/default/squid.conf.erb
index 3a74a6c12..61008078a 100644
--- a/cookbooks/squid/templates/default/squid.conf.erb
+++ b/cookbooks/squid/templates/default/squid.conf.erb
@@ -10,7 +10,7 @@ workers <%= [ node[:cpu][:total] - 2, 1 ].max.ceil %>
 cpu_affinity_map process_numbers=<%= (1..[ node[:cpu][:total] - 2, 1 ].max.ceil).to_a.join(',') %> cores=<%=(1..[ node[:cpu][:total] - 2, 1 ].max.ceil).to_a.join(',') %>
 
 # Set short clean shutdown interval
-shutdown_lifetime 10 seconds
+shutdown_lifetime 2 seconds
 
 error_log_languages off
 <% end -%>