get_instance_security_groups() get_admin_context() causes lots of keystone chatter

[Aaron Rosen]

When nova list is called this line is called for each instance which involes getting a new token from keystone. Instead the token should be extracted from req

https://github.com/openstack/nova/blob/master/nova/network/security_group/quantum_driver.py#L255

[OpenStack Infra (hudson-openstack)]

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

[Chris Behrens (cbehrens)]

This bug isn't really valid... that call doesn't grab a new context from keystone. However, I do think the code should be using the context from the request, not creating a new admin context.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/25557
Committed: http://github.com/openstack/nova/commit/1ea02dbcf5ae3e822378a580ac1e54467f664be7
Submitter: Jenkins
Branch: master

commit 1ea02dbcf5ae3e822378a580ac1e54467f664be7
Author: Aaron Rosen <email address hidden>
Date: Wed Mar 27 12:51:33 2013 -0700

    get context from req rather than getting a new admin context

    This patch removes the call to context.get_admin_context() and instead
    obtains the context from the req object. Allowing the context.auth_token
    to be reused.

    Fixes bug 1161081

    Change-Id: I3b104edc2966fc3263fad824b1c4c99df832672e

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: milestone-proposed
Review: https://review.openstack.org/25737

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/grizzly
Review: https://review.openstack.org/27392

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/27392
Committed: http://github.com/openstack/nova/commit/f7bcd951138b4d7dc37c567d6725353782376659
Submitter: Jenkins
Branch: stable/grizzly

commit f7bcd951138b4d7dc37c567d6725353782376659
Author: Aaron Rosen <email address hidden>
Date: Wed Mar 27 12:51:33 2013 -0700

    get context from req rather than getting a new admin context

    This patch removes the call to context.get_admin_context() and instead
    obtains the context from the req object. Allowing the context.auth_token
    to be reused.

    Fixes bug 1161081

    Conflicts:

     nova/api/openstack/compute/contrib/security_groups.py

    Change-Id: I3b104edc2966fc3263fad824b1c4c99df832672e
    (cherry picked from commit 1ea02dbcf5ae3e822378a580ac1e54467f664be7)