openstack/devstack
Code Issues Proposed changes

Remove the heat stack owner role

Browse Source 
We should not require any special role for heat
since very long time.

We should use the same roles as with the primary user.

Change-Id: Id9150f94c30505ed0da33b8fbc2a5a7bd4fcf5d0
This commit is contained in:
Attila Fazekas
2016-10-11 08:23:48 +02:00
parent 148d0e6e08
commit cac6ef09c5
1 changed files with 1 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
lib/tempest
View File

@@ -390,11 +390,6 @@ function configure_tempest {
# Orchestration Tests
if is_service_enabled heat; then
# Though this is not needed by heat, some tempest tests explicitly
# try to set this role. Removing them from the tempest tests breaks
# some non-devstack CIs.
get_or_create_role "heat_stack_owner"
if [[ ! -z "$HEAT_CFN_IMAGE_URL" ]]; then
iniset $TEMPEST_CONFIG orchestration image_ref $(basename "${HEAT_CFN_IMAGE_URL%.*}")
fi
@@ -408,7 +403,7 @@ function configure_tempest {
iniset $TEMPEST_CONFIG orchestration instance_type "m1.heat"
fi
iniset $TEMPEST_CONFIG orchestration build_timeout 900
iniset $TEMPEST_CONFIG orchestration stack_owner_role "heat_stack_owner"
iniset $TEMPEST_CONFIG orchestration stack_owner_role Member
fi
# Scenario