rundall/ncsa/svcplan-4943:

Allow slurmdbd_storage_port to be an empty string ("" or ''), which supports
slurmdbd connecting to the DB via a socket rather than via TCP/IP.

Adjust the template for slurmdbd.conf so that if any value is an empty string (''),
then put "" in as the value.

REFERENCE.md

@@ -1076,7 +1076,7 @@ Default value: `'slurmdbd'`
 
 ##### <a name="-slurm--slurmdbd_storage_port"></a>`slurmdbd_storage_port`
 
-Data type: `Stdlib::Port`
+Data type: `Variant[Stdlib::Port, String[0,0]]`
 
 
 

manifests/init.pp

@@ -335,7 +335,7 @@
   Stdlib::Host $slurmdbd_storage_host  = 'localhost',
   String[1] $slurmdbd_storage_loc   = 'slurm_acct_db',
   String[1] $slurmdbd_storage_pass  = 'slurmdbd',
-  Stdlib::Port $slurmdbd_storage_port  = 3306,
+  Variant[Stdlib::Port, String[0,0]] $slurmdbd_storage_port  = 3306,
   String[1] $slurmdbd_storage_type  = 'accounting_storage/mysql',
   String[1] $slurmdbd_storage_user  = 'slurmdbd',
   String[1] $slurmdbd_db_charset = 'utf8',

templates/slurmdbd/slurmdbd.conf.erb

@@ -13,6 +13,8 @@
 # <%= key %>
 <%- elsif value.is_a?(Array) -%>
 <%= key %>=<%= value.join(',') %>
+<%- elsif value == '' -%>
+<%= key %>=""
 <%- elsif value.is_a?(Hash) -%>
 <%- v = value.map {|k,v| "#{k}=#{v}" }.join(',') %>
 <%= key %>=<%= v %>