Comment 2 for bug 916199

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/3047
Committed: http://github.com/openstack/keystone/commit/7c0529fac859c9131aae3986d81fdb10810a49c1
Submitter: Jenkins
Branch: master

commit 7c0529fac859c9131aae3986d81fdb10810a49c1
Author: Ewan Mellor <email address hidden>
Date: Fri Jan 13 14:01:34 2012 -0800

    Bug #916199: keystone-manage service list fails with AttributeError on Service.description

    Make sqlalchemy.api.service take a keystone.models.Service into create, and
    return the same from get_all. This replaces their use of
    sqlalchemy.models.Service in these positions. This is consistent with
    User, Tenant, Token, and fixes the bug referenced above.

    Added unit tests for keystone.manage.api.{list_services,add_service,
    add_token}. The latter isn't actually related to this bug (that call turned
    out to be working fine) but I've written it so you may as well have it!

    Fix tearDown in test_backends. These tests were leaving the DB layer in an
    invalid state, so the subsequent test_commands_v1 tests couldn't initialize.

    Change-Id: I63e24189832cb96454c085797bff4c5c09e10b70