Skip to content

Commit 67bc8e8

Browse files
author
Chris Dent
committed
Cleanup create_accounts functions
lib/nova and lib/ceilometer had function calls not in the desired form. Change-Id: I6b848e51654a48fe2df6084efdb0f67fd5e180f0
1 parent f33e76b commit 67bc8e8

2 files changed

Lines changed: 4 additions & 7 deletions

File tree

lib/ceilometer

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -73,13 +73,12 @@ function is_ceilometer_enabled {
7373
}
7474

7575
# create_ceilometer_accounts() - Set up common required ceilometer accounts
76-
76+
#
7777
# Project User Roles
7878
# ------------------------------------------------------------------
7979
# SERVICE_TENANT_NAME ceilometer admin
8080
# SERVICE_TENANT_NAME ceilometer ResellerAdmin (if Swift is enabled)
81-
82-
create_ceilometer_accounts() {
81+
function create_ceilometer_accounts {
8382

8483
local service_tenant=$(openstack project list | awk "/ $SERVICE_TENANT_NAME / { print \$2 }")
8584
local admin_role=$(openstack role list | awk "/ admin / { print \$2 }")

lib/nova

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -330,14 +330,12 @@ function configure_nova {
330330
}
331331

332332
# create_nova_accounts() - Set up common required nova accounts
333-
333+
#
334334
# Project User Roles
335335
# ------------------------------------------------------------------
336336
# SERVICE_TENANT_NAME nova admin
337337
# SERVICE_TENANT_NAME nova ResellerAdmin (if Swift is enabled)
338-
339-
# Migrated from keystone_data.sh
340-
create_nova_accounts() {
338+
function create_nova_accounts {
341339

342340
local service_tenant=$(openstack project list | awk "/ $SERVICE_TENANT_NAME / { print \$2 }")
343341
local admin_role=$(openstack role list | awk "/ admin / { print \$2 }")

0 commit comments

Comments
 (0)