glance_api_servers=None should use the catalogue

[Duncan Thomas]

We have the config option glance_api_servers to give a list of glance API URLs. This information, for most installs, is in the catalogue, so we should be able to set the to none and use the catalogue value.

A similar change went in recently to cinder-backup to use the catalogue for find the swift endpoint.

[John Griffith (john-griffith)]

Can you please add the specifics here around where you want these changed etc. That way somebody can easily take a look at the triaged list and pick up work on it.

[Sean McGinnis (sean-mcginnis)]

Duncan, can you point me at where this was change in cinder-backup? I am willing to fix this, but not clear to me where the impact is.

[Sean McGinnis (sean-mcginnis)]

Automatically unassigning due to inactivity.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/252476
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=3aa3238cd0323ea8fc4cd4a2e5c1f990d87bfc04
Submitter: Jenkins
Branch: master

commit 3aa3238cd0323ea8fc4cd4a2e5c1f990d87bfc04
Author: Nate Potter <email address hidden>
Date: Wed Dec 2 15:54:08 2015 +0000

    Use the catalog to retrieve glance_api_servers

    Currently the config option glance_api_servers must be
    set to give the correct list of glance API URLs. This
    patch enables the use of the catalog to find the glance
    endpoint when this config option is set to None, and
    changes its default to None as well.

    Closes-bug: #1371644
    DocImpact: The config options glance_host and glance_port
    have been removed, the default value for glance_api_servers
    has been changed to None, and a new config option
    glance_catalog_info has been added to glance_opts.
    Change-Id: I261da528eb008f6b2520aef043c06697bd5ce352

[Doug Hellmann (doug-hellmann)]

This issue was fixed in the openstack/cinder 9.0.0.0b1 development milestone.