API call in context processor called multiple times

[Gabriel Hurley]

The call to tenant_list_for_token() in the horizon context processor is called numerous times for each request. This is because the context is re-generated for each template or template fragment which takes context that is used to render the response for the request. This results in an average of 5-9 calls to the API per request, which is completely unnecessary.

Moving the call to the middleware layer and storing the response on the user object will work far better.

[Jay Pipes (jaypipes)]

++. I've noticed this.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/3070
Committed: http://github.com/openstack/horizon/commit/a77ed5e534e9ccb9f31baa20480b62fdc8f2430b
Submitter: Jenkins
Branch: master

commit a77ed5e534e9ccb9f31baa20480b62fdc8f2430b
Author: Gabriel Hurley <email address hidden>
Date: Mon Jan 16 10:05:11 2012 -0800

    Moved authorized_tenants retrieval to middleware.

    Fixes bug 917263. By moving the tenant API call from the context
    processor to the middleware the API call is no longer made
    multiple times per request/response cycle.

    Additionally, there are various PEP8 fixes included, with the
    addition of one "ignore" flag to the PEP8 runner (related to
    github issue #34 for pep8.py).

    Change-Id: I5c755dfe381b1c38dbeeb99eb2b7ed9172d16f86