Admin user used in test_encryption_vol_type should be configurable

Bug #1546993 reported by Rakesh H S
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Heat
Fix Released
Medium
Rakesh H S

Bug Description

Since volume encryption type can be created only by the admin user, the integration test tries to switch to admin user internally https://github.com/openstack/heat/blob/master/heat_integrationtests/functional/test_encryption_vol_type.py#L49

This will work in devstack, but in a environment where admin user is named differently or does not have the same password as that of the demo user, it will fail.

The admin creds should be made available via heat_integrationtests.conf.
The tests which require admin user should be run only if admin creds are available and use the same.

Rakesh H S (rh-s)
Changed in heat:
assignee: nobody → Rakesh H S (rh-s)
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (master)

Fix proposed to branch: master
Review: https://review.openstack.org/281856

Changed in heat:
status: New → In Progress
Changed in heat:
milestone: none → mitaka-3
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to heat (master)

Reviewed: https://review.openstack.org/281856
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=c7ab4e82afc4cf3fa7d130ecaad5ace18eb46852
Submitter: Jenkins
Branch: master

commit c7ab4e82afc4cf3fa7d130ecaad5ace18eb46852
Author: Rakesh H S <email address hidden>
Date: Thu Feb 18 18:54:00 2016 +0530

    Fix test_encryption_vol_type to take admin creds from conf

    Heat integration test test_encryption_vol_type will not work in environments
    where admin user is named differently or does not have the same password as
    that of the demo user.

    The admin credentials is now made available via heat_integrationtests.conf.
    The tests which require admin privileges should be run only if admin
    credentials are available and use the same.

    Change-Id: I5f329be6d4c4249763e0aec8696fb87605a0812a
    Closes-Bug: #1546993

Changed in heat:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (stable/liberty)

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/287173

Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/heat 6.0.0.0b3

This issue was fixed in the openstack/heat 6.0.0.0b3 development milestone.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to heat (stable/liberty)

Reviewed: https://review.openstack.org/287173
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=05a7694cdcb7573ca1f67f029299bf5f3b6f1240
Submitter: Jenkins
Branch: stable/liberty

commit 05a7694cdcb7573ca1f67f029299bf5f3b6f1240
Author: Rakesh H S <email address hidden>
Date: Thu Feb 18 18:54:00 2016 +0530

    Fix test_encryption_vol_type to take admin creds from conf

    Heat integration test test_encryption_vol_type will not work in environments
    where admin user is named differently or does not have the same password as
    that of the demo user.

    The admin credentials is now made available via heat_integrationtests.conf.
    The tests which require admin privileges should be run only if admin
    credentials are available and use the same.

    Change-Id: I5f329be6d4c4249763e0aec8696fb87605a0812a
    Closes-Bug: #1546993

tags: added: in-stable-liberty
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/heat 5.0.2

This issue was fixed in the openstack/heat 5.0.2 release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.