openstack/devstack
Code Issues Proposed changes

Address nits on lib/placement for extracted placement

Browse Source 
Earlier review [1] suggested some cleanups which have been
done here:

* Removing a redundant call from cleanup_placement
* Fixing a typo in a comment

[1] https://review.openstack.org/#/c/600162/15/lib/placement

Change-Id: I8abd2f02b123c6c1937c026ff13eb4e600de3202
This commit is contained in:
Chris Dent 2018-11-20 22:18:26 +00:00
parent 78a564bb03
commit 3027c20545
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lib/placement
View File

@ -62,7 +62,6 @@ function is_placement_enabled {
# cleanup_placement() - Remove residual data files, anything left over from previous # cleanup_placement() - Remove residual data files, anything left over from previous
# runs that a clean run would need to clean up # runs that a clean run would need to clean up
function cleanup_placement { function cleanup_placement {
sudo rm -f $(apache_site_config_for nova-placement-api)
sudo rm -f $(apache_site_config_for placement-api) sudo rm -f $(apache_site_config_for placement-api)
remove_uwsgi_config "$PLACEMENT_UWSGI_CONF" "$PLACEMENT_UWSGI" remove_uwsgi_config "$PLACEMENT_UWSGI_CONF" "$PLACEMENT_UWSGI"
sudo rm -f $PLACEMENT_AUTH_CACHE_DIR/* sudo rm -f $PLACEMENT_AUTH_CACHE_DIR/*
@ -107,7 +106,7 @@ function configure_placement_nova_compute {
iniset $conf placement auth_strategy $PLACEMENT_AUTH_STRATEGY iniset $conf placement auth_strategy $PLACEMENT_AUTH_STRATEGY
} }
# create_placement_conf() - Write confg # create_placement_conf() - Write config
function create_placement_conf { function create_placement_conf {
rm -f $PLACEMENT_CONF rm -f $PLACEMENT_CONF
iniset $PLACEMENT_CONF placement_database connection `database_connection_url placement` iniset $PLACEMENT_CONF placement_database connection `database_connection_url placement`