endpoint-create fails

Bug #961412 reported by Alan Pevec
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
Critical
Dolph Mathews

Bug Description

Creating endpoints with "keystone endpoint-create" and sql catalog backend fails:

Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/keystone/common/wsgi.py", line 184, in
__call__
    result = method(context, **params)
  File "/usr/lib/python2.7/site-packages/keystone/catalog/core.py", line 169, in
 create_endpoint
    if not self.catalog_api.service_exists(context, service_id):
  File "/usr/lib/python2.7/site-packages/keystone/common/manager.py", line 44, i
n __getattr__
    f = getattr(self.driver, name)
AttributeError: 'Catalog' object has no attribute 'service_exists'

This is a regression after https://review.openstack.org/5358
https://review.openstack.org/#patch,unified,5358,8,keystone/catalog/core.py

Dolph Mathews (dolph)
Changed in keystone:
assignee: nobody → Dolph Mathews (dolph)
importance: Undecided → Critical
status: New → Confirmed
milestone: none → essex-rc1
Revision history for this message
Jay Pipes (jaypipes) wrote :

I removed the service_exists method from the base driver here:

https://review.openstack.org/#change,5294

Perhaps there was a bad merge conflict resolution?

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/5636

Changed in keystone:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/5636
Committed: http://github.com/openstack/keystone/commit/9feb00085f75ea2697fd2225e6003c2384904d08
Submitter: Jenkins
Branch: master

commit 9feb00085f75ea2697fd2225e6003c2384904d08
Author: Dolph Mathews <email address hidden>
Date: Wed Mar 21 13:11:31 2012 -0500

    Fix critical typo in endpoint_create (bug 961412)

    It looks like catalog crud was previously untested.

    Change-Id: I8e3060b6d6c737d3d97a5bd9076e9a5fdf9945e2

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: essex-rc1 → 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.