python-nova: not region aware when used with quantum

[Stephen Gran]

Hi,

When used with quantum, python-nova does not pass in any region information when constructing a quantum-client, making the compute services on my multi-region setup fail when multiple endpoints are returned by keystone.

The problem is that in network/quantumv2/__init__.py, _get_auth_token() a quantumclient.client.HTTPClient is created without a region_name argument.

In order to make this work, I had to edit nova/network/quantumv2/api.py and add a new parameter to quantum_opts:

    cfg.StrOpt('quantum_region_name',
               help='region name for connecting to quantum in admin context'),

and then pass that to the httpclient constructor in nova/network/quantumv2/__init__.py with:

            timeout=FLAGS.quantum_url_timeout,
            region_name=FLAGS.quantum_region_name,
            auth_strategy=FLAGS.quantum_auth_strategy)

and finally set quantum_region_name in nova.conf.

There may be a more elegant way of achieving all this.

[Stephen Gran (sgran)]

I've added a patch and sent a pull request:
https://github.com/sgran/nova/commit/c2cc8e21b6d35a2f86cbcdb22ea5f3baf82708d5

[OpenStack Infra (hudson-openstack)]

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

[Davanum Srinivas (DIMS) (dims-v)]

Stephen, I can help with this bug, next time, can you please submit via gerritt? [1]

[1] http://wiki.openstack.org/HowToContribute#If_you.27re_a_developer.2C_start_here:

thanks,
dims

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/20573
Committed: http://github.com/openstack/nova/commit/30c68f47e9afb323761c24cb2417e11a404f64ce
Submitter: Jenkins
Branch: master

commit 30c68f47e9afb323761c24cb2417e11a404f64ce
Author: Davanum Srinivas <email address hidden>
Date: Sun Jan 27 21:41:10 2013 -0500

    Add Region name to quantum client

    Looks like multi-region setups need the region name

    Fixes LP# 1081037

    DocImpact

    Change-Id: Id8d5b1e1adef75a93df9a3527fd6da463fed0d09