<p dir="ltr">I don't think you need an entry per driver, you need an entry per connection type - iSCSI, FC, DRDB, CEPH being the ones I can think of off the top of my head</p>
<div class="gmail_quote">On 10 Jun 2015 16:57, "Matt Riedemann" <<a href="mailto:mriedem@linux.vnet.ibm.com">mriedem@linux.vnet.ibm.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">While investigating/discussing bug 1463525 [1] I remembered how little I know about what can actually come out of the connection_info dict returned from the os-initialize_connection cinder API call.<br>
<br>
So we added some debug logging in nova and I remembered that there are potentially credentials (auth_password) stored in connection_info, so we have a bug to clean that up in Nova [2].<br>
<br>
The plan is to model connection_info using objects where we have a parent object BdmConnectionInfo containing the common keys, like 'driver_volume_type' and 'data', and then child objects for the vendor-specific connection_info objects, like RbdBdmConnectionInfo, ISCSIBdmConnectionInfo, etc.<br>
<br>
The problem I have right now is knowing what can all be in there, since there are a ton of vendor drivers in Cinder.<br>
<br>
Is anyone aware of a wiki page or devref or anything that documents what can be in that wild west connection_info dict?  If not, the first thing I was going to do was start documenting that - but where?  It seems it should really be modeled in Cinder since it is part of the API contract and if a vendor driver were to say rename or drop a key from the connection_info they were returning in os-initialize_connection it would be a backwards incompatible change.<br>
<br>
Is devref best for this with a listing for each vendor driver?  At least then it would be versioned with the code and updates could be made as new keys are added to connection_info or new drivers are added to Cinder.<br>
<br>
I'm looking for any advice here in how to get started since I don't primarily work on Cinder and don't have a full history here.<br>
<br>
[1] <a href="https://bugs.launchpad.net/cinder/+bug/1463525" target="_blank">https://bugs.launchpad.net/cinder/+bug/1463525</a><br>
[2] <a href="https://bugs.launchpad.net/nova/+bug/1321785" target="_blank">https://bugs.launchpad.net/nova/+bug/1321785</a><br>
<br>
-- <br>
<br>
Thanks,<br>
<br>
Matt Riedemann<br>
<br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div>