)]}'
{"specs/liberty/approved/emc-sdc-libvirt-volume-driver.rst":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"9e5ff6827a956b2cb26b5a0cf1e6442c9f46ce76","unresolved":false,"context_lines":[{"line_number":155,"context_line":"Testing"},{"line_number":156,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"A 3rd party CI system is being set up to test the Cinder driver."},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"Documentation Impact"},{"line_number":161,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1af86dd1_cde7413f","line":158,"updated":"2015-06-12 16:11:57.000000000","message":"Can you please clarify you will run this against the nova tree as well as cinder?","commit_id":"7583c0f01c38a36597c7b5db2f4b14da12305f33"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"95bf0a38e266764e4c69c3ca56c99898b976538a","unresolved":false,"context_lines":[{"line_number":155,"context_line":"Testing"},{"line_number":156,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"A 3rd party CI system is being set up to test the Cinder driver."},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"Documentation Impact"},{"line_number":161,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1af86dd1_21b01e43","line":158,"in_reply_to":"1af86dd1_cde7413f","updated":"2015-06-12 17:41:55.000000000","message":"Done","commit_id":"7583c0f01c38a36597c7b5db2f4b14da12305f33"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"c1cbbcc2149404134e305ef09f64a095955b4176","unresolved":false,"context_lines":[{"line_number":150,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"Cinder blueprint for EMC ScaleIO driver, already approved for Liberty-1:"},{"line_number":153,"context_line":"https://blueprints.launchpad.net/cinder/+spec/scaleio-cinder-volume-driver"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"Testing"},{"line_number":156,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa32b979_8d6b7485","line":153,"updated":"2015-06-19 17:28:24.000000000","message":"So the driver is in cinder:\n\nhttps://review.openstack.org/#/c/183762/\n\nBut it also needs to be in os-brick, right?  I think this should depend on blueprint use-os-brick-library so that we possibly don\u0027t even need the volume driver in nova, we\u0027d just have the libvirt driver registration but the actual connect/disconnect code would be in the volume driver in os-brick and nova would just interface with brick, rather than have a duplicated driver in nova and os-brick (and cinder).\n\nThe os-brick spec was approved for liberty, the WIP code is here:\n\nhttps://review.openstack.org/#/c/175569/","commit_id":"f9db5c595f5801b46580d0344a6a7760402a09c2"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"54d1741a94045cc9465cd9e38fd50133c2d3341e","unresolved":false,"context_lines":[{"line_number":150,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"Cinder blueprint for EMC ScaleIO driver, already approved for Liberty-1:"},{"line_number":153,"context_line":"https://blueprints.launchpad.net/cinder/+spec/scaleio-cinder-volume-driver"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"Testing"},{"line_number":156,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa32b979_edcd605e","line":153,"in_reply_to":"fa32b979_8d6b7485","updated":"2015-06-19 17:29:56.000000000","message":"Also, be aware of blueprint consolidate-libvirt-fs-volume-drivers for implementation dependencies, I\u0027m working on refactoring the libvirt volume drivers right now, so this would need to align with that even if this isn\u0027t a file system type volume driver - that blueprint has morphed into a generic refactor + the fs-style driver consolidation.","commit_id":"f9db5c595f5801b46580d0344a6a7760402a09c2"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"de82591fe4608b2ee24218c73e64683719bf4231","unresolved":false,"context_lines":[{"line_number":150,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"Cinder blueprint for EMC ScaleIO driver, already approved for Liberty-1:"},{"line_number":153,"context_line":"https://blueprints.launchpad.net/cinder/+spec/scaleio-cinder-volume-driver"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"Testing"},{"line_number":156,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa32b979_d0a2739a","line":153,"in_reply_to":"fa32b979_8d6b7485","updated":"2015-06-19 17:57:35.000000000","message":"The merged Cinder driver doesn\u0027t have a connector driver inside os-brick yet. It has code inside Cinder driver that uses os-brick to connect volumes. I submitted a WIP os-brick connector driver: https://review.openstack.org/#/c/185835/.   I\u0027ll update this spec to reference that.","commit_id":"f9db5c595f5801b46580d0344a6a7760402a09c2"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"de82591fe4608b2ee24218c73e64683719bf4231","unresolved":false,"context_lines":[{"line_number":150,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"Cinder blueprint for EMC ScaleIO driver, already approved for Liberty-1:"},{"line_number":153,"context_line":"https://blueprints.launchpad.net/cinder/+spec/scaleio-cinder-volume-driver"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"Testing"},{"line_number":156,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa32b979_10af7b7e","line":153,"in_reply_to":"fa32b979_edcd605e","updated":"2015-06-19 17:57:35.000000000","message":"Sure, thanks for the note.","commit_id":"f9db5c595f5801b46580d0344a6a7760402a09c2"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"34130e2378f6de141e17cf9011e76aa19daf0ad0","unresolved":false,"context_lines":[{"line_number":45,"context_line":"Proposed change"},{"line_number":46,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"A libvirt volume driver for the ScaleIO protocol used by EMC ScaleIO will be"},{"line_number":49,"context_line":"added to the nova/virt/libvirt directory."},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"A section will be added to the list of libvirt volume drivers in the file"}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa32b979_ec73b77e","line":48,"updated":"2015-06-23 14:21:49.000000000","message":"This is written very specifically to how this would be done in nova the \u0027old way\u0027 before os-brick integration.  I think your spec is getting caught up in the middle of that transition, but really this should be a cinder-spec for os-brick.\n\nSince the cinder driver code was already merged and there is a change proposed for os-brick, I\u0027m not sure we even need a nova spec anymore since if you get your change into os-brick and we get https://review.openstack.org/#/c/175569/ into nova, then your code is automatically picked up.","commit_id":"cce0325cea593197da393ed212556a7e70903e2e"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"bbf34d07607cc47f70d2e647be17d24ffeea781c","unresolved":false,"context_lines":[{"line_number":45,"context_line":"Proposed change"},{"line_number":46,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"A libvirt volume driver for the ScaleIO protocol used by EMC ScaleIO will be"},{"line_number":49,"context_line":"added to the nova/virt/libvirt directory."},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"A section will be added to the list of libvirt volume drivers in the file"}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa32b979_87cd2433","line":48,"in_reply_to":"fa32b979_ec73b77e","updated":"2015-06-23 14:44:02.000000000","message":"Sure.  It will be much simpler if Nova just uses os-brick.\n\nI\u0027ve submitted a patch depending on the os-brick patch:\n\nhttps://review.openstack.org/#/c/194454/","commit_id":"cce0325cea593197da393ed212556a7e70903e2e"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"34130e2378f6de141e17cf9011e76aa19daf0ad0","unresolved":false,"context_lines":[{"line_number":64,"context_line":""},{"line_number":65,"context_line":"To use this driver, the user has to install additional component:"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"* The SDC Linux driver"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"This software can be downloaded from the following web site for free:"},{"line_number":70,"context_line":"http://www.emc.com/storage/scaleio/index.htm"}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa32b979_4c118303","line":67,"updated":"2015-06-23 14:21:49.000000000","message":"I assume there aren\u0027t any distros that package this up today?","commit_id":"cce0325cea593197da393ed212556a7e70903e2e"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"bbf34d07607cc47f70d2e647be17d24ffeea781c","unresolved":false,"context_lines":[{"line_number":64,"context_line":""},{"line_number":65,"context_line":"To use this driver, the user has to install additional component:"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"* The SDC Linux driver"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"This software can be downloaded from the following web site for free:"},{"line_number":70,"context_line":"http://www.emc.com/storage/scaleio/index.htm"}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa32b979_2780d8be","line":67,"in_reply_to":"fa32b979_4c118303","updated":"2015-06-23 14:44:02.000000000","message":"It is not in any distro yet.","commit_id":"cce0325cea593197da393ed212556a7e70903e2e"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"34130e2378f6de141e17cf9011e76aa19daf0ad0","unresolved":false,"context_lines":[{"line_number":67,"context_line":"* The SDC Linux driver"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"This software can be downloaded from the following web site for free:"},{"line_number":70,"context_line":"http://www.emc.com/storage/scaleio/index.htm"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"Once these dependencies are met, and this driver is installed, Nova nodes"},{"line_number":73,"context_line":"can mount block volumes from ScaleIO and use them just like any other block"}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa32b979_ec7df71c","line":70,"updated":"2015-06-23 14:21:49.000000000","message":"Looks like it\u0027s actually:\n\nhttp://www.emc.com/products-solutions/trial-software-download/scaleio.htm","commit_id":"cce0325cea593197da393ed212556a7e70903e2e"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"bbf34d07607cc47f70d2e647be17d24ffeea781c","unresolved":false,"context_lines":[{"line_number":67,"context_line":"* The SDC Linux driver"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"This software can be downloaded from the following web site for free:"},{"line_number":70,"context_line":"http://www.emc.com/storage/scaleio/index.htm"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"Once these dependencies are met, and this driver is installed, Nova nodes"},{"line_number":73,"context_line":"can mount block volumes from ScaleIO and use them just like any other block"}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa32b979_e7cbd051","line":70,"in_reply_to":"fa32b979_ec7df71c","updated":"2015-06-23 14:44:02.000000000","message":"Sure, the link on line 70 will lead to your link which is more accurate.","commit_id":"cce0325cea593197da393ed212556a7e70903e2e"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"34130e2378f6de141e17cf9011e76aa19daf0ad0","unresolved":false,"context_lines":[{"line_number":142,"context_line":""},{"line_number":143,"context_line":"* ScaleIO Libvirt volume driver -- this is already developed"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"* An entry in the volume.py file"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"* a filter for scaleio in nova/rootwrap"},{"line_number":148,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa32b979_4114ec4b","line":145,"updated":"2015-06-23 14:21:49.000000000","message":"We don\u0027t need this if it makes it into os-brick right?  I think this spec needs to depend on blueprint use-os-brick-library so you should write it assuming that\u0027s going to be the path forward.","commit_id":"cce0325cea593197da393ed212556a7e70903e2e"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"bbf34d07607cc47f70d2e647be17d24ffeea781c","unresolved":false,"context_lines":[{"line_number":142,"context_line":""},{"line_number":143,"context_line":"* ScaleIO Libvirt volume driver -- this is already developed"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"* An entry in the volume.py file"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"* a filter for scaleio in nova/rootwrap"},{"line_number":148,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa32b979_671340b9","line":145,"in_reply_to":"fa32b979_276bf8ba","updated":"2015-06-23 14:44:02.000000000","message":"I submitted a shim ScaleIOLibvirtVolumeDriver patch depending on hemna\u0027s os-brick patch:\n\nhttps://review.openstack.org/#/c/194454/","commit_id":"cce0325cea593197da393ed212556a7e70903e2e"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"873f77da0740f3c6711ffe2bcfc224e34877fa59","unresolved":false,"context_lines":[{"line_number":142,"context_line":""},{"line_number":143,"context_line":"* ScaleIO Libvirt volume driver -- this is already developed"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"* An entry in the volume.py file"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"* a filter for scaleio in nova/rootwrap"},{"line_number":148,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa32b979_276bf8ba","line":145,"in_reply_to":"fa32b979_4114ec4b","updated":"2015-06-23 14:32:03.000000000","message":"Actually, I\u0027m trying to figure that out with hemna.  For example, in the os-brick nova integration change:\n\nhttps://review.openstack.org/#/c/175569/\n\nMost of the LibvirtFibreChannelVolumeDriver class is removed but the get_config method is left intact, so I\u0027m not sure if that will be the story for all other volume backends that are added even after the os-brick integration.  If so, then we will have nova changes.","commit_id":"cce0325cea593197da393ed212556a7e70903e2e"}]}