Parameterize the jira::dburl variable

This is so $dburl can be overriden as a workaround for the following bug in Mysql Connector/J 5.1.16: https://confluence.atlassian.com/display/JIRAKB/%27NullPointerException+at+com.mysql.jdbc.ConnectionImpl.getServerCharacterEncoding%27+Error+When+Using+MySQL+Driver+v5.1.16
voxpupuli · Nov 24, 2014 · 93175f7 · 93175f7
1 parent 5c6d35a
commit 93175f7
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 4 deletions.
diff --git a/manifests/init.pp b/manifests/init.pp
@@ -52,6 +52,7 @@
   $dbport                  = '5432',
   $dbdriver                = 'org.postgresql.Driver',
   $dbtype                  = 'postgres72',
+  $dburl                   = undef,
   $poolsize                = '20',
   $mysql_connector_package = $jira::params::mysql_connector_package,
   $mysql_connector_jar     = $jira::params::mysql_connector_jar,
@@ -118,7 +119,15 @@
   }
 
   $webappdir    = "${installdir}/atlassian-${product}-${version}-standalone"
-  $dburl        = "jdbc:${db}://${dbserver}:${dbport}/${dbname}"
+  if $dburl {
+    $dburl_real = $dburl
+  }
+  else {
+    $dburl_real = $db ? {
+      'postgresql' => "jdbc:${db}://${dbserver}:${dbport}/${dbname}",
+      'mysql'      => "jdbc:${db}://${dbserver}:${dbport}/${dbname}?useUnicode=true&amp;characterEncoding=UTF8&amp;sessionVariables=storage_engine=InnoDB",
+    }
+  }
 
   anchor { 'jira::start':
   } ->

diff --git a/templates/dbconfig.mysql.xml.erb b/templates/dbconfig.mysql.xml.erb
@@ -5,7 +5,7 @@
   <delegator-name>default</delegator-name>
   <database-type>mysql</database-type>
   <jdbc-datasource>
-    <url><%= scope.lookupvar('jira::dburl') %>?useUnicode=true&amp;characterEncoding=UTF8&amp;sessionVariables=storage_engine=InnoDB</url>
+    <url><%= scope.lookupvar('jira::dburl_real') %></url>
     <driver-class><%= scope.lookupvar('jira::dbdriver') %></driver-class>
     <username><%= scope.lookupvar('jira::dbuser') %></username>
     <password><%= scope.lookupvar('jira::dbpassword') %></password>
@@ -22,4 +22,4 @@
     <pool-test-while-idle><%= scope.lookupvar('jira::poolTestWhileIdle') %></pool-test-while-idle>
     <validation-query-timeout>3</validation-query-timeout>
   </jdbc-datasource>
-</jira-database-config>
+</jira-database-config>
diff --git a/templates/dbconfig.postgresql.xml.erb b/templates/dbconfig.postgresql.xml.erb
@@ -6,7 +6,7 @@
   <database-type><%= scope.lookupvar('jira::dbtype') %></database-type>
   <schema-name>public</schema-name>
   <jdbc-datasource>
-    <url><%= scope.lookupvar('jira::dburl') %></url>
+    <url><%= scope.lookupvar('jira::dburl_real') %></url>
     <driver-class><%= scope.lookupvar('jira::dbdriver') %></driver-class>
     <username><%= scope.lookupvar('jira::dbuser') %></username>
     <password><%= scope.lookupvar('jira::dbpassword') %></password>