Nova does not need to authenticate with Neutron as admin to get the list of API extensions.

Bug #1250850 reported by Phil Day
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Medium
Phil Day

Bug Description

Currently _refresh_neutron_extensions_cache() always authenticates with Neutron using the Configured Neutron admin credentials because it doesn’t have access to the calling context. But it’s actually always called when a valid context exists, and the call to get the list of extensions is not itself an admin operation. By passing the context down we can avoid this additional Authentication step.

Phil Day (philip-day)
description: updated
Revision history for this message
Phil Day (philip-day) wrote :
Changed in nova:
assignee: nobody → Phil Day (philip-day)
status: New → In Progress
Phil Day (philip-day)
Changed in nova:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/56258
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=af067cffbfbb3e18c48e52b761d371f91bd8c287
Submitter: Jenkins
Branch: master

commit af067cffbfbb3e18c48e52b761d371f91bd8c287
Author: Phil Day <email address hidden>
Date: Wed Nov 13 17:10:42 2013 +0000

    Remove admin auth when getting the list of Neutron API extensions.

    Currently _refresh_neutron_extensions_cache() always authenticates
    with Neutron using the configured Neutron admin credentials because
    it doesn't have access to the calling context. But it's actually
    always called when a valid context exists, and the call to get the
    list of extensions is not itself an admin operation. By passing
    the context down we can avoid this additional authentication step.

    Change-Id: I9e7d9c5e424a6b39c8f8ee1c79f5fd8f48ad81a2
    Closes-Bug: 1250850

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