)]}'
{"specs/juno/rbd-clone-image-handler.rst":[{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"e1278fdea757db10820525ae4b366067c57f9c8d","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"Extract RBD specific utility code into a new file, align its structure and"},{"line_number":50,"context_line":"provided functionality in line with similar code in Cinder. This includes the"},{"line_number":51,"context_line":"volume cleanup code that should be converted from rbd CLI to using the RBD"},{"line_number":52,"context_line":"library."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Add utility functions to support cloning, including checks whether image exists"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7adec928_d02a8c2b","line":51,"updated":"2014-05-09 21:20:42.000000000","message":"Can this code be pulled out into an oslo library like the vmware one that is used by both cinder and nova?","commit_id":"f3d17e50609eb0f4690f52e2cb62e7978db42af2"},{"author":{"_account_id":8787,"name":"Dmitry Borodaenko","email":"angdraug@gmail.com","username":"angdraug"},"change_message_id":"80010ae3f812edc86f1fb4f1d0dc184f5975578e","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"Extract RBD specific utility code into a new file, align its structure and"},{"line_number":50,"context_line":"provided functionality in line with similar code in Cinder. This includes the"},{"line_number":51,"context_line":"volume cleanup code that should be converted from rbd CLI to using the RBD"},{"line_number":52,"context_line":"library."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Add utility functions to support cloning, including checks whether image exists"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7adec928_10695425","line":51,"in_reply_to":"7adec928_d02a8c2b","updated":"2014-05-09 22:01:23.000000000","message":"Yes, extracting this code into an oslo library would be the next logical step after implementing this blueprint.","commit_id":"f3d17e50609eb0f4690f52e2cb62e7978db42af2"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"e1278fdea757db10820525ae4b366067c57f9c8d","unresolved":false,"context_lines":[{"line_number":120,"context_line":"there will be a significant performance improvement."},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"In other use cases, libvirt driver will introduce one more API call to Glance"},{"line_number":123,"context_line":"to retrieve a list of image locations. The performance impact of that call is"},{"line_number":124,"context_line":"insignificant compared to the time and resources it takes to fetch a full image"},{"line_number":125,"context_line":"from Glance in these use cases."},{"line_number":126,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"7adec928_90d72423","line":123,"updated":"2014-05-09 21:20:42.000000000","message":"An additional API call to glance will only happen if RBD is being used right? (-1 for clarification on this point)","commit_id":"f3d17e50609eb0f4690f52e2cb62e7978db42af2"},{"author":{"_account_id":8787,"name":"Dmitry Borodaenko","email":"angdraug@gmail.com","username":"angdraug"},"change_message_id":"80010ae3f812edc86f1fb4f1d0dc184f5975578e","unresolved":false,"context_lines":[{"line_number":120,"context_line":"there will be a significant performance improvement."},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"In other use cases, libvirt driver will introduce one more API call to Glance"},{"line_number":123,"context_line":"to retrieve a list of image locations. The performance impact of that call is"},{"line_number":124,"context_line":"insignificant compared to the time and resources it takes to fetch a full image"},{"line_number":125,"context_line":"from Glance in these use cases."},{"line_number":126,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"7adec928_eb3a39cc","line":123,"in_reply_to":"7adec928_90d72423","updated":"2014-05-09 22:01:23.000000000","message":"Yes, should only happen with RBD (not the case with the current implementation but easy enough to address). Spec updated.","commit_id":"f3d17e50609eb0f4690f52e2cb62e7978db42af2"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"e1278fdea757db10820525ae4b366067c57f9c8d","unresolved":false,"context_lines":[{"line_number":171,"context_line":"be covered."},{"line_number":172,"context_line":""},{"line_number":173,"context_line":"That said, due to insufficient coverage of Ceph related use cases in OpenStack"},{"line_number":174,"context_line":"CI, testing this change will have to rely on external CI and on manual testing"},{"line_number":175,"context_line":"by interested parties."},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"Fuel CI includes a suite of tests for OpenStack deployments with Ceph:"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7adec928_70d45820","line":174,"updated":"2014-05-09 21:20:42.000000000","message":"I would really like to see a ceph CI system coem online to help make sure this blueprint works etc.","commit_id":"f3d17e50609eb0f4690f52e2cb62e7978db42af2"},{"author":{"_account_id":8787,"name":"Dmitry Borodaenko","email":"angdraug@gmail.com","username":"angdraug"},"change_message_id":"80010ae3f812edc86f1fb4f1d0dc184f5975578e","unresolved":false,"context_lines":[{"line_number":171,"context_line":"be covered."},{"line_number":172,"context_line":""},{"line_number":173,"context_line":"That said, due to insufficient coverage of Ceph related use cases in OpenStack"},{"line_number":174,"context_line":"CI, testing this change will have to rely on external CI and on manual testing"},{"line_number":175,"context_line":"by interested parties."},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"Fuel CI includes a suite of tests for OpenStack deployments with Ceph:"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7adec928_ebaf99e1","line":174,"in_reply_to":"7adec928_70d45820","updated":"2014-05-09 22:01:23.000000000","message":"Fuel CI covers Ceph use cases. At the moment it runs only with packages built from stable/* based branches, although it includes the patches described in this blueprint. I think making it verify review.openstack.com commits targeted for master branches would be a worthy goal and would help with Ceph CI in particular.","commit_id":"f3d17e50609eb0f4690f52e2cb62e7978db42af2"},{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"d148daefde4845db57dec5225f28df9b52ab7478","unresolved":false,"context_lines":[{"line_number":55,"context_line":"and whether it can be cloned."},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"Add direct_fetch() method to nova.virt.libvirt.imagebackend, make its"},{"line_number":58,"context_line":"implementation in the Rbd subclass try to clone the image when possible."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Extend fetch_to_raw() in nova.virt.images to try direct_fetch() when a new"},{"line_number":61,"context_line":"optional backend parameter is passed. Make the libvirt driver pass the backend"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1ae5cdf2_0d33fdf7","line":58,"updated":"2014-06-24 15:55:39.000000000","message":"Do Nova need to know glance backend type? How to export glance backend information to nova? Can you add some detail about these?","commit_id":"7cb0092bf25672c745b988c8d4f1607042a64d3f"},{"author":{"_account_id":8787,"name":"Dmitry Borodaenko","email":"angdraug@gmail.com","username":"angdraug"},"change_message_id":"9fa3ade94bba8fdfdd5c08624462d46c8016f2d8","unresolved":false,"context_lines":[{"line_number":55,"context_line":"and whether it can be cloned."},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"Add direct_fetch() method to nova.virt.libvirt.imagebackend, make its"},{"line_number":58,"context_line":"implementation in the Rbd subclass try to clone the image when possible."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Extend fetch_to_raw() in nova.virt.images to try direct_fetch() when a new"},{"line_number":61,"context_line":"optional backend parameter is passed. Make the libvirt driver pass the backend"}],"source_content_type":"text/x-rst","patch_set":5,"id":"baada198_40b150e0","line":58,"in_reply_to":"1ae5cdf2_0d33fdf7","updated":"2014-07-01 17:44:14.000000000","message":"Done","commit_id":"7cb0092bf25672c745b988c8d4f1607042a64d3f"},{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"d148daefde4845db57dec5225f28df9b52ab7478","unresolved":false,"context_lines":[{"line_number":69,"context_line":"------------"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"An alternative implementation based on the image-multiple-location blueprint"},{"line_number":72,"context_line":"(https://blueprints.launchpad.net/nova/+spec/image-multiple-location) was tried"},{"line_number":73,"context_line":"in Icehouse. It was ultimately reverted, which can be attributed to a sum of"},{"line_number":74,"context_line":"multiple reasons:"},{"line_number":75,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"1ae5cdf2_8b9387a3","line":72,"updated":"2014-06-24 15:55:39.000000000","message":"Http page not found.","commit_id":"7cb0092bf25672c745b988c8d4f1607042a64d3f"},{"author":{"_account_id":8787,"name":"Dmitry Borodaenko","email":"angdraug@gmail.com","username":"angdraug"},"change_message_id":"9fa3ade94bba8fdfdd5c08624462d46c8016f2d8","unresolved":false,"context_lines":[{"line_number":69,"context_line":"------------"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"An alternative implementation based on the image-multiple-location blueprint"},{"line_number":72,"context_line":"(https://blueprints.launchpad.net/nova/+spec/image-multiple-location) was tried"},{"line_number":73,"context_line":"in Icehouse. It was ultimately reverted, which can be attributed to a sum of"},{"line_number":74,"context_line":"multiple reasons:"},{"line_number":75,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"baada198_80ca884d","line":72,"in_reply_to":"1ae5cdf2_8b9387a3","updated":"2014-07-01 17:44:14.000000000","message":"Done","commit_id":"7cb0092bf25672c745b988c8d4f1607042a64d3f"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"80d0c511abc3c6cb1fc548c37abca7c61c94e403","unresolved":false,"context_lines":[{"line_number":176,"context_line":""},{"line_number":177,"context_line":""},{"line_number":178,"context_line":"Testing"},{"line_number":179,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"This is a non-functional change with no impact on the test cases that need to"},{"line_number":182,"context_line":"be covered."}],"source_content_type":"text/x-rst","patch_set":6,"id":"baada198_d3e04ecf","line":179,"updated":"2014-07-05 21:51:47.000000000","message":"This is a pretty weak testing story. Are there any plans to set up a third party CI system to test ceph?","commit_id":"dc69ef325e080fc4000c2abbb0c37e023f837248"},{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"cc820b2969e905b719cca37a21fa1206848ecf0c","unresolved":false,"context_lines":[{"line_number":176,"context_line":""},{"line_number":177,"context_line":""},{"line_number":178,"context_line":"Testing"},{"line_number":179,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"This is a non-functional change with no impact on the test cases that need to"},{"line_number":182,"context_line":"be covered."}],"source_content_type":"text/x-rst","patch_set":6,"id":"baada198_ed75d833","line":179,"in_reply_to":"baada198_543688e8","updated":"2014-07-17 13:25:30.000000000","message":"I\u0027ve been looking into getting ceph running in our existing infrastructure.  The devstack patch seems close to being ready to merge:\n\n    https://review.openstack.org/#/c/107472/2\n\nOnce that\u0027s in, I\u0027m going to set up some jobs that use ceph for Nova, Glance, and Cinder.","commit_id":"dc69ef325e080fc4000c2abbb0c37e023f837248"},{"author":{"_account_id":8787,"name":"Dmitry Borodaenko","email":"angdraug@gmail.com","username":"angdraug"},"change_message_id":"f035e735d5018b023e7d2a8d7da00f055fbd851e","unresolved":false,"context_lines":[{"line_number":176,"context_line":""},{"line_number":177,"context_line":""},{"line_number":178,"context_line":"Testing"},{"line_number":179,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"This is a non-functional change with no impact on the test cases that need to"},{"line_number":182,"context_line":"be covered."}],"source_content_type":"text/x-rst","patch_set":6,"id":"baada198_543688e8","line":179,"in_reply_to":"baada198_d3e04ecf","updated":"2014-07-09 02:51:24.000000000","message":"Unfortunately I have to work with what I have, and all I have is plans for Fuel to support deploying OpenStack from git master branch:\nhttps://blueprints.launchpad.net/fuel/+spec/openstack-from-master\n\nThis is a pre-requisite to running Fuel integration tests against OpenStack gerrit.\n\nOn the flip side, this particular set of changes (Havana and Icehouse versions as available via GitHub link in References section) has been included in Mirantis OpenStack since version 4.0 (released in December 2013), and has been subject to the Fuel CI test set linked below since then.\n\nSo the biggest testing risk is not in the changes themselves but in porting them to Juno. To mitigate that, I\u0027ve been continuously tracking Juno in the branch linked in References.","commit_id":"dc69ef325e080fc4000c2abbb0c37e023f837248"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"dc3427f4ef534f273ed55c2f2b82d8cb36bf0e45","unresolved":false,"context_lines":[{"line_number":176,"context_line":""},{"line_number":177,"context_line":""},{"line_number":178,"context_line":"Testing"},{"line_number":179,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"This is a non-functional change with no impact on the test cases that need to"},{"line_number":182,"context_line":"be covered."}],"source_content_type":"text/x-rst","patch_set":6,"id":"baada198_b74ddbc6","line":179,"in_reply_to":"baada198_ed75d833","updated":"2014-07-17 19:00:23.000000000","message":"There are failures, looks like mostly volume failures, there could be other race bugs in there too, but definitely looks like some problems with encrypted volume tests:\n\nhttp://logs.openstack.org/72/107472/5/check/check-tempest-dsvm-full/7dd2d54/logs/screen-n-cpu.txt.gz?level\u003dTRACE","commit_id":"dc69ef325e080fc4000c2abbb0c37e023f837248"}]}