In response to Jamie's comment I've abandoned my patch in favor of this one:
https://review.openstack.org/#/c/388232 It simply removes the ClientCache from the openstackclient code and replaces it with property(). On 18/10/16 17:10, Adrian Turjak wrote: > > On 18/10/16 15:52, Jamie Lennox wrote: >> >> A comment from the cheap seats, almost all clients are using a >> keystoneauth1 session at this point and that's where your >> authentication information is being cached. There is essentially no >> cost to creating a client with an existing session as auth happens on >> demand. >> >> The region_name is not part of the authentication request, it's used >> to lookup the endpoint and so is passed to Client creation. >> >> Given this maybe there is no longer any value in a ClientCache? It >> was mostly useful to prevent clients doing dumb and share auth >> amongst them. So long as the session/auth is created and saved once, >> a client can be created per use/request with this information >> (including region) with no real performance impact. >> >> Jamie >> > Getting rid of the cache would solve my problem, and considering it's > all one shared session it shouldn't cause any problems. > > Updating region_name would still be a problem for interactive mode, > although really that is a problem that is present right now anyway. > > > __________________________________________________________________________ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
__________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev