openstack/cinder
Code Issues Proposed changes

Minor cleanup on tools/test-setup.sh

Browse Source 
Remove unused variable.
Fix comment nit.

This addresses a review comment in https://review.openstack.org/430069.

Change-Id: I47e75a68c7e97340dab8b51b9fab9ed2983061ef
This commit is contained in:
Andreas Jaeger 2017-02-07 11:39:40 +01:00
parent 45f983807a
commit afeb6f5aeb
1 changed files with 1 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
tools/test-setup.sh
View File

@ -18,7 +18,7 @@ DB_PW=openstack_citest
sudo -H mysqladmin -u root password $DB_ROOT_PW sudo -H mysqladmin -u root password $DB_ROOT_PW
# It's best practice to remove anonymous users from the database. If # It's best practice to remove anonymous users from the database. If
# a anonymous user exists, then it matches first for connections and # an anonymous user exists, then it matches first for connections and
# other connections from that host will not work. # other connections from that host will not work.
sudo -H mysql -u root -p$DB_ROOT_PW -h localhost -e " sudo -H mysql -u root -p$DB_ROOT_PW -h localhost -e "
DELETE FROM mysql.user WHERE User=''; DELETE FROM mysql.user WHERE User='';
@ -33,9 +33,6 @@ mysql -u $DB_USER -p$DB_PW -h 127.0.0.1 -e "
CREATE DATABASE openstack_citest CHARACTER SET utf8;" CREATE DATABASE openstack_citest CHARACTER SET utf8;"
# Same for PostgreSQL # Same for PostgreSQL
# The root password for the PostgreSQL database; pass it in via
# POSTGRES_ROOT_PW.
DB_ROOT_PW=${POSTGRES_ROOT_PW:-insecure_slave}
# Setup user # Setup user
root_roles=$(sudo -H -u postgres psql -t -c " root_roles=$(sudo -H -u postgres psql -t -c "