)]}'
{"specs/rocky/approved/powervm-virt-rocky.rst":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"c5a47149e6fb67d3e92132db741e4cfc4c03b66f","unresolved":false,"context_lines":[{"line_number":8,"context_line":"PowerVM Driver Integration - Queens"},{"line_number":9,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"`\u003chttps://blueprints.launchpad.net/nova/+spec/powervm-nova-it-compute-driver\u003e`_"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The PowerVM driver provides OpenStack enablement for AIX, IBM i and Linux"},{"line_number":14,"context_line":"virtual machines running on a PowerVM hypervisor. It has been developed"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3fa0c359_9da9d891","line":11,"range":{"start_line":11,"start_character":2,"end_line":11,"end_character":76},"updated":"2018-02-15 20:48:42.000000000","message":"fix","commit_id":"5d8474354a8037b9ccfd687e9f49ce42a66e25ab"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"c5a47149e6fb67d3e92132db741e4cfc4c03b66f","unresolved":false,"context_lines":[{"line_number":72,"context_line":"This does not bring the entirety of the PowerVM out-of-tree driver in-tree."},{"line_number":73,"context_line":"There are other features that will not be implemented in-tree this release."},{"line_number":74,"context_line":"Implementing the PowerVM driver in-tree is a long-term process that will"},{"line_number":75,"context_line":"continue through multiple OpenStack releases. "},{"line_number":76,"context_line":""},{"line_number":77,"context_line":".. _`vSCSI`: https://www.ibm.com/support/knowledgecenter/en/8284-21A/p8hat/p8hat_virtualscsi.htm"},{"line_number":78,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3fa0c359_7dcc0c61","line":75,"range":{"start_line":75,"start_character":45,"end_line":75,"end_character":46},"updated":"2018-02-15 20:48:42.000000000","message":"gasp","commit_id":"5d8474354a8037b9ccfd687e9f49ce42a66e25ab"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"7d496f3c9f9c8f3c97284e4c0a5b6dfa514cb0f5","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":"* A user should be able to attach and detach cinder volumes."},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"* A user should be able to boot using local disks."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"* A user should be able to snapshot a server instance."},{"line_number":50,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa0c359_db3e88d9","line":47,"range":{"start_line":47,"start_character":2,"end_line":47,"end_character":50},"updated":"2018-02-15 22:48:51.000000000","message":"How are you creating VMs today if not backed by local storage and you don\u0027t yet support volumes?\n\nIs it all remote storage?","commit_id":"183b9c355f0efdcf6c5daa430d09d13bbaa56905"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"346efeed21a704f7000b70d1b0ab22cbe3ab65fe","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":"* A user should be able to attach and detach cinder volumes."},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"* A user should be able to boot using local disks."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"* A user should be able to snapshot a server instance."},{"line_number":50,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa0c359_d6389d3e","line":47,"range":{"start_line":47,"start_character":2,"end_line":47,"end_character":50},"in_reply_to":"3fa0c359_36b7813d","updated":"2018-02-15 23:22:22.000000000","message":"OK that makes sense now, thanks. I figured it mean ephemeral but didn\u0027t make the connection with shared disk.","commit_id":"183b9c355f0efdcf6c5daa430d09d13bbaa56905"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"a7f47f4a0c8e338c1603f32750d860bae3a5ff99","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":"* A user should be able to attach and detach cinder volumes."},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"* A user should be able to boot using local disks."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"* A user should be able to snapshot a server instance."},{"line_number":50,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa0c359_c7db027b","line":47,"range":{"start_line":47,"start_character":2,"end_line":47,"end_character":50},"in_reply_to":"3fa0c359_d6389d3e","updated":"2018-02-16 17:12:38.000000000","message":"Yep, what efried said. This is noted in the history at the bottom of this spec: https://review.openstack.org/#/c/545111/3/specs/rocky/approved/powervm-virt-rocky.rst@323","commit_id":"183b9c355f0efdcf6c5daa430d09d13bbaa56905"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"d1d37554397bbe5499de9f3453b04778caba8078","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":"* A user should be able to attach and detach cinder volumes."},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"* A user should be able to boot using local disks."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"* A user should be able to snapshot a server instance."},{"line_number":50,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa0c359_36b7813d","line":47,"range":{"start_line":47,"start_character":2,"end_line":47,"end_character":50},"in_reply_to":"3fa0c359_db3e88d9","updated":"2018-02-15 22:56:08.000000000","message":"We\u0027re using PowerVM Shared Storage Pools.  It\u0027s a clustered shared file system brokered through our Virtual I/O Servers.  Supports cool things like thin clones, so deploys are way quick after the first time because you don\u0027t have to transfer the image across the wire.  Landed that via [1].\n\nIn contrast, the \"localdisk\" driver carves logical volumes out of physical drives on the actual host.\n\nBoth are for ephemeral boot.\n\n[1] https://review.openstack.org/#/c/443189/","commit_id":"183b9c355f0efdcf6c5daa430d09d13bbaa56905"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"7d496f3c9f9c8f3c97284e4c0a5b6dfa514cb0f5","unresolved":false,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"* A user should be able to move a server instance."},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"* A user should be able to change a server instance\u0027s properties."},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"Proposed change"},{"line_number":56,"context_line":"\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":"3fa0c359_fb182485","line":53,"range":{"start_line":53,"start_character":2,"end_line":53,"end_character":65},"updated":"2018-02-15 22:48:51.000000000","message":"I\u0027m not sure what this means specifically. I guess you mean resize. Maybe re-word as \"server instance\u0027s physical attributes such as vcpu, memory and/or disk\"?","commit_id":"183b9c355f0efdcf6c5daa430d09d13bbaa56905"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"a7f47f4a0c8e338c1603f32750d860bae3a5ff99","unresolved":false,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"* A user should be able to move a server instance."},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"* A user should be able to change a server instance\u0027s properties."},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"Proposed change"},{"line_number":56,"context_line":"\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":"3fa0c359_e7d87e80","line":53,"range":{"start_line":53,"start_character":2,"end_line":53,"end_character":65},"in_reply_to":"3fa0c359_fb182485","updated":"2018-02-16 17:12:38.000000000","message":"Done","commit_id":"183b9c355f0efdcf6c5daa430d09d13bbaa56905"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"7d496f3c9f9c8f3c97284e4c0a5b6dfa514cb0f5","unresolved":false,"context_lines":[{"line_number":67,"context_line":""},{"line_number":68,"context_line":"* At least one of Cold / Live Migration"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"* At least one of Resize / Rebuild"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"This does not bring the entirety of the PowerVM out-of-tree driver in-tree."},{"line_number":73,"context_line":"There are other features that will not be implemented in-tree this release."}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa0c359_56180d74","line":70,"range":{"start_line":70,"start_character":27,"end_line":70,"end_character":34},"updated":"2018-02-15 22:48:51.000000000","message":"Rebuild is not comparable to resize, the flavor is the same. I guess the point here is you want to change the backing image? And it\u0027s a trade off of whether or not we work toward supporting resize or not?\n\nNote that if we implemented cold migration, you should be able to get resize support pretty easily.\n\nRebuild has a default implementation in the compute manager that relies on the driver\u0027s existing spawn/destroy methods, and the only driver that overrides that is the ironic driver; will the powervm driver need to actually do something special for rebuild?\n\nOtherwise it seems shooting for cold migrate/resize this release if we have to pick options between these 4 things on these 2 lines is the better / safer option.","commit_id":"183b9c355f0efdcf6c5daa430d09d13bbaa56905"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"a7f47f4a0c8e338c1603f32750d860bae3a5ff99","unresolved":false,"context_lines":[{"line_number":67,"context_line":""},{"line_number":68,"context_line":"* At least one of Cold / Live Migration"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"* At least one of Resize / Rebuild"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"This does not bring the entirety of the PowerVM out-of-tree driver in-tree."},{"line_number":73,"context_line":"There are other features that will not be implemented in-tree this release."}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa0c359_7962d2da","line":70,"range":{"start_line":70,"start_character":27,"end_line":70,"end_character":34},"in_reply_to":"3fa0c359_56180d74","updated":"2018-02-16 17:12:38.000000000","message":"oh, it\u0027s called evacuate when you\u0027re rebuilding to another host. Updating this to say we\u0027ll handle cold migrate, evacuate, and resize, with live migrate as a stretch goal.","commit_id":"183b9c355f0efdcf6c5daa430d09d13bbaa56905"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"7d496f3c9f9c8f3c97284e4c0a5b6dfa514cb0f5","unresolved":false,"context_lines":[{"line_number":166,"context_line":""},{"line_number":167,"context_line":"* Add support for instance migration"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"* Add support for changing instance properties"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"* Update PowerVM CI to stay up to date with new in-tree functionality as it is"},{"line_number":172,"context_line":"  implemented."}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa0c359_b620719d","line":169,"range":{"start_line":169,"start_character":0,"end_line":169,"end_character":46},"updated":"2018-02-15 22:48:51.000000000","message":"I\u0027d like to know if we\u0027re focusing on resize or rebuild, or both and if rebuild, why don\u0027t you get that for free?","commit_id":"183b9c355f0efdcf6c5daa430d09d13bbaa56905"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"a7f47f4a0c8e338c1603f32750d860bae3a5ff99","unresolved":false,"context_lines":[{"line_number":166,"context_line":""},{"line_number":167,"context_line":"* Add support for instance migration"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"* Add support for changing instance properties"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"* Update PowerVM CI to stay up to date with new in-tree functionality as it is"},{"line_number":172,"context_line":"  implemented."}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa0c359_bc02883d","line":169,"range":{"start_line":169,"start_character":0,"end_line":169,"end_character":46},"in_reply_to":"3fa0c359_b620719d","updated":"2018-02-16 17:12:38.000000000","message":"Will update per answer above.","commit_id":"183b9c355f0efdcf6c5daa430d09d13bbaa56905"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"7d496f3c9f9c8f3c97284e4c0a5b6dfa514cb0f5","unresolved":false,"context_lines":[{"line_number":168,"context_line":""},{"line_number":169,"context_line":"* Add support for changing instance properties"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"* Update PowerVM CI to stay up to date with new in-tree functionality as it is"},{"line_number":172,"context_line":"  implemented."},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"Dependencies"},{"line_number":175,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa0c359_763ae9cd","line":172,"range":{"start_line":171,"start_character":2,"end_line":172,"end_character":14},"updated":"2018-02-15 22:48:51.000000000","message":"Can you do resize to the same host in your CI?\n\nIf you did live migration, you\u0027d need a multi-host CI job that can test live migration.","commit_id":"183b9c355f0efdcf6c5daa430d09d13bbaa56905"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"a7f47f4a0c8e338c1603f32750d860bae3a5ff99","unresolved":false,"context_lines":[{"line_number":168,"context_line":""},{"line_number":169,"context_line":"* Add support for changing instance properties"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"* Update PowerVM CI to stay up to date with new in-tree functionality as it is"},{"line_number":172,"context_line":"  implemented."},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"Dependencies"},{"line_number":175,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa0c359_dc8434b4","line":172,"range":{"start_line":171,"start_character":2,"end_line":172,"end_character":14},"in_reply_to":"3fa0c359_763ae9cd","updated":"2018-02-16 17:12:38.000000000","message":"Good questions. I\u0027m pretty sure that our CI is currently single-host, but our CI expert (esberglu) is out today, so I\u0027ll have to talk to him about this on Monday.","commit_id":"183b9c355f0efdcf6c5daa430d09d13bbaa56905"},{"author":{"_account_id":16710,"name":"Eric Berglund","email":"esberglu@us.ibm.com","username":"esberglu"},"change_message_id":"fb2f4281c80ce060f761a4692d0423642a4a02d4","unresolved":false,"context_lines":[{"line_number":168,"context_line":""},{"line_number":169,"context_line":"* Add support for changing instance properties"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"* Update PowerVM CI to stay up to date with new in-tree functionality as it is"},{"line_number":172,"context_line":"  implemented."},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"Dependencies"},{"line_number":175,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa0c359_4ff901c6","line":172,"range":{"start_line":171,"start_character":2,"end_line":172,"end_character":14},"in_reply_to":"3fa0c359_dc8434b4","updated":"2018-02-20 17:27:29.000000000","message":"We can do resize to the same host. And yes we will need to update our CI to include a multi-host job to test live migration","commit_id":"183b9c355f0efdcf6c5daa430d09d13bbaa56905"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"1e8e664485b643855214a17bdeaa4e8d2fcdac4d","unresolved":false,"context_lines":[{"line_number":317,"context_line":""},{"line_number":318,"context_line":"* February 2018: Queens Release - `Phase 2`_ implemented including"},{"line_number":319,"context_line":"    * Config drive"},{"line_number":320,"context_line":"    * Open vSwitch networking"},{"line_number":321,"context_line":"    * PowerVM Shared Ethernet Adapter networking"},{"line_number":322,"context_line":""},{"line_number":323,"context_line":".. _`First commit`: https://github.com/openstack/nova-powervm/commit/095e1c183baf4f9083d6b0d363818be21f64f992"},{"line_number":324,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"3fa0c359_5328c395","line":321,"range":{"start_line":320,"start_character":0,"end_line":321,"end_character":48},"updated":"2018-02-20 22:30:49.000000000","message":"I just realized that this only included support during spawn, and not hotplugging interfaces. That should be an easy add, and I think that is important, so I\u0027ll push a new patch set adding that to this spec.","commit_id":"6b7da04e62e0e7488258476bc3c2845cbbc5d36f"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"2f374bffc4706de05123799b9aaa440679d3453f","unresolved":false,"context_lines":[{"line_number":63,"context_line":"* Boot via host local disk"},{"line_number":64,"context_line":"* Snapshot"},{"line_number":65,"context_line":"* Cold Migrate"},{"line_number":66,"context_line":"* Evacuate"},{"line_number":67,"context_line":"* Resize"},{"line_number":68,"context_line":"* Live Migrate (stretch goal)"},{"line_number":69,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"df7087c5_fe8c7036","line":66,"range":{"start_line":66,"start_character":2,"end_line":66,"end_character":10},"updated":"2018-03-20 20:07:30.000000000","message":"This is weird, as noted in PS3, because evacuate (rebuild to another host) has a default implementation in the compute manager which just relies on the existing spawn() and destroy() virt driver methods. I\u0027m not sure what you actually need to change to support this.","commit_id":"75ee6bb9808855954cbd88b49f1f4780d37022d0"}]}