<div>I too would prefer option 2.  Would rather do the pack ports than remove the functionality.<br><br>Jay</div><br><div class="gmail_quote"><div dir="ltr">On Wed, Jun 24, 2015 at 9:40 AM Gorka Eguileor <<a href="mailto:geguileo@redhat.com">geguileo@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Jun 23, 2015 at 08:49:55AM -0700, Mike Perez wrote:<br>
> There was a bug raised [1] from some large deployments that the Cinder<br>
> client 1.2.0 and beyond is not working because of version discovery.<br>
> Unfortunately it's not taking into account of deployments that have a<br>
> proxy.<br>
<br>
A little bit unrelated, but volume pagination in Cinder client is also<br>
broken due to Version Discovery:<br>
<a href="https://bugs.launchpad.net/python-cinderclient/+bug/1453755" rel="noreferrer" target="_blank">https://bugs.launchpad.net/python-cinderclient/+bug/1453755</a><br>
<br>
><br>
> Cinder client asks Keystone to find a publicURL based on a version.<br>
> Keystone will gather data from the service catalog and ask Cinder for<br>
> a list of the public endpoints and compare. For the proxy cases,<br>
> Cinder is giving internal URLs back to the proxy and Keystone ends up<br>
> using that instead of the publicURL in the service catalog. As a<br>
> result, clients usually won't be able to use the internal URL and<br>
> rightfully so.<br>
><br>
> This is all correctly setup on the deployer's side, this an issue with<br>
> the server side code of Cinder.<br>
><br>
> There is a patch that allows the deployer to specify a configuration<br>
> option public_endpoint [2] which was introduced in a patch in Kilo<br>
> [3]. The problem though is we can't expect people to already be<br>
> running Kilo to take advantage of this, and it leaves deployers<br>
> running stable releases of Juno in the dark with clients upgrading and<br>
> using the latest.<br>
><br>
> Two options:<br>
><br>
> 1) Revert version discovery which was introduced in Kilo for Cinder client.<br>
><br>
> 2) Grant exception on backporting [4] a patch that helps with this<br>
> problem, and introduces a config option that does not change default<br>
> behavior. I'm also not sure if this should be considered for Icehouse.<br>
<br>
+1 to option 2 and I wouldn't be totally against considering it for<br>
Icehouse.<br>
<br>
Cheers,<br>
Gorka.<br>
><br>
><br>
> [1] - <a href="https://launchpad.net/bugs/1464160" rel="noreferrer" target="_blank">https://launchpad.net/bugs/1464160</a><br>
> [2] - <a href="http://docs.openstack.org/kilo/config-reference/content/cinder-conf-changes-kilo.html" rel="noreferrer" target="_blank">http://docs.openstack.org/kilo/config-reference/content/cinder-conf-changes-kilo.html</a><br>
> [3] - <a href="https://review.openstack.org/#/c/159374/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/159374/</a><br>
> [4] - <a href="https://review.openstack.org/#/c/194719/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/194719/</a><br>
><br>
> --<br>
> Mike Perez<br>
><br>
> __________________________________________________________________________<br>
> OpenStack Development Mailing List (not for usage questions)<br>
> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div>