Remove password hash generation in each puppet modules

... and migrate it to openstacklib so that all logics about database
configuration are implemented in one common place.

Depends-on: https://review.opendev.org/#/c/728595/
Change-Id: I6a46e45b8af4c8c1cdf7913bc625b57e49a79687

diff --git a/manifests/db/mysql.pp b/manifests/db/mysql.pp
index 8963f0bff3d3cc0232193010281ee4c01f7f546b..daa802c808c4bc42586f95e475e5f2c403a7cd55 100644 (file)
--- a/manifests/db/mysql.pp
+++ b/manifests/db/mysql.pp
@@ -48,7 +48,7 @@ class ceilometer::db::mysql(
 
   openstacklib::db::mysql { 'ceilometer':
     user          => $user,
-    password_hash => mysql::password($password),
+    password      => $password,
     dbname        => $dbname,
     host          => $host,
     charset       => $charset,

diff --git a/manifests/db/postgresql.pp b/manifests/db/postgresql.pp
index 818d871f758a7e94ef9856585a4cded779948935..7bff4a8b2ac5e31250d05265db7e8ca4f558ed93 100644 (file)
--- a/manifests/db/postgresql.pp
+++ b/manifests/db/postgresql.pp
@@ -35,11 +35,11 @@ class ceilometer::db::postgresql(
   include ceilometer::deps
 
   openstacklib::db::postgresql { 'ceilometer':
-    password_hash => postgresql_password($user, $password),
-    dbname        => $dbname,
-    user          => $user,
-    encoding      => $encoding,
-    privileges    => $privileges,
+    password   => $password,
+    dbname     => $dbname,
+    user       => $user,
+    encoding   => $encoding,
+    privileges => $privileges,
   }
 
   Anchor['ceilometer::db::begin']

diff --git a/metadata.json b/metadata.json
index bf298e664736ac0dfc103ade8a91711c4f12b357..89df6cf15e34534f3db2c6f684a0f0b9dbe2e9d7 100644 (file)
--- a/metadata.json
+++ b/metadata.json
@@ -13,10 +13,6 @@
             "name": "puppet/rabbitmq",
             "version_requirement": ">=8.4.0 <11.0.0"
         },
-        {
-            "name": "puppetlabs/mysql",
-            "version_requirement": ">=6.0.0 <11.0.0"
-        },
         {
             "name": "openstack/keystone",
             "version_requirement": ">=16.3.0 <17.0.0"

diff --git a/spec/classes/ceilometer_db_mysql_spec.rb b/spec/classes/ceilometer_db_mysql_spec.rb
index 5075a4d1c8f2be60096259024392dc55487b436a..377b214401c45a296e621ee05e5b44ae5d8f081f 100644 (file)
--- a/spec/classes/ceilometer_db_mysql_spec.rb
+++ b/spec/classes/ceilometer_db_mysql_spec.rb
@@ -7,7 +7,7 @@ describe 'ceilometer::db::mysql' do
   end
 
   let :params do
-    { :password     => 's3cr3t',
+    { :password     => 'ceilometerpass',
       :dbname       => 'ceilometer',
       :user         => 'ceilometer',
       :host         => 'localhost',
@@ -25,10 +25,10 @@ describe 'ceilometer::db::mysql' do
 
     it 'creates a mysql database' do
       is_expected.to contain_openstacklib__db__mysql( params[:dbname] ).with(
-        :user          => params[:user],
-        :password_hash => '*58C036CDA51D8E8BBBBF2F9EA5ABF111ADA444F0',
-        :host          => params[:host],
-        :charset       => params[:charset]
+        :user     => params[:user],
+        :password => params[:password],
+        :host     => params[:host],
+        :charset  => params[:charset]
       )
     end
 

diff --git a/spec/classes/ceilometer_db_postgresql_spec.rb b/spec/classes/ceilometer_db_postgresql_spec.rb
index 3420affbc3fd82ed488ea7359303df258ee0c242..77202b755a9f88344403303400def321af5200b4 100644 (file)
--- a/spec/classes/ceilometer_db_postgresql_spec.rb
+++ b/spec/classes/ceilometer_db_postgresql_spec.rb
@@ -4,7 +4,7 @@ describe 'ceilometer::db::postgresql' do
 
   shared_examples_for 'ceilometer::db::postgresql' do
     let :req_params do
-      { :password => 'pw' }
+      { :password => 'ceilometerpass' }
     end
 
     let :pre_condition do
@@ -16,9 +16,12 @@ describe 'ceilometer::db::postgresql' do
         req_params
       end
 
-      it { is_expected.to contain_postgresql__server__db('ceilometer').with(
-        :user     => 'ceilometer',
-        :password => 'md52899c518e96bce6633ab342b0ca38292'
+      it { is_expected.to contain_openstacklib__db__postgresql('ceilometer').with(
+        :user       => 'ceilometer',
+        :password   => 'ceilometerpass',
+        :dbname     => 'ceilometer',
+        :encoding   => nil,
+        :privileges => 'ALL',
       )}
     end