keystone-manage service list fails with AttributeError on Service.description

Bug #916199 reported by Ewan Mellor
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
Undecided
Ewan Mellor

Bug Description

keystone-manage service list fails with AttributeError: 'Service' object has no attribute 'description'.

This is because the column is called 'desc' on the DB model object but 'description' on the resource model object, and the two have been mixed up.

Ewan Mellor (ewanmellor)
Changed in keystone:
status: New → In Progress
assignee: nobody → Ewan Mellor (ewanmellor)
Ewan Mellor (ewanmellor)
summary: - AttributeError: 'Service' object has no attribute 'description'
+ keystone-manage service list fails with AttributeError on
+ Service.description
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to keystone (master)

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

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

Changed in keystone:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in keystone:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in keystone:
milestone: none → essex-3
Thierry Carrez (ttx)
Changed in keystone:
milestone: essex-3 → 2012.1
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.