keystone integration redux

Bug #917408 reported by Jesse Andrews
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Medium
Vish Ishaya
OpenStack Identity (keystone)
Fix Released
Medium
Ziad Sawalha
devstack
Fix Released
Medium
Vish Ishaya

Bug Description

Currently the keystone middleware for nova is located in keystone (nova_auth_token).

It was put there due to the timing of keystone + nova during the diablo release. The middleware should be moved to nova (as it is a keystone middleware to internal nova datastructure middleware).

This will be done with several changes:

* add nova_auth_token into nova
* update nova_auth_token in keystone to say that it is deprecated, use the version in nova
* update devstack to use the nova version
* remove nova_auth_token during E4

Changed in devstack:
assignee: nobody → Vish Ishaya (vishvananda)
Changed in nova:
assignee: nobody → Vish Ishaya (vishvananda)
Changed in nova:
status: New → In Progress
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

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

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

Reviewed: https://review.openstack.org/3135
Committed: http://github.com/openstack/nova/commit/a47b5276244dba9c0a9058e6868522cf3fb3cada
Submitter: Jenkins
Branch: master

commit a47b5276244dba9c0a9058e6868522cf3fb3cada
Author: Vishvananda Ishaya <email address hidden>
Date: Wed Jan 11 10:42:00 2012 -0800

    Brings back keystone middleware

     * Adds ec2 auth middleware
     * Adds keystonecontext middleware
     * Notes alternative configurations in paste config
     * Fixes nova portion of bug 917408

    Change-Id: I9909d6d235445b1413d1cc0fb61e99e1ab819438

Changed in nova:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

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

Changed in nova:
assignee: Vish Ishaya (vishvananda) → Ziad Sawalha (ziad-sawalha)
status: Fix Committed → In Progress
Revision history for this message
Dolph Mathews (dolph) wrote :

I assume this is in progress on both sides of the fence ;)

Changed in keystone:
assignee: nobody → Ziad Sawalha (ziad-sawalha)
importance: Undecided → Medium
status: New → In Progress
Changed in devstack:
status: New → Fix Committed
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/3695
Committed: http://github.com/openstack/keystone/commit/9858e08d44c89f5aa7bf38e5235e4c05fffdb96d
Submitter: Jenkins
Branch: master

commit 9858e08d44c89f5aa7bf38e5235e4c05fffdb96d
Author: Vishvananda Ishaya <email address hidden>
Date: Thu Feb 2 11:36:26 2012 -0800

    Removes nova middleware and config from keystone

     * Everything has been moved back into nova
     * Fixes bug 917408

    Change-Id: Id7e3a0eaa90cc0a32f84fdd587d50e2eba00800b

Changed in keystone:
status: In Progress → Fix Committed
Changed in nova:
assignee: Ziad Sawalha (ziad-sawalha) → Vish Ishaya (vishvananda)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/3365
Committed: http://github.com/openstack/nova/commit/6b532096f6c72627dccb32fa49e8045d6afbdc6d
Submitter: Jenkins
Branch: master

commit 6b532096f6c72627dccb32fa49e8045d6afbdc6d
Author: Ziad Sawalha <email address hidden>
Date: Tue Jan 24 13:03:32 2012 -0600

    Use Keystone Extension Syntax for EC2 Creds

    - Syntax is OS-KSEC2:ec2Credentials (not ec2Credentials)
    - adds to fix for bug 917408
    - Syntax also requires a call to /tokens instead of /ec2tokens
    - Changes are compatible (old syntax supported)
    - Warning will be issued to log if URL syntax is old

    Change-Id: I5ff9d1643122de2f383517e60f15852720c54490

Changed in nova:
status: In Progress → Fix Committed
Joseph Heck (heckj)
Changed in keystone:
milestone: none → essex-4
Thierry Carrez (ttx)
Changed in keystone:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: none → essex-4
status: Fix Committed → Fix Released
Dean Troyer (dtroyer)
Changed in devstack:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in keystone:
milestone: essex-4 → 2012.1
Thierry Carrez (ttx)
Changed in nova:
milestone: essex-4 → 2012.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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