TenantName is set as TenantId in Instance and Glance Images Tables

Bug #870351 reported by Antony Messerli
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
Undecided
Kevin L. Mitchell

Bug Description

If I provision an instance using TenantName, the project_id of the instance is set as TenantId. This should remain as TenantName.

For instance my TenantName is 'os-project' and the ID of it is 1 in Keystone. This should still utilize TenantName in Instances and Images tables or at least have a setting to use one of the other.

Revision history for this message
Kevin L. Mitchell (klmitch) wrote :

I talked with Ziad, and the basic idea here is that tenant names are potentially mutable, while tenant IDs are immutable. I then talked with Ant, who pointed out that, given that keystone has absolutely no database migration capability, it's hard to keep IDs lined up.

My approach to fixing this, in the short term, is to restore the behavior that X_TENANT reports the tenant name; however, I also added X_TENANT_ID to report the tenant name, and the relevant shim layers can figure out what the heck they want to do with it. Ultimately, long term, the IDs are what need to be stored, while the names should be used for presentation purposes, but Keystone really needs database migrations...

Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : A change has been merged to openstack/keystone

Reviewed: https://review.openstack.org/824
Committed: http://github.com/openstack/keystone/commit/625fcebc0c572ecf8fa738d720d5424dc8d39546
Submitter: Jenkins
Branch: master

 status fixcommitted
 done

commit 625fcebc0c572ecf8fa738d720d5424dc8d39546
Author: Kevin L. Mitchell <email address hidden>
Date: Mon Oct 10 14:24:32 2011 -0500

    Use the tenant name for X_TENANT

    Fixes bug 870351

    Change-Id: I2cf12a28c6dc29595ef490acdb511f604d86d3ee

Changed in keystone:
status: New → Fix Committed
Joe Savak (jsavak)
tags: added: diablo-backport
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix merged to keystone (stable/diablo)

Reviewed: https://review.openstack.org/1333
Committed: http://github.com/openstack/keystone/commit/e17b0fc80946414819c47f354d5574c170e872e9
Submitter: Jenkins
Branch: stable/diablo

 status fixcommitted
 done

commit e17b0fc80946414819c47f354d5574c170e872e9
Author: Kevin L. Mitchell <email address hidden>
Date: Mon Oct 10 14:24:32 2011 -0500

    Use the tenant name for X_TENANT

    Fixes bug 870351

    Change-Id: I2cf12a28c6dc29595ef490acdb511f604d86d3ee

Thierry Carrez (ttx)
Changed in keystone:
milestone: none → essex-1
Changed in keystone:
assignee: nobody → Kevin L. Mitchell (klmitch)
Thierry Carrez (ttx)
Changed in keystone:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in keystone:
milestone: essex-1 → 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.