On Wed, Jan 27, 2016 at 1:01 PM, Matt Riedemann <mriedem at linux.vnet.ibm.com> wrote: <snip> > I've replied on https://review.openstack.org/#/c/266095/ and the related > cinder change https://review.openstack.org/#/c/272899/ which are adding a > new key to the volume connector dict being passed around between nova and > cinder, which is not ideal. > > I'd really like to see us start modeling the volume connector with > versioned objects so we can (1) tell what's actually in this mystery > connector dict in the nova virt driver interface and (2) handle version > compat with adding new keys to it. > I agree with you. Actually, I think it would be more correct to have Cinder store it, and not pass it at all to terminate_connection(). -- *Avishay Traeger, PhD* *System Architect* Mobile: +972 54 447 1475 E-mail: avishay at stratoscale.com Web <http://www.stratoscale.com/> | Blog <http://www.stratoscale.com/blog/> | Twitter <https://twitter.com/Stratoscale> | Google+ <https://plus.google.com/u/1/b/108421603458396133912/108421603458396133912/posts> | Linkedin <https://www.linkedin.com/company/stratoscale> -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20160127/d8840fea/attachment.html>