From 6fca8a6e1140321bfb99051f8ffcc1796267c777 Mon Sep 17 00:00:00 2001 From: Andrew Clay Shafer Date: Thu, 14 Jun 2012 14:42:44 -0400 Subject: [PATCH 1/2] Refactor database creation Use helper function to create db, user and grants --- recipes/nova-setup.rb | 44 +++++++------------------------------------ 1 file changed, 7 insertions(+), 37 deletions(-) diff --git a/recipes/nova-setup.rb b/recipes/nova-setup.rb index e20e8dd4..d18e896d 100644 --- a/recipes/nova-setup.rb +++ b/recipes/nova-setup.rb @@ -35,43 +35,13 @@ end include_recipe "nova::nova-common" include_recipe "mysql::client" -if Chef::Config[:solo] - Chef::Log.warn("This recipe uses search. Chef Solo does not support search.") -else - # Lookup mysql ip address - mysql_server = search(:node, "roles:mysql-master AND chef_environment:#{node.chef_environment}") - if mysql_server.length > 0 - Chef::Log.info("nova::nova-common/mysql: using search") - db_ip_address = mysql_server[0]['mysql']['bind_address'] - db_root_password = mysql_server[0]['mysql']['server_root_password'] - else - Chef::Log.info("nova::nova-common/mysql: NOT using search") - db_ip_address = node['mysql']['bind_address'] - db_root_password = node['mysql']['server_root_password'] - end -end - -connection_info = {:host => db_ip_address, :username => "root", :password => db_root_password} -mysql_database "create nova database" do - connection connection_info - database_name node["nova"]["db"]["name"] - action :create -end - -mysql_database_user node["nova"]["db"]["username"] do - connection connection_info - password node["nova"]["db"]["password"] - action :create -end - -mysql_database_user node["nova"]["db"]["username"] do - connection connection_info - password node["nova"]["db"]["password"] - database_name node["nova"]["db"]["name"] - host '%' - privileges ["all"] - action :grant -end +#creates db and user +#function defined in osops-utils/libraries +create_db_and_user("mysql", + "nova", + node["nova"]["db"]["name"], + node["nova"]["db"]["username"], + node["nova"]["db"]["password"]) execute "nova-manage db sync" do command "nova-manage db sync" From 5d87c38b26e54c8e748afaea3d2adfbfbd96e89b Mon Sep 17 00:00:00 2001 From: Andrew Clay Shafer Date: Thu, 14 Jun 2012 17:09:18 -0400 Subject: [PATCH 2/2] now with less parameters take out redunant service parameter --- recipes/nova-setup.rb | 1 - 1 file changed, 1 deletion(-) diff --git a/recipes/nova-setup.rb b/recipes/nova-setup.rb index d18e896d..1be37d62 100644 --- a/recipes/nova-setup.rb +++ b/recipes/nova-setup.rb @@ -38,7 +38,6 @@ include_recipe "mysql::client" #creates db and user #function defined in osops-utils/libraries create_db_and_user("mysql", - "nova", node["nova"]["db"]["name"], node["nova"]["db"]["username"], node["nova"]["db"]["password"])