]> review.fuel-infra Code Review - puppet-modules/puppet-ceilometer.git/commitdiff
Add pool_timeout option
authorZhongShengping <chdzsp@163.com>
Thu, 22 Mar 2018 03:09:50 +0000 (11:09 +0800)
committerZhongShengping <chdzsp@163.com>
Thu, 22 Mar 2018 03:09:50 +0000 (11:09 +0800)
Add pool_timeout option to configure this value for pool_timeout with
SQLAlchemy.

Change-Id: Ib84d68f367c043b40f15ec50d45093467f11a4f4
Closes-Bug: #1757581

manifests/db.pp
releasenotes/notes/add-pool_timeout-option-for-db-b8801b27b2caa1c9.yaml [new file with mode: 0644]
spec/classes/ceilometer_db_spec.rb

index d7aef852b669f75e5daf1ad5998b93eb7fd9fe4b..974933b08b689d2ac924802217d23fbd637dd0c4 100644 (file)
 #   (Optional) enable database schema installation.
 #   Defaults to true.
 #
+# [*database_pool_timeout*]
+#   (Optional) If set, use this value for pool_timeout with SQLAlchemy.
+#   Defaults to $::os_service_default
+#
 class ceilometer::db (
   $database_db_max_retries = $::os_service_default,
   $database_connection     = 'mysql+pymysql://ceilometer:ceilometer@localhost/ceilometer',
@@ -53,6 +57,7 @@ class ceilometer::db (
   $database_max_retries    = $::os_service_default,
   $database_retry_interval = $::os_service_default,
   $database_max_overflow   = $::os_service_default,
+  $database_pool_timeout   = $::os_service_default,
   $sync_db                 = true,
 ) {
 
@@ -67,6 +72,7 @@ class ceilometer::db (
     retry_interval => $database_retry_interval,
     max_pool_size  => $database_max_pool_size,
     max_overflow   => $database_max_overflow,
+    pool_timeout   => $database_pool_timeout,
   }
 
   if $sync_db {
diff --git a/releasenotes/notes/add-pool_timeout-option-for-db-b8801b27b2caa1c9.yaml b/releasenotes/notes/add-pool_timeout-option-for-db-b8801b27b2caa1c9.yaml
new file mode 100644 (file)
index 0000000..be30389
--- /dev/null
@@ -0,0 +1,5 @@
+---
+features:
+  - |
+    Adds the pool_timeout option for configuring oslo.db. This will
+    configure this value for pool_timeout with SQLAlchemy.
index 18c9882b62147eda66f1bb5b0798d91d37fb9ed4..4fae14a104299c283dab176a5424823bad567d1a 100644 (file)
@@ -17,6 +17,7 @@ describe 'ceilometer::db' do
         :max_retries    => '<SERVICE DEFAULT>',
         :retry_interval => '<SERVICE DEFAULT>',
         :max_overflow   => '<SERVICE DEFAULT>',
+        :pool_timeout   => '<SERVICE DEFAULT>',
       )}
 
     end
@@ -32,6 +33,7 @@ describe 'ceilometer::db' do
           :database_max_retries    => '11',
           :database_retry_interval => '11',
           :database_max_overflow   => '21',
+          :database_pool_timeout   => '21',
           :sync_db                 => false }
       end
 
@@ -45,6 +47,7 @@ describe 'ceilometer::db' do
         :max_retries    => '11',
         :retry_interval => '11',
         :max_overflow   => '21',
+        :pool_timeout   => '21',
       )}
 
     end