From 76bce0ada4dbb0b59f4936ef4f9a6d76f82066f0 Mon Sep 17 00:00:00 2001
From: ZhongShengping <chdzsp@163.com>
Date: Mon, 5 Mar 2018 08:51:30 +0800
Subject: [PATCH] Fix mysql connection

URL mysql://scott:***@some_host/some_db does not contain a '+drivername'
portion, and will make use of a default driver. A full dbname+drivername://
protocol is recommended. For MySQL, it is strongly recommended that
mysql+pymysql:// be specified for maximum service compatibility.

Change-Id: Ib5cbfcc412c04381f2fce9f319a313eb86ea7fa8
Closes-Bug: #1753342
---
 manifests/db.pp                    | 4 ++--
 spec/classes/ceilometer_db_spec.rb | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/manifests/db.pp b/manifests/db.pp
index 1b299e48..d7aef852 100644
--- a/manifests/db.pp
+++ b/manifests/db.pp
@@ -13,7 +13,7 @@
 #
 # [*database_connection*]
 #   (Optional) Url used to connect to database.
-#   Defaults to 'mysql://ceilometer:ceilometer@localhost/ceilometer'.
+#   Defaults to 'mysql+pymysql://ceilometer:ceilometer@localhost/ceilometer'.
 #
 # [*database_idle_timeout*]
 #   (Optional) Timeout when db connections should be reaped.
@@ -46,7 +46,7 @@
 #
 class ceilometer::db (
   $database_db_max_retries = $::os_service_default,
-  $database_connection     = 'mysql://ceilometer:ceilometer@localhost/ceilometer',
+  $database_connection     = 'mysql+pymysql://ceilometer:ceilometer@localhost/ceilometer',
   $database_idle_timeout   = $::os_service_default,
   $database_min_pool_size  = $::os_service_default,
   $database_max_pool_size  = $::os_service_default,
diff --git a/spec/classes/ceilometer_db_spec.rb b/spec/classes/ceilometer_db_spec.rb
index 6221dced..18c9882b 100644
--- a/spec/classes/ceilometer_db_spec.rb
+++ b/spec/classes/ceilometer_db_spec.rb
@@ -10,7 +10,7 @@ describe 'ceilometer::db' do
       it { is_expected.to contain_class('ceilometer::db::sync') }
       it { is_expected.to contain_oslo__db('ceilometer_config').with(
         :db_max_retries => '<SERVICE DEFAULT>',
-        :connection     => 'mysql://ceilometer:ceilometer@localhost/ceilometer',
+        :connection     => 'mysql+pymysql://ceilometer:ceilometer@localhost/ceilometer',
         :idle_timeout   => '<SERVICE DEFAULT>',
         :min_pool_size  => '<SERVICE DEFAULT>',
         :max_pool_size  => '<SERVICE DEFAULT>',